mtd: nand: pxa3xx: read_page() returns max_bitflips
As per the ecc.read_page() prototype, we must return the maximum number of bitflips that were corrected on any one region covering an ecc step. Signed-off-by: Ezequiel Garcia <ezequiel.garcia@free-electrons.com> Tested-by: Daniel Mack <zonque@gmail.com> Signed-off-by: Brian Norris <computersforpeace@gmail.com>
This commit is contained in:
parent
664c7f5e81
commit
4e86fd22af
|
@ -751,6 +751,7 @@ static int pxa3xx_nand_read_page_hwecc(struct mtd_info *mtd,
|
||||||
{
|
{
|
||||||
struct pxa3xx_nand_host *host = mtd->priv;
|
struct pxa3xx_nand_host *host = mtd->priv;
|
||||||
struct pxa3xx_nand_info *info = host->info_data;
|
struct pxa3xx_nand_info *info = host->info_data;
|
||||||
|
int max_bitflips = 0;
|
||||||
|
|
||||||
chip->read_buf(mtd, buf, mtd->writesize);
|
chip->read_buf(mtd, buf, mtd->writesize);
|
||||||
chip->read_buf(mtd, chip->oob_poi, mtd->oobsize);
|
chip->read_buf(mtd, chip->oob_poi, mtd->oobsize);
|
||||||
|
@ -758,6 +759,7 @@ static int pxa3xx_nand_read_page_hwecc(struct mtd_info *mtd,
|
||||||
if (info->retcode == ERR_SBERR) {
|
if (info->retcode == ERR_SBERR) {
|
||||||
switch (info->use_ecc) {
|
switch (info->use_ecc) {
|
||||||
case 1:
|
case 1:
|
||||||
|
max_bitflips = 1;
|
||||||
mtd->ecc_stats.corrected++;
|
mtd->ecc_stats.corrected++;
|
||||||
break;
|
break;
|
||||||
case 0:
|
case 0:
|
||||||
|
@ -776,7 +778,7 @@ static int pxa3xx_nand_read_page_hwecc(struct mtd_info *mtd,
|
||||||
mtd->ecc_stats.failed++;
|
mtd->ecc_stats.failed++;
|
||||||
}
|
}
|
||||||
|
|
||||||
return 0;
|
return max_bitflips;
|
||||||
}
|
}
|
||||||
|
|
||||||
static uint8_t pxa3xx_nand_read_byte(struct mtd_info *mtd)
|
static uint8_t pxa3xx_nand_read_byte(struct mtd_info *mtd)
|
||||||
|
|
Loading…
Reference in a new issue