mtd: rawnand: Give more information about the ECC weakness
authorMiquel Raynal <miquel.raynal@bootlin.com>
Tue, 21 Apr 2020 16:39:06 +0000 (18:39 +0200)
committerMiquel Raynal <miquel.raynal@bootlin.com>
Mon, 11 May 2020 07:51:39 +0000 (09:51 +0200)
When the ECC strength is too weak compared to the NAND chip
requirements, display the values so that it is clear for people how
much they are far from the requirements (and might get in troubles in
the future).

Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Link: https://lore.kernel.org/linux-mtd/20200421163906.7515-1-miquel.raynal@bootlin.com
drivers/mtd/nand/raw/nand_base.c

index c24e5e2..52cf73c 100644 (file)
@@ -5781,8 +5781,10 @@ static int nand_scan_tail(struct nand_chip *chip)
 
        /* ECC sanity check: warn if it's too weak */
        if (!nand_ecc_strength_good(chip))
-               pr_warn("WARNING: %s: the ECC used on your system is too weak compared to the one required by the NAND chip\n",
-                       mtd->name);
+               pr_warn("WARNING: %s: the ECC used on your system (%db/%dB) is too weak compared to the one required by the NAND chip (%db/%dB)\n",
+                       mtd->name, chip->ecc.strength, chip->ecc.size,
+                       chip->base.eccreq.strength,
+                       chip->base.eccreq.step_size);
 
        /* Allow subpage writes up to ecc.steps. Not possible for MLC flash */
        if (!(chip->options & NAND_NO_SUBPAGE_WRITE) && nand_is_slc(chip)) {