]> git.baikalelectronics.ru Git - uboot.git/commit
mtd: rawnand: pxa3xx: fix 2kiB pages with 8b strength chips layout
authorMiquel Raynal <miquel.raynal@bootlin.com>
Thu, 11 Oct 2018 15:45:44 +0000 (17:45 +0200)
committerJagan Teki <jagan@amarulasolutions.com>
Thu, 22 Nov 2018 06:08:13 +0000 (11:38 +0530)
commit22b0ed27b0e1790f1b8bf1bde14cc656020d1acc
tree37c71e8f2a9780fa06276cdb166a1c87b7a0cf96
parent20a1ff2e543e0294a970bccc0023472ff2df9dd0
mtd: rawnand: pxa3xx: fix 2kiB pages with 8b strength chips layout

The initial layout for such NAND chips was the following:

+----------------------------------------------------------------------------+
| 1024 (data) | 30 (ECC) | 1024 (data) | 30 (ECC) | 32 (free OOB) | 30 (ECC) |
+----------------------------------------------------------------------------+

This layout has a weakness: reading empty pages trigger ECC errors
(this is expected), but the hardware ECC engine tries to correct the
data anyway and creates itself bitflips, hence bitflips are detected
in erased pages while actually there are none in the NAND chip.

Two solutions have been found at the same time. One was to enlarge the
free OOB area to 64 bytes, changing the layout to be:

+----------------------------------------------------------------------------+
| 1024 (data) | 30 (ECC) | 1024 (data) | 30 (ECC) | 64 (free OOB) | 30 (ECC) |
+----------------------------------------------------------------------------+
                                                    ^^

The very big drawbacks of this solution are:
1/ It prevents booting from NAND.
2/ The current Linux driver (marvell_nand) does not have such problem
because it already re-reads possible empty pages in raw mode before
checking for bitflips. Using different layouts in U-Boot and Linux
would simply not work.

As this driver does support raw reads now and uses it to check for
empty pages, let's forget about this broken hack and return to the
initial layout with only 32 free OOB bytes.

Fixes: a780a29127 ("mtd: nand: pxa3xx: add support for 2KB 8-bit flash")
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Acked-by: Jagan Teki <jagan@openedev.com>
drivers/mtd/nand/raw/pxa3xx_nand.c