Currently, is_erased() is called against "buf" twice, so the second call is meaningless. The second one should be checked against chip->oob_poi.
Signed-off-by: Masahiro Yamada <yamada.masah...@socionext.com> --- drivers/mtd/nand/denali.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/mtd/nand/denali.c b/drivers/mtd/nand/denali.c index cbc7f75..753e9a02 100644 --- a/drivers/mtd/nand/denali.c +++ b/drivers/mtd/nand/denali.c @@ -1160,7 +1160,7 @@ static int denali_read_page(struct mtd_info *mtd, struct nand_chip *chip, if (check_erased_page) { if (!is_erased(buf, mtd->writesize)) mtd->ecc_stats.failed++; - if (!is_erased(buf, mtd->oobsize)) + if (!is_erased(chip->oob_poi, mtd->oobsize)) mtd->ecc_stats.failed++; } } -- 2.7.4