mtd: rawnand: micron: handle on-die "ECC-off" devices correctly
authorMarco Felsch <m.felsch@pengutronix.de>
Tue, 30 Jul 2019 13:44:07 +0000 (15:44 +0200)
committerMiquel Raynal <miquel.raynal@bootlin.com>
Sat, 3 Aug 2019 00:00:01 +0000 (02:00 +0200)
commit8493b2a06fc5b77ef5c579dc32b12761f7b7a84c
tree81c8bca70f29f328f9d7721d6c982e99110bbd25
parent5f9e832c137075045d15cd6899ab0505cfb2ca4b
mtd: rawnand: micron: handle on-die "ECC-off" devices correctly

Some devices are not supposed to support on-die ECC but experience
shows that internal ECC machinery can actually be enabled through the
"SET FEATURE (EFh)" command, even if a read of the "READ ID Parameter
Tables" returns that it is not.

Currently, the driver checks the "READ ID Parameter" field directly
after having enabled the feature. If the check fails it returns
immediately but leaves the ECC on. When using buggy chips like
MT29F2G08ABAGA and MT29F2G08ABBGA, all future read/program cycles will
go through the on-die ECC, confusing the host controller which is
supposed to be the one handling correction.

To address this in a common way we need to turn off the on-die ECC
directly after reading the "READ ID Parameter" and before checking the
"ECC status".

Cc: stable@vger.kernel.org
Fixes: dbc44edbf833 ("mtd: rawnand: micron: Fix on-die ECC detection logic")
Signed-off-by: Marco Felsch <m.felsch@pengutronix.de>
Reviewed-by: Boris Brezillon <boris.brezillon@collabora.com>
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
drivers/mtd/nand/raw/nand_micron.c