[PATCH v3 3/3] mtd: nand: omap: fix ecclayout->oobfree->length

Gupta, Pekon pekon at ti.com
Mon Feb 17 02:50:18 EST 2014


removing  stable at vger.kernel.org from CC list.
(git send-email --suppress-cc=cc din't work, may be it should have been --supress-cc=bodycc)

>From: Gupta, Pekon
>
>This patch excludes reserved-marker byte-position from oobfree->length
>calculation. Thus all bytes from oobfree->offset till end of OOB are free.
>
>CC: <stable at vger.kernel.org> # 3.13.x+
>Signed-off-by: Pekon Gupta <pekon at ti.com>
>---
> drivers/mtd/nand/omap2.c | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
>diff --git a/drivers/mtd/nand/omap2.c b/drivers/mtd/nand/omap2.c
>index 58685ab..bf642ce 100644
>--- a/drivers/mtd/nand/omap2.c
>+++ b/drivers/mtd/nand/omap2.c
>@@ -1994,9 +1994,8 @@ static int omap_nand_probe(struct platform_device *pdev)
> 		goto return_error;
> 	}
>
>-	/* populate remaining ECC layout data */
>-	ecclayout->oobfree->length = mtd->oobsize - (BADBLOCK_MARKER_LENGTH +
>-							ecclayout->eccbytes);
>+	/* all OOB bytes from oobfree->offset till end off OOB are free */
>+	ecclayout->oobfree->length = mtd->oobsize - ecclayout->oobfree->offset;
> 	/* check if NAND device's OOB is enough to store ECC signatures */
> 	if (mtd->oobsize < (ecclayout->eccbytes + BADBLOCK_MARKER_LENGTH)) {
> 		pr_err("not enough OOB bytes required = %d, available=%d\n",
>--
>1.8.5.1.163.gd7aced9


with regards, pekon



More information about the linux-mtd mailing list