[Top] [All Lists]

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

To: Rafał Miłecki <>
Subject: Re: [PATCH mips-fixes] MIPS: BCM47XX: Setup struct device for the SoC
From: kbuild test robot <>
Date: Mon, 7 Jan 2019 06:32:48 +0800
Cc:, Paul Burton <>, Ralf Baechle <>, James Hogan <>, Christoph Hellwig <>, Linus Walleij <>, Hauke Mehrtens <>,,,, Rafał Miłecki <>
In-reply-to: <>
List-archive: <>
List-help: <>
List-id: linux-mips <>
List-owner: <>
List-post: <>
List-software: Ecartis version 1.0.0
List-subscribe: <>
List-unsubscribe: <>
Original-recipient: rfc822;
References: <>
User-agent: Mutt/1.5.23 (2014-03-12)
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]

config: mips-bcm47xx_defconfig (attached as .config)
compiler: mipsel-linux-gnu-gcc (Debian 7.2.0-11) 7.2.0
        wget -O 
        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]
   cc1: all warnings being treated as errors

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

   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(;

0-DAY kernel test infrastructure                Open Source Technology Center                   Intel Corporation

Attachment: .config.gz
Description: application/gzip

<Prev in Thread] Current Thread [Next in Thread>