[PATCH] mtd: rawnand: micron: Fix support for on-die ECC

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



It looks like the NAND_STATUS_FAIL bit is sticky after an ECC failure,
which leads all READ operations following the failing one to report
an ECC failure. Reset the chip to clear the NAND_STATUS_FAIL bit.

Note that this behavior is not document in the datasheet, but resetting
the chip is the only solution we found to fix the problem.

Fixes: 9748e1d87573 ("mtd: nand: add support for Micron on-die ECC")
Cc: <stable@xxxxxxxxxxxxxxx>
Signed-off-by: Boris Brezillon <boris.brezillon@xxxxxxxxxxx>
Cc: Thomas Petazzoni <thomas.petazzoni@xxxxxxxxxxx>
Cc: Bean Huo <beanhuo@xxxxxxxxxx>
Cc: Peter Pan <peterpandong@xxxxxxxxxx>
---
Peter, Bean,

Can you confirm this behavior, or ask someone in Micron who can confirm
it? Also, if a RESET is actually needed, it would be good to update the
datasheet accordingly. And if that's not the case, can you explain why
the NAND_STATUS_FAIL bit is stuck and how to clear it (I tried a 0x00
command, A.K.A. READ STATUS EXIT, but it does not clear this bit, ERASE
and PROGRAM seem to clear the bit, but that's clearly not the kind of
operation I can do when the user asks for a READ)?

Thanks,

Boris
---
 drivers/mtd/nand/raw/nand_micron.c | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

diff --git a/drivers/mtd/nand/raw/nand_micron.c b/drivers/mtd/nand/raw/nand_micron.c
index 0af45b134c0c..a915f568f6a3 100644
--- a/drivers/mtd/nand/raw/nand_micron.c
+++ b/drivers/mtd/nand/raw/nand_micron.c
@@ -153,6 +153,23 @@ micron_nand_read_page_on_die_ecc(struct mtd_info *mtd, struct nand_chip *chip,
 		ret = nand_read_data_op(chip, chip->oob_poi, mtd->oobsize,
 					false);
 
+	/*
+	 * Looks like the NAND_STATUS_FAIL bit is sticky after an ECC failure,
+	 * which leads all READ operations following the failing one to report
+	 * an ECC failure.
+	 * Reset the chip to clear it.
+	 *
+	 * Note that this behavior is not document in the datasheet, but
+	 * resetting the chip is the only solution we found to clear this bit.
+	 */
+	if (status & NAND_STATUS_FAIL) {
+		int cs = page >> (chip->chip_shift - chip->page_shift);
+
+		chip->select_chip(mtd, -1);
+		nand_reset(chip, cs);
+		chip->select_chip(mtd, cs);
+	}
+
 out:
 	micron_nand_on_die_ecc_setup(chip, false);
 
-- 
2.14.1




[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux