Re: [PATCH] mtd: nand: nuc900: allow compiling with COMPILE_TEST

From: RafaÅ MiÅecki
Date: Sat Apr 09 2016 - 04:42:24 EST


On 9 April 2016 at 01:40, kbuild test robot <lkp@xxxxxxxxx> wrote:
> [auto build test WARNING on mtd/master]
> [also build test WARNING on v4.6-rc2 next-20160408]
> [if your patch is applied to the wrong git tree, please drop us a note to help improving the system]
>
> url: https://github.com/0day-ci/linux/commits/Rafa-Mi-ecki/mtd-nand-nuc900-allow-compiling-with-COMPILE_TEST/20160408-185814
> base: git://git.infradead.org/linux-mtd.git master
> config: blackfin-allyesconfig (attached as .config)
> 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=blackfin
>
> All warnings (new ones prefixed by >>):
>
>>> drivers/mtd/nand/nuc900_nand.c:36:0: warning: "SWRST" redefined [enabled by default]
> arch/blackfin/mach-bf533/include/mach/defBF532.h:25:0: note: this is the location of the previous definition

Uh, I'm not sure what to do about this. The name SWRST sounds a bit
generic so I guess it shouldn't be defined by any common header.

In case of blackfin it's part of the System Interrupt Controller and
other defines are prefixed with SIC_. Maybe we should rename
blackfin's SWRST to SIC_SWRST?