summaryrefslogtreecommitdiffstats
path: root/drivers/mtd
diff options
context:
space:
mode:
authorBoris Brezillon2016-06-06 13:59:12 +0200
committerBoris Brezillon2016-07-11 08:40:12 +0200
commit872164e41fc8dee154e5f5d22580b34e198eed69 (patch)
tree86c7fd439d623bab6921d6ec06e3244e9f09f33c /drivers/mtd
parentmtd: nand: sunxi: Fix OOB bytes retrieval in read_chunks_dma() (diff)
downloadkernel-qcow2-linux-872164e41fc8dee154e5f5d22580b34e198eed69.tar.gz
kernel-qcow2-linux-872164e41fc8dee154e5f5d22580b34e198eed69.tar.xz
kernel-qcow2-linux-872164e41fc8dee154e5f5d22580b34e198eed69.zip
mtd: nand: sunxi: prefer 1k ECC blocks when applicable
Switching to 1k ECC blocks when possible provides better resistance against concentrated bitflips. Say you have those two configurations: 1/ 16bits/512bytes 2/ 32bits/1024bytes Both of them require the same amount of ECC bytes (only true for this specific engine), but the second config allows you to correct the case where most of your bitflips are concentrated in a single 512bytes portion. This fact makes the 1k ECC block size more advantageous than the 512bytes one. Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com>
Diffstat (limited to 'drivers/mtd')
-rw-r--r--drivers/mtd/nand/sunxi_nand.c6
1 files changed, 6 insertions, 0 deletions
diff --git a/drivers/mtd/nand/sunxi_nand.c b/drivers/mtd/nand/sunxi_nand.c
index b9a2e5d28754..bb440b91096f 100644
--- a/drivers/mtd/nand/sunxi_nand.c
+++ b/drivers/mtd/nand/sunxi_nand.c
@@ -1786,6 +1786,12 @@ static int sunxi_nand_hw_common_ecc_ctrl_init(struct mtd_info *mtd,
if (!data)
return -ENOMEM;
+ /* Prefer 1k ECC chunk over 512 ones */
+ if (ecc->size == 512 && mtd->writesize > 512) {
+ ecc->size = 1024;
+ ecc->strength *= 2;
+ }
+
/* Add ECC info retrieval from DT */
for (i = 0; i < ARRAY_SIZE(strengths); i++) {
if (ecc->strength <= strengths[i])