[PATCH v2] mtd: spinand: micron: add support for MT29F2G01AAAED

kernel test robot lkp at intel.com
Sun Aug 23 08:52:11 EDT 2020


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 <lkp at 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-all@lists.01.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: .config.gz
Type: application/gzip
Size: 29289 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-mtd/attachments/20200823/d9496d46/attachment-0001.gz>


More information about the linux-mtd mailing list