All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] mtd: nand: nuc900: allow compiling with COMPILE_TEST
@ 2016-04-08 10:55 Rafał Miłecki
  2016-04-08 23:40 ` kbuild test robot
  0 siblings, 1 reply; 3+ messages in thread
From: Rafał Miłecki @ 2016-04-08 10:55 UTC (permalink / raw)
  To: Boris Brezillon
  Cc: Richard Weinberger, linux-mtd, Wan ZongShun,
	Rafał Miłecki, David Woodhouse, Brian Norris,
	open list

This driver doesn't seem to have any compile-time arch dependencies. I
was able to compile it on CONFIG_MIPS and CONFIG_ARM.

Signed-off-by: Rafał Miłecki <zajec5@gmail.com>
---
 drivers/mtd/nand/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mtd/nand/Kconfig b/drivers/mtd/nand/Kconfig
index f05e0e9..382e0f4 100644
--- a/drivers/mtd/nand/Kconfig
+++ b/drivers/mtd/nand/Kconfig
@@ -511,7 +511,7 @@ config MTD_NAND_SOCRATES
 
 config MTD_NAND_NUC900
 	tristate "Support for NAND on Nuvoton NUC9xx/w90p910 evaluation boards."
-	depends on ARCH_W90X900
+	depends on ARCH_W90X900 || COMPILE_TEST
 	help
 	  This enables the driver for the NAND Flash on evaluation board based
 	  on w90p910 / NUC9xx.
-- 
1.8.4.5

^ permalink raw reply related	[flat|nested] 3+ messages in thread
* Re: [PATCH] mtd: nand: nuc900: allow compiling with COMPILE_TEST
  2016-04-08 10:55 [PATCH] mtd: nand: nuc900: allow compiling with COMPILE_TEST Rafał Miłecki
@ 2016-04-08 23:40 ` kbuild test robot
  2016-04-09  8:42   ` Rafał Miłecki
  0 siblings, 1 reply; 3+ messages in thread
From: kbuild test robot @ 2016-04-08 23:40 UTC (permalink / raw)
  To: Rafał Miłecki
  Cc: kbuild-all, Boris Brezillon, Richard Weinberger, linux-mtd,
	Wan ZongShun, Rafał Miłecki, David Woodhouse,
	Brian Norris, open list

[-- Attachment #1: Type: text/plain, Size: 3690 bytes --]

Hi Rafał,

[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

vim +/SWRST +36 drivers/mtd/nand/nuc900_nand.c

8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  20  
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  21  #include <linux/mtd/mtd.h>
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  22  #include <linux/mtd/nand.h>
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  23  #include <linux/mtd/partitions.h>
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  24  
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  25  #define REG_FMICSR   	0x00
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  26  #define REG_SMCSR    	0xa0
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  27  #define REG_SMISR    	0xac
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  28  #define REG_SMCMD    	0xb0
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  29  #define REG_SMADDR   	0xb4
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  30  #define REG_SMDATA   	0xb8
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  31  
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  32  #define RESET_FMI	0x01
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  33  #define NAND_EN		0x08
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  34  #define READYBUSY	(0x01 << 18)
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  35  
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10 @36  #define SWRST		0x01
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  37  #define PSIZE		(0x01 << 3)
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  38  #define DMARWEN		(0x03 << 1)
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  39  #define BUSWID		(0x01 << 4)
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  40  #define ECC4EN		(0x01 << 5)
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  41  #define WP		(0x01 << 24)
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  42  #define NANDCS		(0x01 << 25)
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  43  #define ENDADDR		(0x01 << 31)
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  44  

:::::: The code at line 36 was first introduced by commit
:::::: 8bff82cbc30884fc52969608d090d874641e7196 mtd: add nand support for w90p910 (v2)

:::::: TO: Wan ZongShun <mcuos.com@gmail.com>
:::::: CC: David Woodhouse <David.Woodhouse@intel.com>

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

[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 40348 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread
* Re: [PATCH] mtd: nand: nuc900: allow compiling with COMPILE_TEST
  2016-04-08 23:40 ` kbuild test robot
@ 2016-04-09  8:42   ` Rafał Miłecki
  0 siblings, 0 replies; 3+ messages in thread
From: Rafał Miłecki @ 2016-04-09  8:42 UTC (permalink / raw)
  To: kbuild test robot, Steven Miao, adi-buildroot-devel
  Cc: kbuild-all, Boris Brezillon, Richard Weinberger, linux-mtd,
	Wan ZongShun, David Woodhouse, Brian Norris, open list

On 9 April 2016 at 01:40, kbuild test robot <lkp@intel.com> 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?

^ permalink raw reply	[flat|nested] 3+ messages in thread
end of thread, other threads:[~2016-04-09  8:42 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-04-08 10:55 [PATCH] mtd: nand: nuc900: allow compiling with COMPILE_TEST Rafał Miłecki
2016-04-08 23:40 ` kbuild test robot
2016-04-09  8:42   ` Rafał Miłecki

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.