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

Reply via email to