Re: [PATCH] CFLAGS: add -Wunused-but-set-parameter

From	kbuild test robot <lkp@intel.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH] CFLAGS: add -Wunused-but-set-parameter
Date	2016-07-11 12:10 +0200
Message-ID	<rTGk3-5Co-47@gated-at.bofh.it> (permalink)
References	<rSplf-43R-3@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

[Multipart message — attachments visible in raw view] - view raw

Hi,

[auto build test WARNING on kbuild/for-next]
[also build test WARNING on v4.7-rc7 next-20160708]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Alexey-Dobriyan/CFLAGS-add-Wunused-but-set-parameter/20160708-055418
base:   https://git.kernel.org/pub/scm/linux/kernel/git/mmarek/kbuild.git for-next
config: m68k-sun3_defconfig (attached as .config)
compiler: m68k-linux-gcc (GCC) 4.9.0
reproduce:
        wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=m68k 

All warnings (new ones prefixed by >>):

   drivers/video/fbdev/core/fbmon.c: In function 'fb_edid_to_monspecs':
>> drivers/video/fbdev/core/fbmon.c:1497:67: warning: parameter 'specs' set but not used [-Wunused-but-set-parameter]
    void fb_edid_to_monspecs(unsigned char *edid, struct fb_monspecs *specs)
                                                                      ^

vim +/specs +1497 drivers/video/fbdev/core/fbmon.c

79089044 drivers/video/fbmon.c Steffen Trumtrar      2012-11-14  1481  
79089044 drivers/video/fbmon.c Steffen Trumtrar      2012-11-14  1482  	pr_debug("%s: got %dx%d display mode from %s\n",
79089044 drivers/video/fbmon.c Steffen Trumtrar      2012-11-14  1483  		of_node_full_name(np), vm.hactive, vm.vactive, np->name);
79089044 drivers/video/fbmon.c Steffen Trumtrar      2012-11-14  1484  	dump_fb_videomode(fb);
79089044 drivers/video/fbmon.c Steffen Trumtrar      2012-11-14  1485  
79089044 drivers/video/fbmon.c Steffen Trumtrar      2012-11-14  1486  	return 0;
79089044 drivers/video/fbmon.c Steffen Trumtrar      2012-11-14  1487  }
79089044 drivers/video/fbmon.c Steffen Trumtrar      2012-11-14  1488  EXPORT_SYMBOL_GPL(of_get_fb_videomode);
a38884f6 drivers/video/fbmon.c Tomi Valkeinen        2013-03-12  1489  #endif /* CONFIG_OF */
a38884f6 drivers/video/fbmon.c Tomi Valkeinen        2013-03-12  1490  #endif /* CONFIG_VIDEOMODE_HELPERS */
79089044 drivers/video/fbmon.c Steffen Trumtrar      2012-11-14  1491  
^1da177e drivers/video/fbmon.c Linus Torvalds        2005-04-16  1492  #else
^1da177e drivers/video/fbmon.c Linus Torvalds        2005-04-16  1493  int fb_parse_edid(unsigned char *edid, struct fb_var_screeninfo *var)
^1da177e drivers/video/fbmon.c Linus Torvalds        2005-04-16  1494  {
^1da177e drivers/video/fbmon.c Linus Torvalds        2005-04-16  1495  	return 1;
^1da177e drivers/video/fbmon.c Linus Torvalds        2005-04-16  1496  }
^1da177e drivers/video/fbmon.c Linus Torvalds        2005-04-16 @1497  void fb_edid_to_monspecs(unsigned char *edid, struct fb_monspecs *specs)
^1da177e drivers/video/fbmon.c Linus Torvalds        2005-04-16  1498  {
^1da177e drivers/video/fbmon.c Linus Torvalds        2005-04-16  1499  	specs = NULL;
^1da177e drivers/video/fbmon.c Linus Torvalds        2005-04-16  1500  }
e4105119 drivers/video/fbmon.c Guennadi Liakhovetski 2010-11-11  1501  void fb_edid_add_monspecs(unsigned char *edid, struct fb_monspecs *specs)
e4105119 drivers/video/fbmon.c Guennadi Liakhovetski 2010-11-11  1502  {
e4105119 drivers/video/fbmon.c Guennadi Liakhovetski 2010-11-11  1503  }
^1da177e drivers/video/fbmon.c Linus Torvalds        2005-04-16  1504  void fb_destroy_modedb(struct fb_videomode *modedb)
^1da177e drivers/video/fbmon.c Linus Torvalds        2005-04-16  1505  {

:::::: The code at line 1497 was first introduced by commit
:::::: 1da177e4c3f41524e886b7f1b8a0c1fc7321cac2 Linux-2.6.12-rc2

:::::: TO: Linus Torvalds <torvalds@ppc970.osdl.org>
:::::: CC: Linus Torvalds <torvalds@ppc970.osdl.org>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Thread

[PATCH] CFLAGS: add -Wunused-but-set-parameter Alexey Dobriyan <adobriyan@gmail.com> - 2016-07-07 23:50 +0200
  Re: [PATCH] CFLAGS: add -Wunused-but-set-parameter kbuild test robot <lkp@intel.com> - 2016-07-11 12:10 +0200
  Re: [PATCH] CFLAGS: add -Wunused-but-set-parameter Andrew Morton <akpm@linux-foundation.org> - 2016-07-12 23:30 +0200
    [PATCH v2] CFLAGS: add -Wunused-but-set-parameter Alexey Dobriyan <adobriyan@gmail.com> - 2016-07-13 21:10 +0200

csiph-web