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

2016-04-09 Thread Rafał Miłecki 
On 9 April 2016 at 01:40, kbuild test robot  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?

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

2016-04-09 Thread Rafał Miłecki 
On 9 April 2016 at 01:40, kbuild test robot  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?

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

2016-04-08 Thread kbuild test robot 
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 

8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  22  #include 

8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  23  #include 

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_FMICSR0x00
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_SMADDR0xb4
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  30  #define 
REG_SMDATA0xb8
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 
:: CC: David Woodhouse 

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


.config.gz
Description: Binary data

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

2016-04-08 Thread kbuild test robot 
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 

8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  22  #include 

8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  23  #include 

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_FMICSR0x00
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_SMADDR0xb4
8bff82cb drivers/mtd/nand/w90p910_nand.c Wan ZongShun 2009-07-10  30  #define 
REG_SMDATA0xb8
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 
:: CC: David Woodhouse 

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


.config.gz
Description: Binary data

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

2016-04-08 Thread Rafał Miłecki 
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 
---
 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

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

2016-04-08 Thread Rafał Miłecki 
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 
---
 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