nand_ecc_is_strong_enough() doesn't trigger

Sascha Hauer
2024-04-11T10:07:00Z

Hi,

I am currently working with a NAND chip requiring 4 bit correction per
512 byte data while my NAND driver only supports 1 bit correction, so
I would assume this to trigger:

/* ECC sanity check: warn if it's too weak */
if (!nand_ecc_is_strong_enough(&chip->base))
pr_warn("WARNING: %s: the ECC used on your system (%db/%dB) is too weak compared to the one required by the NAND chip (%db/%dB)\n",
mtd->name, chip->ecc.strength, chip->ecc.size,
nanddev_get_ecc_requirements(&chip->base)->strength,
nanddev_get_ecc_requirements(&chip->base)->step_size);

nand_ecc_is_strong_enough() works on nand->ecc.ctx.conf.step_size, but
this only ever seems to be initialized by the various ecc-sw-*
algorithms.

This was changed in 8c126720fe10 ("mtd: rawnand: Use the ECC framework
nand_ecc_is_strong_enough() helper"). Before this patch the initialized
values in &(struct nand_chip *chip)->ecc were used (and which are still
used in the warning message above).

Do you have a direction how this should be fixed?

Thanks
Sascha

--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |

______________________________________________________
Linux MTD discussion mailing list