mtd: rawnand: arasan: Include ECC syndrome along with in-band data while checking...
authorAmit Kumar Mahapatra <amit.kumar-mahapatra@amd.com>
Wed, 27 Sep 2023 05:56:21 +0000 (11:26 +0530)
committerMiquel Raynal <miquel.raynal@bootlin.com>
Mon, 16 Oct 2023 09:16:49 +0000 (11:16 +0200)
commitf693b6485e251ddf849d862cf6406a4dfc0143b3
tree3e88ec94978776255b19374318630a611739c1f7
parent48919c6c48380aa1aec4243c7e6ad39e89d78539
mtd: rawnand: arasan: Include ECC syndrome along with in-band data while checking for ECC failure

Following an ECC failure condition upon page reads, we shall distinguish
between a real ECC failure and an empty page. This is handled with a call
to nand_check_erased_ecc_chunk() which looks at the data and counts the
number of bits which are not 'ones'. If we get less zeros than the ECC
strength, we assume the page was erased and we are in the presence of
natural bitflips. Otherwise, if we are above, we assume some data was
written and the ECC engine could not recover it all, so we report an ECC
failure.

In order for this logic to be as close as the reality as we can (this is
already a simplified condition but we can hardly be more precise), we
should check all the data that is covered by the ECC step not only the
in-band data, so we should also include the ECC syndrome in the check.

Fixes: 88ffef1b65cf ("mtd: rawnand: arasan: Support the hardware BCH ECC engine")
Signed-off-by: Amit Kumar Mahapatra <amit.kumar-mahapatra@amd.com>
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Link: https://lore.kernel.org/linux-mtd/20230927055621.2906454-1-amit.kumar-mahapatra@amd.com
drivers/mtd/nand/raw/arasan-nand-controller.c