Hi Thirumalesha,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on linus/master]
[also build test ERROR on v5.9-rc1 next-20200821]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]

url:    
https://github.com/0day-ci/linux/commits/Thirumalesha-Narasimhappa/mtd-spinand-micron-add-support-for-MT29F2G01AAAED/20200823-191310
base:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git 
c3d8f220d01220a5b253e422be407d068dc65511
config: openrisc-randconfig-m031-20200823 (attached as .config)
compiler: or1k-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
        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
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross 
ARCH=openrisc 

If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <l...@intel.com>

All errors (new ones prefixed by >>):

>> drivers/mtd/nand/spi/micron.c:323: error: unterminated argument list 
>> invoking macro "SPINAND_INFO"
     323 | };
         | 
>> drivers/mtd/nand/spi/micron.c:288:2: error: 'SPINAND_INFO' undeclared here 
>> (not in a function)
     288 |  SPINAND_INFO("MT29F2G01AAAED",
         |  ^~~~~~~~~~~~
>> drivers/mtd/nand/spi/micron.c:323: error: expected '}' at end of input
     323 | };
         | 
   drivers/mtd/nand/spi/micron.c:184:59: note: to match this '{'
     184 | static const struct spinand_info micron_spinand_table[] = {
         |                                                           ^
   drivers/mtd/nand/spi/micron.c:184:34: warning: 'micron_spinand_table' 
defined but not used [-Wunused-const-variable=]
     184 | static const struct spinand_info micron_spinand_table[] = {
         |                                  ^~~~~~~~~~~~~~~~~~~~
   drivers/mtd/nand/spi/micron.c:163:12: warning: 
'mt29f2g01aaaed_ecc_get_status' defined but not used [-Wunused-function]
     163 | static int mt29f2g01aaaed_ecc_get_status(struct spinand_device 
*spinand,
         |            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   drivers/mtd/nand/spi/micron.c:118:39: warning: 'mt29f2g01aaaed_ooblayout' 
defined but not used [-Wunused-const-variable=]
     118 | static const struct mtd_ooblayout_ops mt29f2g01aaaed_ooblayout = {
         |                                       ^~~~~~~~~~~~~~~~~~~~~~~~
   In file included from drivers/mtd/nand/spi/micron.c:11:
   drivers/mtd/nand/spi/micron.c:60:28: warning: 
'update_cache_variants_mt29f2g01aaaed' defined but not used 
[-Wunused-const-variable=]
      60 | static SPINAND_OP_VARIANTS(update_cache_variants_mt29f2g01aaaed,
         |                            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/mtd/spinand.h:265:35: note: in definition of macro 
'SPINAND_OP_VARIANTS'
     265 |  const struct spinand_op_variants name = {   \
         |                                   ^~~~
   drivers/mtd/nand/spi/micron.c:57:28: warning: 
'write_cache_variants_mt29f2g01aaaed' defined but not used 
[-Wunused-const-variable=]
      57 | static SPINAND_OP_VARIANTS(write_cache_variants_mt29f2g01aaaed,
         |                            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/mtd/spinand.h:265:35: note: in definition of macro 
'SPINAND_OP_VARIANTS'
     265 |  const struct spinand_op_variants name = {   \
         |                                   ^~~~
   drivers/mtd/nand/spi/micron.c:51:28: warning: 
'read_cache_variants_mt29f2g01aaaed' defined but not used 
[-Wunused-const-variable=]
      51 | static SPINAND_OP_VARIANTS(read_cache_variants_mt29f2g01aaaed,
         |                            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/mtd/spinand.h:265:35: note: in definition of macro 
'SPINAND_OP_VARIANTS'
     265 |  const struct spinand_op_variants name = {   \
         |                                   ^~~~

# 
https://github.com/0day-ci/linux/commit/8dc175bd1853ebc961fea42976cffc290b5fbf22
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review 
Thirumalesha-Narasimhappa/mtd-spinand-micron-add-support-for-MT29F2G01AAAED/20200823-191310
git checkout 8dc175bd1853ebc961fea42976cffc290b5fbf22
vim +/SPINAND_INFO +323 drivers/mtd/nand/spi/micron.c

8dc175bd1853eb Thirumalesha Narasimhappa 2020-08-23  183  
a508e8875e135d Peter Pan                 2018-06-22  184  static const struct 
spinand_info micron_spinand_table[] = {
8511a3a9937e30 Shivamurthy Shastri       2020-03-11  185        /* M79A 2Gb 
3.3V */
f1541773af49ec Chuanhong Guo             2020-02-08  186        
SPINAND_INFO("MT29F2G01ABAGD",
f1541773af49ec Chuanhong Guo             2020-02-08  187                     
SPINAND_ID(SPINAND_READID_METHOD_OPCODE_DUMMY, 0x24),
377e517b5fa535 Boris Brezillon           2018-11-04  188                     
NAND_MEMORG(1, 2048, 128, 64, 2048, 40, 2, 1, 1),
a508e8875e135d Peter Pan                 2018-06-22  189                     
NAND_ECCREQ(8, 512),
a508e8875e135d Peter Pan                 2018-06-22  190                     
SPINAND_INFO_OP_VARIANTS(&read_cache_variants,
a508e8875e135d Peter Pan                 2018-06-22  191                        
                      &write_cache_variants,
a508e8875e135d Peter Pan                 2018-06-22  192                        
                      &update_cache_variants),
a508e8875e135d Peter Pan                 2018-06-22  193                     0,
d3137043440fb1 Shivamurthy Shastri       2020-03-11  194                     
SPINAND_ECCINFO(&micron_8_ooblayout,
d3137043440fb1 Shivamurthy Shastri       2020-03-11  195                        
             micron_8_ecc_get_status)),
a15335a17f4abf Shivamurthy Shastri       2020-03-11  196        /* M79A 2Gb 
1.8V */
a15335a17f4abf Shivamurthy Shastri       2020-03-11  197        
SPINAND_INFO("MT29F2G01ABBGD",
a15335a17f4abf Shivamurthy Shastri       2020-03-11  198                     
SPINAND_ID(SPINAND_READID_METHOD_OPCODE_DUMMY, 0x25),
a15335a17f4abf Shivamurthy Shastri       2020-03-11  199                     
NAND_MEMORG(1, 2048, 128, 64, 2048, 40, 2, 1, 1),
a15335a17f4abf Shivamurthy Shastri       2020-03-11  200                     
NAND_ECCREQ(8, 512),
a15335a17f4abf Shivamurthy Shastri       2020-03-11  201                     
SPINAND_INFO_OP_VARIANTS(&read_cache_variants,
a15335a17f4abf Shivamurthy Shastri       2020-03-11  202                        
                      &write_cache_variants,
a15335a17f4abf Shivamurthy Shastri       2020-03-11  203                        
                      &update_cache_variants),
a15335a17f4abf Shivamurthy Shastri       2020-03-11  204                     0,
a15335a17f4abf Shivamurthy Shastri       2020-03-11  205                     
SPINAND_ECCINFO(&micron_8_ooblayout,
a15335a17f4abf Shivamurthy Shastri       2020-03-11  206                        
             micron_8_ecc_get_status)),
a15335a17f4abf Shivamurthy Shastri       2020-03-11  207        /* M78A 1Gb 
3.3V */
a15335a17f4abf Shivamurthy Shastri       2020-03-11  208        
SPINAND_INFO("MT29F1G01ABAFD",
a15335a17f4abf Shivamurthy Shastri       2020-03-11  209                     
SPINAND_ID(SPINAND_READID_METHOD_OPCODE_DUMMY, 0x14),
a15335a17f4abf Shivamurthy Shastri       2020-03-11  210                     
NAND_MEMORG(1, 2048, 128, 64, 1024, 20, 1, 1, 1),
a15335a17f4abf Shivamurthy Shastri       2020-03-11  211                     
NAND_ECCREQ(8, 512),
a15335a17f4abf Shivamurthy Shastri       2020-03-11  212                     
SPINAND_INFO_OP_VARIANTS(&read_cache_variants,
a15335a17f4abf Shivamurthy Shastri       2020-03-11  213                        
                      &write_cache_variants,
a15335a17f4abf Shivamurthy Shastri       2020-03-11  214                        
                      &update_cache_variants),
a15335a17f4abf Shivamurthy Shastri       2020-03-11  215                     0,
a15335a17f4abf Shivamurthy Shastri       2020-03-11  216                     
SPINAND_ECCINFO(&micron_8_ooblayout,
a15335a17f4abf Shivamurthy Shastri       2020-03-11  217                        
             micron_8_ecc_get_status)),
a15335a17f4abf Shivamurthy Shastri       2020-03-11  218        /* M78A 1Gb 
1.8V */
a15335a17f4abf Shivamurthy Shastri       2020-03-11  219        
SPINAND_INFO("MT29F1G01ABAFD",
a15335a17f4abf Shivamurthy Shastri       2020-03-11  220                     
SPINAND_ID(SPINAND_READID_METHOD_OPCODE_DUMMY, 0x15),
a15335a17f4abf Shivamurthy Shastri       2020-03-11  221                     
NAND_MEMORG(1, 2048, 128, 64, 1024, 20, 1, 1, 1),
a15335a17f4abf Shivamurthy Shastri       2020-03-11  222                     
NAND_ECCREQ(8, 512),
a15335a17f4abf Shivamurthy Shastri       2020-03-11  223                     
SPINAND_INFO_OP_VARIANTS(&read_cache_variants,
a15335a17f4abf Shivamurthy Shastri       2020-03-11  224                        
                      &write_cache_variants,
a15335a17f4abf Shivamurthy Shastri       2020-03-11  225                        
                      &update_cache_variants),
a15335a17f4abf Shivamurthy Shastri       2020-03-11  226                     0,
a15335a17f4abf Shivamurthy Shastri       2020-03-11  227                     
SPINAND_ECCINFO(&micron_8_ooblayout,
a15335a17f4abf Shivamurthy Shastri       2020-03-11  228                        
             micron_8_ecc_get_status)),
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  229        /* M79A 4Gb 
3.3V */
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  230        
SPINAND_INFO("MT29F4G01ADAGD",
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  231                     
SPINAND_ID(SPINAND_READID_METHOD_OPCODE_DUMMY, 0x36),
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  232                     
NAND_MEMORG(1, 2048, 128, 64, 2048, 80, 2, 1, 2),
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  233                     
NAND_ECCREQ(8, 512),
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  234                     
SPINAND_INFO_OP_VARIANTS(&read_cache_variants,
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  235                        
                      &write_cache_variants,
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  236                        
                      &update_cache_variants),
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  237                     0,
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  238                     
SPINAND_ECCINFO(&micron_8_ooblayout,
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  239                        
             micron_8_ecc_get_status),
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  240                     
SPINAND_SELECT_TARGET(micron_select_target)),
a7e5daccc310c3 Shivamurthy Shastri       2020-03-11  241        /* M70A 4Gb 
3.3V */
a7e5daccc310c3 Shivamurthy Shastri       2020-03-11  242        
SPINAND_INFO("MT29F4G01ABAFD",
a7e5daccc310c3 Shivamurthy Shastri       2020-03-11  243                     
SPINAND_ID(SPINAND_READID_METHOD_OPCODE_DUMMY, 0x34),
a7e5daccc310c3 Shivamurthy Shastri       2020-03-11  244                     
NAND_MEMORG(1, 4096, 256, 64, 2048, 40, 1, 1, 1),
a7e5daccc310c3 Shivamurthy Shastri       2020-03-11  245                     
NAND_ECCREQ(8, 512),
a7e5daccc310c3 Shivamurthy Shastri       2020-03-11  246                     
SPINAND_INFO_OP_VARIANTS(&read_cache_variants,
a7e5daccc310c3 Shivamurthy Shastri       2020-03-11  247                        
                      &write_cache_variants,
a7e5daccc310c3 Shivamurthy Shastri       2020-03-11  248                        
                      &update_cache_variants),
a7e5daccc310c3 Shivamurthy Shastri       2020-03-11  249                     
SPINAND_HAS_CR_FEAT_BIT,
a7e5daccc310c3 Shivamurthy Shastri       2020-03-11  250                     
SPINAND_ECCINFO(&micron_8_ooblayout,
a7e5daccc310c3 Shivamurthy Shastri       2020-03-11  251                        
             micron_8_ecc_get_status)),
a7e5daccc310c3 Shivamurthy Shastri       2020-03-11  252        /* M70A 4Gb 
1.8V */
a7e5daccc310c3 Shivamurthy Shastri       2020-03-11  253        
SPINAND_INFO("MT29F4G01ABBFD",
a7e5daccc310c3 Shivamurthy Shastri       2020-03-11  254                     
SPINAND_ID(SPINAND_READID_METHOD_OPCODE_DUMMY, 0x35),
a7e5daccc310c3 Shivamurthy Shastri       2020-03-11  255                     
NAND_MEMORG(1, 4096, 256, 64, 2048, 40, 1, 1, 1),
a7e5daccc310c3 Shivamurthy Shastri       2020-03-11  256                     
NAND_ECCREQ(8, 512),
a7e5daccc310c3 Shivamurthy Shastri       2020-03-11  257                     
SPINAND_INFO_OP_VARIANTS(&read_cache_variants,
a7e5daccc310c3 Shivamurthy Shastri       2020-03-11  258                        
                      &write_cache_variants,
a7e5daccc310c3 Shivamurthy Shastri       2020-03-11  259                        
                      &update_cache_variants),
a7e5daccc310c3 Shivamurthy Shastri       2020-03-11  260                     
SPINAND_HAS_CR_FEAT_BIT,
a7e5daccc310c3 Shivamurthy Shastri       2020-03-11  261                     
SPINAND_ECCINFO(&micron_8_ooblayout,
a7e5daccc310c3 Shivamurthy Shastri       2020-03-11  262                        
             micron_8_ecc_get_status)),
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  263        /* M70A 8Gb 
3.3V */
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  264        
SPINAND_INFO("MT29F8G01ADAFD",
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  265                     
SPINAND_ID(SPINAND_READID_METHOD_OPCODE_DUMMY, 0x46),
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  266                     
NAND_MEMORG(1, 4096, 256, 64, 2048, 40, 1, 1, 2),
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  267                     
NAND_ECCREQ(8, 512),
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  268                     
SPINAND_INFO_OP_VARIANTS(&read_cache_variants,
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  269                        
                      &write_cache_variants,
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  270                        
                      &update_cache_variants),
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  271                     
SPINAND_HAS_CR_FEAT_BIT,
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  272                     
SPINAND_ECCINFO(&micron_8_ooblayout,
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  273                        
             micron_8_ecc_get_status),
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  274                     
SPINAND_SELECT_TARGET(micron_select_target)),
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  275        /* M70A 8Gb 
1.8V */
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  276        
SPINAND_INFO("MT29F8G01ADBFD",
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  277                     
SPINAND_ID(SPINAND_READID_METHOD_OPCODE_DUMMY, 0x47),
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  278                     
NAND_MEMORG(1, 4096, 256, 64, 2048, 40, 1, 1, 2),
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  279                     
NAND_ECCREQ(8, 512),
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  280                     
SPINAND_INFO_OP_VARIANTS(&read_cache_variants,
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  281                        
                      &write_cache_variants,
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  282                        
                      &update_cache_variants),
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  283                     
SPINAND_HAS_CR_FEAT_BIT,
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  284                     
SPINAND_ECCINFO(&micron_8_ooblayout,
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  285                        
             micron_8_ecc_get_status),
9f9ae0c253c1e0 Shivamurthy Shastri       2020-03-11  286                     
SPINAND_SELECT_TARGET(micron_select_target)),
8dc175bd1853eb Thirumalesha Narasimhappa 2020-08-23  287        /* M69A 2Gb 
3.3V */
8dc175bd1853eb Thirumalesha Narasimhappa 2020-08-23 @288        
SPINAND_INFO("MT29F2G01AAAED",
8dc175bd1853eb Thirumalesha Narasimhappa 2020-08-23  289                     
SPINAND_ID(SPINAND_READID_METHOD_OPCODE_DUMMY, 0x9F),
8dc175bd1853eb Thirumalesha Narasimhappa 2020-08-23  290                     
NAND_MEMORG(1, 2048, 64, 64, 2048, 80, 2, 1, 1),
8dc175bd1853eb Thirumalesha Narasimhappa 2020-08-23  291                     
NAND_ECCREQ(4, 512),
8dc175bd1853eb Thirumalesha Narasimhappa 2020-08-23  292                     
SPINAND_INFO_OP_VARIANTS(&read_cache_variants_mt29f2g01aaaed,
8dc175bd1853eb Thirumalesha Narasimhappa 2020-08-23  293                        
                      &write_cache_variants_mt29f2g01aaaed,
8dc175bd1853eb Thirumalesha Narasimhappa 2020-08-23  294                        
                      &update_cache_variants_mt29f2g01aaaed),
8dc175bd1853eb Thirumalesha Narasimhappa 2020-08-23  295                     0,
8dc175bd1853eb Thirumalesha Narasimhappa 2020-08-23  296                     
SPINAND_ECCINFO(&mt29f2g01aaaed_ooblayout,
8dc175bd1853eb Thirumalesha Narasimhappa 2020-08-23  297                        
             mt29f2g01aaaed_ecc_get_status),
a508e8875e135d Peter Pan                 2018-06-22  298  };
a508e8875e135d Peter Pan                 2018-06-22  299  
0bc68af9137dc3 Shivamurthy Shastri       2020-03-11  300  static int 
micron_spinand_init(struct spinand_device *spinand)
0bc68af9137dc3 Shivamurthy Shastri       2020-03-11  301  {
0bc68af9137dc3 Shivamurthy Shastri       2020-03-11  302        /*
0bc68af9137dc3 Shivamurthy Shastri       2020-03-11  303         * M70A device 
series enable Continuous Read feature at Power-up,
0bc68af9137dc3 Shivamurthy Shastri       2020-03-11  304         * which is not 
supported. Disable this bit to avoid any possible
0bc68af9137dc3 Shivamurthy Shastri       2020-03-11  305         * failure.
0bc68af9137dc3 Shivamurthy Shastri       2020-03-11  306         */
0bc68af9137dc3 Shivamurthy Shastri       2020-03-11  307        if 
(spinand->flags & SPINAND_HAS_CR_FEAT_BIT)
0bc68af9137dc3 Shivamurthy Shastri       2020-03-11  308                return 
spinand_upd_cfg(spinand, MICRON_CFG_CR, 0);
0bc68af9137dc3 Shivamurthy Shastri       2020-03-11  309  
0bc68af9137dc3 Shivamurthy Shastri       2020-03-11  310        return 0;
0bc68af9137dc3 Shivamurthy Shastri       2020-03-11  311  }
0bc68af9137dc3 Shivamurthy Shastri       2020-03-11  312  
a508e8875e135d Peter Pan                 2018-06-22  313  static const struct 
spinand_manufacturer_ops micron_spinand_manuf_ops = {
0bc68af9137dc3 Shivamurthy Shastri       2020-03-11  314        .init = 
micron_spinand_init,
a508e8875e135d Peter Pan                 2018-06-22  315  };
a508e8875e135d Peter Pan                 2018-06-22  316  
a508e8875e135d Peter Pan                 2018-06-22  317  const struct 
spinand_manufacturer micron_spinand_manufacturer = {
a508e8875e135d Peter Pan                 2018-06-22  318        .id = 
SPINAND_MFR_MICRON,
a508e8875e135d Peter Pan                 2018-06-22  319        .name = 
"Micron",
f1541773af49ec Chuanhong Guo             2020-02-08  320        .chips = 
micron_spinand_table,
f1541773af49ec Chuanhong Guo             2020-02-08  321        .nchips = 
ARRAY_SIZE(micron_spinand_table),
a508e8875e135d Peter Pan                 2018-06-22  322        .ops = 
&micron_spinand_manuf_ops,
a508e8875e135d Peter Pan                 2018-06-22 @323  };

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org

Attachment: .config.gz
Description: application/gzip

Reply via email to