[PATCH v3] mtd: spinand: micron: correct bitmask for ecc status

Martin Kurbanov mmkurbanov at sberdevices.ru
Tue Sep 5 07:56:37 PDT 2023


Valid bitmask is 0x70 in the status register.

Fixes: a508e8875e13 ("mtd: spinand: Add initial support for Micron MT29F2G01ABAGD")
Signed-off-by: Martin Kurbanov <mmkurbanov at sberdevices.ru>
Reviewed-by: Frieder Schrempf <frieder.schrempf at kontron.de>
---
Changes v3 since v2 at [1]:
  - Remove the commit for fixing OOB layout.
  - Add Fixes tag.

Changes v2 since v1 at [2]:
  - Split into two individual patches.
  - Remove the fix for using only non-protected ECC bytes from OOB area.

Links:
  [1] https://lore.kernel.org/all/20230822122534.872646-1-mmkurbanov@sberdevices.ru/
  [2] https://lore.kernel.org/all/20230815161024.810729-1-mmkurbanov@sberdevices.ru/

 drivers/mtd/nand/spi/micron.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mtd/nand/spi/micron.c b/drivers/mtd/nand/spi/micron.c
index 50b7295bc922..12601bc4227a 100644
--- a/drivers/mtd/nand/spi/micron.c
+++ b/drivers/mtd/nand/spi/micron.c
@@ -12,7 +12,7 @@

 #define SPINAND_MFR_MICRON		0x2c

-#define MICRON_STATUS_ECC_MASK		GENMASK(7, 4)
+#define MICRON_STATUS_ECC_MASK		GENMASK(6, 4)
 #define MICRON_STATUS_ECC_NO_BITFLIPS	(0 << 4)
 #define MICRON_STATUS_ECC_1TO3_BITFLIPS	(1 << 4)
 #define MICRON_STATUS_ECC_4TO6_BITFLIPS	(3 << 4)
--
2.40.0




More information about the linux-mtd mailing list