tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/driver-core.git debugfs_remove_return_value head: 5187c2360ee1d023078e4302dad32fda1e895772 commit: 72f2bf74b31aae983fb200aa7e84a05943bf27fc [2/8] debugfs: remove return value of debugfs_create_bool() config: i386-allyesconfig (attached as .config) compiler: gcc-9 (Debian 9.3.0-15) 9.3.0 reproduce (this is a W=1 build): # https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/driver-core.git/commit/?id=72f2bf74b31aae983fb200aa7e84a05943bf27fc git remote add driver-core https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/driver-core.git git fetch --no-tags driver-core debugfs_remove_return_value git checkout 72f2bf74b31aae983fb200aa7e84a05943bf27fc # save the attached .config to linux build tree make W=1 ARCH=i386
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <l...@intel.com>
All errors (new ones prefixed by >>):
drivers/net/wireless/broadcom/b43legacy/debugfs.c: In function
'b43legacy_add_dynamic_debug':
>> drivers/net/wireless/broadcom/b43legacy/debugfs.c:354:28: error: void value
>> not ignored as it ought to be
354 | e->dyn_debug_dentries[id] = \
| ^
drivers/net/wireless/broadcom/b43legacy/debugfs.c:359:2: note: in expansion
of macro 'add_dyn_dbg'
359 | add_dyn_dbg("debug_xmitpower", B43legacy_DBG_XMITPOWER, false);
| ^~~~~~~~~~~
>> drivers/net/wireless/broadcom/b43legacy/debugfs.c:354:28: error: void value
>> not ignored as it ought to be
354 | e->dyn_debug_dentries[id] = \
| ^
drivers/net/wireless/broadcom/b43legacy/debugfs.c:360:2: note: in expansion
of macro 'add_dyn_dbg'
360 | add_dyn_dbg("debug_dmaoverflow", B43legacy_DBG_DMAOVERFLOW, false);
| ^~~~~~~~~~~
>> drivers/net/wireless/broadcom/b43legacy/debugfs.c:354:28: error: void value
>> not ignored as it ought to be
354 | e->dyn_debug_dentries[id] = \
| ^
drivers/net/wireless/broadcom/b43legacy/debugfs.c:361:2: note: in expansion
of macro 'add_dyn_dbg'
361 | add_dyn_dbg("debug_dmaverbose", B43legacy_DBG_DMAVERBOSE, false);
| ^~~~~~~~~~~
>> drivers/net/wireless/broadcom/b43legacy/debugfs.c:354:28: error: void value
>> not ignored as it ought to be
354 | e->dyn_debug_dentries[id] = \
| ^
drivers/net/wireless/broadcom/b43legacy/debugfs.c:362:2: note: in expansion
of macro 'add_dyn_dbg'
362 | add_dyn_dbg("debug_pwork_fast", B43legacy_DBG_PWORK_FAST, false);
| ^~~~~~~~~~~
>> drivers/net/wireless/broadcom/b43legacy/debugfs.c:354:28: error: void value
>> not ignored as it ought to be
354 | e->dyn_debug_dentries[id] = \
| ^
drivers/net/wireless/broadcom/b43legacy/debugfs.c:363:2: note: in expansion
of macro 'add_dyn_dbg'
363 | add_dyn_dbg("debug_pwork_stop", B43legacy_DBG_PWORK_STOP, false);
| ^~~~~~~~~~~
vim +354 drivers/net/wireless/broadcom/b43legacy/debugfs.c
75388acd0cd827 drivers/net/wireless/b43legacy/debugfs.c Larry Finger
2007-09-25 347
75388acd0cd827 drivers/net/wireless/b43legacy/debugfs.c Larry Finger
2007-09-25 348 static void b43legacy_add_dynamic_debug(struct
b43legacy_wldev *dev)
75388acd0cd827 drivers/net/wireless/b43legacy/debugfs.c Larry Finger
2007-09-25 349 {
75388acd0cd827 drivers/net/wireless/b43legacy/debugfs.c Larry Finger
2007-09-25 350 struct b43legacy_dfsentry *e = dev->dfsentry;
75388acd0cd827 drivers/net/wireless/b43legacy/debugfs.c Larry Finger
2007-09-25 351
75388acd0cd827 drivers/net/wireless/b43legacy/debugfs.c Larry Finger
2007-09-25 352 #define add_dyn_dbg(name, id, initstate) do {
\
75388acd0cd827 drivers/net/wireless/b43legacy/debugfs.c Larry Finger
2007-09-25 353 e->dyn_debug[id] = (initstate);
\
32b4ebfe7f12c2 drivers/net/wireless/broadcom/b43legacy/debugfs.c Greg
Kroah-Hartman 2019-01-22 @354 e->dyn_debug_dentries[id] =
\
32b4ebfe7f12c2 drivers/net/wireless/broadcom/b43legacy/debugfs.c Greg
Kroah-Hartman 2019-01-22 355 debugfs_create_bool(name, 0600,
e->subdir, \
75388acd0cd827 drivers/net/wireless/b43legacy/debugfs.c Larry Finger
2007-09-25 356 &(e->dyn_debug[id]));
\
75388acd0cd827 drivers/net/wireless/b43legacy/debugfs.c Larry Finger
2007-09-25 357 } while (0)
75388acd0cd827 drivers/net/wireless/b43legacy/debugfs.c Larry Finger
2007-09-25 358
621a5f7ad9cd1c drivers/net/wireless/b43legacy/debugfs.c Viresh Kumar
2015-09-26 359 add_dyn_dbg("debug_xmitpower", B43legacy_DBG_XMITPOWER,
false);
621a5f7ad9cd1c drivers/net/wireless/b43legacy/debugfs.c Viresh Kumar
2015-09-26 360 add_dyn_dbg("debug_dmaoverflow",
B43legacy_DBG_DMAOVERFLOW, false);
621a5f7ad9cd1c drivers/net/wireless/b43legacy/debugfs.c Viresh Kumar
2015-09-26 361 add_dyn_dbg("debug_dmaverbose",
B43legacy_DBG_DMAVERBOSE, false);
621a5f7ad9cd1c drivers/net/wireless/b43legacy/debugfs.c Viresh Kumar
2015-09-26 362 add_dyn_dbg("debug_pwork_fast",
B43legacy_DBG_PWORK_FAST, false);
621a5f7ad9cd1c drivers/net/wireless/b43legacy/debugfs.c Viresh Kumar
2015-09-26 363 add_dyn_dbg("debug_pwork_stop",
B43legacy_DBG_PWORK_STOP, false);
75388acd0cd827 drivers/net/wireless/b43legacy/debugfs.c Larry Finger
2007-09-25 364
:::::: The code at line 354 was first introduced by commit
:::::: 32b4ebfe7f12c2481a2aea8372683142d1341a4e b43legacy: no need to check
return value of debugfs_create functions
:::::: TO: Greg Kroah-Hartman <gre...@linuxfoundation.org>
:::::: CC: Kalle Valo <kv...@codeaurora.org>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org
.config.gz
Description: application/gzip
_______________________________________________ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
