diff options
| author | Kyungmin Park <kyungmin.park@samsung.com> | 2006-12-29 03:51:40 +0100 |
|---|---|---|
| committer | Artem Bityutskiy <dedekind@infradead.org> | 2007-01-10 13:40:16 +0100 |
| commit | 2fd32d4af83f4535d12d3f6dd23189352a9596fa (patch) | |
| tree | e7d20f3c6dc0e99854ca894c58c0d65295250839 /drivers/mtd | |
| parent | [MTD] OneNAND: release CPU in cycles (diff) | |
| download | linux-2fd32d4af83f4535d12d3f6dd23189352a9596fa.tar.xz linux-2fd32d4af83f4535d12d3f6dd23189352a9596fa.zip | |
[MTD] OneNAND: fix onenand_wait bug in read ecc error
Even though there is ECC error. OneNAND driver updates the buffram as valid
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
Diffstat (limited to 'drivers/mtd')
| -rw-r--r-- | drivers/mtd/onenand/onenand_base.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/mtd/onenand/onenand_base.c b/drivers/mtd/onenand/onenand_base.c index 0037cee5ef27..e80857b3bb83 100644 --- a/drivers/mtd/onenand/onenand_base.c +++ b/drivers/mtd/onenand/onenand_base.c @@ -298,7 +298,7 @@ static int onenand_wait(struct mtd_info *mtd, int state) unsigned long timeout; unsigned int flags = ONENAND_INT_MASTER; unsigned int interrupt = 0; - unsigned int ctrl, ecc; + unsigned int ctrl; /* The 20 msec is enough */ timeout = jiffies + msecs_to_jiffies(20); @@ -324,7 +324,7 @@ static int onenand_wait(struct mtd_info *mtd, int state) } if (interrupt & ONENAND_INT_READ) { - ecc = this->read_word(this->base + ONENAND_REG_ECC_STATUS); + int ecc = this->read_word(this->base + ONENAND_REG_ECC_STATUS); if (ecc) { DEBUG(MTD_DEBUG_LEVEL0, "onenand_wait: ECC error = 0x%04x\n", ecc); if (ecc & ONENAND_ECC_2BIT_ALL) @@ -332,6 +332,7 @@ static int onenand_wait(struct mtd_info *mtd, int state) else if (ecc & ONENAND_ECC_1BIT_ALL) mtd->ecc_stats.corrected++; } + return ecc; } return 0; |