| commit | e44b9a9c135727f3410e029910275f40681dc8bc | |
| author | Albert Hsieh <wen.hsieh@broadcom.com> | |
| Mon, 20 Nov 2017 03:26:26 +0000 (20 11:26 +0800) | ||
| committer | Richard Weinberger <richard@nod.at> | |
| Fri, 15 Dec 2017 21:26:48 +0000 (15 22:26 +0100) | ||
| tree | 94fbaa2bdeadfc3e4d7671abccc3c7d4e3cfffc4 | treesnapshot (tar.gz zip) |
| parent | fdf2e821052958a114618a95ab18a300d0b080cb | commitdiff |
mtd: nand: brcmnand: Zero bitflip is not an error
A negative return value of brcmstb_nand_verify_erased_page() indicates a
real bitflip error of an erased page, and other return values (>= 0) show
the corrected bitflip number. Zero return value means no bitflip, but the
current driver code treats it as an error, and eventually leads to
falsely reported ECC error.
Fixes: 02b88eea9f9c ("mtd: brcmnand: Add check for erased page bitflip")
Signed-off-by: Albert Hsieh <wen.hsieh@broadcom.com>
Acked-by: Boris Brezillon <boris.brezillon@free-electrons.com>
Signed-off-by: Richard Weinberger <richard@nod.at>
A negative return value of brcmstb_nand_verify_erased_page() indicates a
real bitflip error of an erased page, and other return values (>= 0) show
the corrected bitflip number. Zero return value means no bitflip, but the
current driver code treats it as an error, and eventually leads to
falsely reported ECC error.
Fixes: 02b88eea9f9c ("mtd: brcmnand: Add check for erased page bitflip")
Signed-off-by: Albert Hsieh <wen.hsieh@broadcom.com>
Acked-by: Boris Brezillon <boris.brezillon@free-electrons.com>
Signed-off-by: Richard Weinberger <richard@nod.at>
| drivers/mtd/nand/brcmnand/brcmnand.c | diffblobblamehistory |