CC: kbuild-...@lists.01.org TO: Miquel Raynal <miquel.ray...@bootlin.com>
tree: https://github.com/miquelraynal/linux-0day.git nand/next head: 08bcd2f7667339539d11dea12526af583e107def commit: ead4a01a866678226171b87053477cd07a53e16e [15/39] mtd: nand: ecc-hamming: Rename the exported functions :::::: branch date: 6 weeks ago :::::: commit date: 6 weeks ago config: x86_64-randconfig-m001-20201213 (attached as .config) compiler: gcc-9 (Debian 9.3.0-15) 9.3.0 If you fix the issue, kindly add following tag as appropriate Reported-by: kernel test robot <l...@intel.com> Reported-by: Dan Carpenter <dan.carpen...@oracle.com> smatch warnings: drivers/mtd/nand/ecc-sw-hamming.c:348 ecc_sw_hamming_calculate() error: uninitialized symbol 'rp17'. vim +/rp17 +348 drivers/mtd/nand/ecc-sw-hamming.c e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 282 e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 283 /* finally reduce par to 8 bits */ e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 284 par ^= (par >> 8); e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 285 par &= 0xff; e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 286 e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 287 /* d68156cfad0fe09 drivers/mtd/nand/nand_ecc.c Singh, Vimal 2008-08-23 288 * and calculate rp5..rp15..rp17 e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 289 * note that par = rp4 ^ rp5 and due to the commutative property e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 290 * of the ^ operator we can say: e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 291 * rp5 = (par ^ rp4); e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 292 * The & 0xff seems superfluous, but benchmarking learned that e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 293 * leaving it out gives slightly worse results. No idea why, probably e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 294 * it has to do with the way the pipeline in pentium is organized. e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 295 */ e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 296 rp5 = (par ^ rp4) & 0xff; e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 297 rp7 = (par ^ rp6) & 0xff; e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 298 rp9 = (par ^ rp8) & 0xff; e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 299 rp11 = (par ^ rp10) & 0xff; e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 300 rp13 = (par ^ rp12) & 0xff; e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 301 rp15 = (par ^ rp14) & 0xff; d68156cfad0fe09 drivers/mtd/nand/nand_ecc.c Singh, Vimal 2008-08-23 302 if (eccsize_mult == 2) d68156cfad0fe09 drivers/mtd/nand/nand_ecc.c Singh, Vimal 2008-08-23 303 rp17 = (par ^ rp16) & 0xff; e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 304 e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 305 /* 7854d3f7495b11b drivers/mtd/nand/nand_ecc.c Brian Norris 2011-06-23 306 * Finally calculate the ECC bits. e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 307 * Again here it might seem that there are performance optimisations e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 308 * possible, but benchmarks showed that on the system this is developed e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 309 * the code below is the fastest e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 310 */ 309600c14e36d0e drivers/mtd/nand/raw/nand_ecc.c Boris Brezillon 2018-09-04 311 if (sm_order) { 309600c14e36d0e drivers/mtd/nand/raw/nand_ecc.c Boris Brezillon 2018-09-04 312 code[0] = (invparity[rp7] << 7) | (invparity[rp6] << 6) | 309600c14e36d0e drivers/mtd/nand/raw/nand_ecc.c Boris Brezillon 2018-09-04 313 (invparity[rp5] << 5) | (invparity[rp4] << 4) | 309600c14e36d0e drivers/mtd/nand/raw/nand_ecc.c Boris Brezillon 2018-09-04 314 (invparity[rp3] << 3) | (invparity[rp2] << 2) | 309600c14e36d0e drivers/mtd/nand/raw/nand_ecc.c Boris Brezillon 2018-09-04 315 (invparity[rp1] << 1) | (invparity[rp0]); 309600c14e36d0e drivers/mtd/nand/raw/nand_ecc.c Boris Brezillon 2018-09-04 316 code[1] = (invparity[rp15] << 7) | (invparity[rp14] << 6) | 309600c14e36d0e drivers/mtd/nand/raw/nand_ecc.c Boris Brezillon 2018-09-04 317 (invparity[rp13] << 5) | (invparity[rp12] << 4) | 309600c14e36d0e drivers/mtd/nand/raw/nand_ecc.c Boris Brezillon 2018-09-04 318 (invparity[rp11] << 3) | (invparity[rp10] << 2) | 309600c14e36d0e drivers/mtd/nand/raw/nand_ecc.c Boris Brezillon 2018-09-04 319 (invparity[rp9] << 1) | (invparity[rp8]); 309600c14e36d0e drivers/mtd/nand/raw/nand_ecc.c Boris Brezillon 2018-09-04 320 } else { 309600c14e36d0e drivers/mtd/nand/raw/nand_ecc.c Boris Brezillon 2018-09-04 321 code[1] = (invparity[rp7] << 7) | (invparity[rp6] << 6) | 309600c14e36d0e drivers/mtd/nand/raw/nand_ecc.c Boris Brezillon 2018-09-04 322 (invparity[rp5] << 5) | (invparity[rp4] << 4) | 309600c14e36d0e drivers/mtd/nand/raw/nand_ecc.c Boris Brezillon 2018-09-04 323 (invparity[rp3] << 3) | (invparity[rp2] << 2) | 309600c14e36d0e drivers/mtd/nand/raw/nand_ecc.c Boris Brezillon 2018-09-04 324 (invparity[rp1] << 1) | (invparity[rp0]); 309600c14e36d0e drivers/mtd/nand/raw/nand_ecc.c Boris Brezillon 2018-09-04 325 code[0] = (invparity[rp15] << 7) | (invparity[rp14] << 6) | 309600c14e36d0e drivers/mtd/nand/raw/nand_ecc.c Boris Brezillon 2018-09-04 326 (invparity[rp13] << 5) | (invparity[rp12] << 4) | 309600c14e36d0e drivers/mtd/nand/raw/nand_ecc.c Boris Brezillon 2018-09-04 327 (invparity[rp11] << 3) | (invparity[rp10] << 2) | 309600c14e36d0e drivers/mtd/nand/raw/nand_ecc.c Boris Brezillon 2018-09-04 328 (invparity[rp9] << 1) | (invparity[rp8]); 309600c14e36d0e drivers/mtd/nand/raw/nand_ecc.c Boris Brezillon 2018-09-04 329 } 309600c14e36d0e drivers/mtd/nand/raw/nand_ecc.c Boris Brezillon 2018-09-04 330 d68156cfad0fe09 drivers/mtd/nand/nand_ecc.c Singh, Vimal 2008-08-23 331 if (eccsize_mult == 1) e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 332 code[2] = e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 333 (invparity[par & 0xf0] << 7) | e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 334 (invparity[par & 0x0f] << 6) | e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 335 (invparity[par & 0xcc] << 5) | e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 336 (invparity[par & 0x33] << 4) | e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 337 (invparity[par & 0xaa] << 3) | e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 338 (invparity[par & 0x55] << 2) | e6cf5df1838c28b drivers/mtd/nand/nand_ecc.c frans 2008-08-15 339 3; d68156cfad0fe09 drivers/mtd/nand/nand_ecc.c Singh, Vimal 2008-08-23 340 else d68156cfad0fe09 drivers/mtd/nand/nand_ecc.c Singh, Vimal 2008-08-23 341 code[2] = d68156cfad0fe09 drivers/mtd/nand/nand_ecc.c Singh, Vimal 2008-08-23 342 (invparity[par & 0xf0] << 7) | d68156cfad0fe09 drivers/mtd/nand/nand_ecc.c Singh, Vimal 2008-08-23 343 (invparity[par & 0x0f] << 6) | d68156cfad0fe09 drivers/mtd/nand/nand_ecc.c Singh, Vimal 2008-08-23 344 (invparity[par & 0xcc] << 5) | d68156cfad0fe09 drivers/mtd/nand/nand_ecc.c Singh, Vimal 2008-08-23 345 (invparity[par & 0x33] << 4) | d68156cfad0fe09 drivers/mtd/nand/nand_ecc.c Singh, Vimal 2008-08-23 346 (invparity[par & 0xaa] << 3) | d68156cfad0fe09 drivers/mtd/nand/nand_ecc.c Singh, Vimal 2008-08-23 347 (invparity[par & 0x55] << 2) | d68156cfad0fe09 drivers/mtd/nand/nand_ecc.c Singh, Vimal 2008-08-23 @348 (invparity[rp17] << 1) | d68156cfad0fe09 drivers/mtd/nand/nand_ecc.c Singh, Vimal 2008-08-23 349 (invparity[rp16] << 0); ead4a01a8666782 drivers/mtd/nand/ecc-sw-hamming.c Miquel Raynal 2020-09-30 350 ead4a01a8666782 drivers/mtd/nand/ecc-sw-hamming.c Miquel Raynal 2020-09-30 351 return 0; 1c63aca32903efc drivers/mtd/nand/nand_ecc.c Akinobu Mita 2009-10-22 352 } ead4a01a8666782 drivers/mtd/nand/ecc-sw-hamming.c Miquel Raynal 2020-09-30 353 EXPORT_SYMBOL(ecc_sw_hamming_calculate); 1c63aca32903efc drivers/mtd/nand/nand_ecc.c Akinobu Mita 2009-10-22 354 :::::: The code at line 348 was first introduced by commit :::::: d68156cfad0fe09201dd049fff167a8a881427ad [MTD] [NAND] nand_ecc.c: adding support for 512 byte ecc :::::: TO: Singh, Vimal <vimalsi...@ti.com> :::::: CC: David Woodhouse <david.woodho...@intel.com> --- 0-DAY CI Kernel Test Service, Intel Corporation https://lists.01.org/hyperkitty/list/kbuild-...@lists.01.org
.config.gz
Description: application/gzip
_______________________________________________ kbuild mailing list -- kbuild@lists.01.org To unsubscribe send an email to kbuild-le...@lists.01.org
