Messages in this thread | | | Date | Fri, 22 Jun 2018 13:34:36 +0200 | From | Boris Brezillon <> | Subject | Re: [PATCH] mtd: rawnand: fix return value check for bad block status |
| |
On Mon, 18 Jun 2018 15:57:57 +0200 Miquel Raynal <miquel.raynal@bootlin.com> wrote:
> Hi Boris, > > On Wed, 13 Jun 2018 14:32:36 +0530, Abhishek Sahu > <absahu@codeaurora.org> wrote: > > > Positive return value from read_oob() is making false BAD > > blocks. For some of the NAND controllers, OOB bytes will be > > protected with ECC and read_oob() will return number of bitflips. > > If there is any bitflip in ECC protected OOB bytes for BAD block > > status page, then that block is getting treated as BAD. > > > > Fixes: c120e75e0e7d ("mtd: nand: use read_oob() instead of cmdfunc() for bad block check") > > Cc: <stable@vger.kernel.org> > > Signed-off-by: Abhishek Sahu <absahu@codeaurora.org> > > --- > > drivers/mtd/nand/raw/nand_base.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/drivers/mtd/nand/raw/nand_base.c b/drivers/mtd/nand/raw/nand_base.c > > index f28c3a5..4a73f73 100644 > > --- a/drivers/mtd/nand/raw/nand_base.c > > +++ b/drivers/mtd/nand/raw/nand_base.c > > @@ -440,7 +440,7 @@ static int nand_block_bad(struct mtd_info *mtd, loff_t ofs) > > > > for (; page < page_end; page++) { > > res = chip->ecc.read_oob(mtd, chip, page); > > - if (res) > > + if (res < 0) > > return res; > > > > bad = chip->oob_poi[chip->badblockpos]; > > Reviewed-by: Miquel Raynal <miquel.raynal@bootlin.com>
Applied.
Thanks,
Boris
> > I suppose this patch is a good candidate to be part of a future > mtd/fixes PR? > > Regards, > Miquèl > > ______________________________________________________ > Linux MTD discussion mailing list > http://lists.infradead.org/mailman/listinfo/linux-mtd/
| |