Re: [PATCH mips-fixes] MIPS: BCM47XX: Setup struct device for the SoC

kbuild test robot <[email protected]>
Newsgroups gmane.linux.kernel,gmane.linux.ports.mips.general,gmane.linux.kernel.wireless.general
Message-ID <201901070603.fp3up0lD%[email protected]>
Hi Rafał,

I love your patch! Yet something to improve:

[auto build test ERROR on linus/master]
[also build test ERROR on v4.20 next-20190103]
[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/Rafa-Mi-ecki/MIPS-BCM47XX-Setup-struct-device-for-the-SoC/20190102-210611
config: mips-bcm47xx_defconfig (attached as .config)
compiler: mipsel-linux-gnu-gcc (Debian 7.2.0-11) 7.2.0
reproduce:
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        GCC_VERSION=7.2.0 make.cross ARCH=mips 

All errors (new ones prefixed by >>):

   arch/mips/bcm47xx/setup.c: In function 'bcm47xx_register_bus_complete':
>> arch/mips/bcm47xx/setup.c:267:3: error: ignoring return value of 'device_register', declared with attribute warn_unused_result [-Werror=unused-result]
      device_register(bcm47xx_bus.bcma.dev);
      ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   cc1: all warnings being treated as errors

vim +/device_register +267 arch/mips/bcm47xx/setup.c

   256	
   257	static int __init bcm47xx_register_bus_complete(void)
   258	{
   259		switch (bcm47xx_bus_type) {
   260	#ifdef CONFIG_BCM47XX_SSB
   261		case BCM47XX_BUS_TYPE_SSB:
   262			/* Nothing to do */
   263			break;
   264	#endif
   265	#ifdef CONFIG_BCM47XX_BCMA
   266		case BCM47XX_BUS_TYPE_BCMA:
 > 267			device_register(bcm47xx_bus.bcma.dev);

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation
.config.gz (application/gzip, 15.8 KB) - not displayed
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.