Message ID | 9d21903d38de6685c9bf45d6eb8e32ab0e8c65c5.1490609415.git.nsekhar@ti.com |
---|---|
State | Changes Requested |
Delegated to: | Boris Brezillon |
Headers | show |
On Mon, 27 Mar 2017 15:56:55 +0530 Sekhar Nori <nsekhar@ti.com> wrote: > This partly reverts commit 28c015a9daabe4ed3aeb0ccf669a3f1c2b8b81d5. > > Since f6d7c1b5598b ("mtd: nand: davinci: Reinitialize the HW ECC > engine in 4bit hwctl") using subpage and 4bit HWECC works. > > Revert the original workaround made for this. Tested with > mtd_subpagetest and UBIFS on K2E, K2HK, K2L EVMs. > > The addition of keystone specific compatible to davinci_nand_of_match[] > contained in original commit is retained. Hm, it sounds like a bad idea to re-enable sub-page writes afterwards. You might break existing systems using UBI, because UBI detects the sub-page write feature and decides to place the VID header at a different position depending on that. To sum-up, you're likely to break users that have already flashed a UBI image with the old driver. If you want to re-enable sub-page writes, just switch back to the "ti,davinci-nand" compatible. > > Cc: Muralidharan Karicheri <m-karicheri2@ti.com> > Cc: Franklin S Cooper Jr <fcooper@ti.com> > Signed-off-by: Sekhar Nori <nsekhar@ti.com> > --- > drivers/mtd/nand/davinci_nand.c | 5 ----- > 1 file changed, 5 deletions(-) > > diff --git a/drivers/mtd/nand/davinci_nand.c b/drivers/mtd/nand/davinci_nand.c > index 27fa8b87cd5f..56c4afed8d74 100644 > --- a/drivers/mtd/nand/davinci_nand.c > +++ b/drivers/mtd/nand/davinci_nand.c > @@ -580,11 +580,6 @@ static struct davinci_nand_pdata > if (of_property_read_bool(pdev->dev.of_node, > "ti,davinci-nand-use-bbt")) > pdata->bbt_options = NAND_BBT_USE_FLASH; > - > - if (of_device_is_compatible(pdev->dev.of_node, > - "ti,keystone-nand")) { > - pdata->options |= NAND_NO_SUBPAGE_WRITE; > - } > } > > return dev_get_platdata(&pdev->dev);
diff --git a/drivers/mtd/nand/davinci_nand.c b/drivers/mtd/nand/davinci_nand.c index 27fa8b87cd5f..56c4afed8d74 100644 --- a/drivers/mtd/nand/davinci_nand.c +++ b/drivers/mtd/nand/davinci_nand.c @@ -580,11 +580,6 @@ static struct davinci_nand_pdata if (of_property_read_bool(pdev->dev.of_node, "ti,davinci-nand-use-bbt")) pdata->bbt_options = NAND_BBT_USE_FLASH; - - if (of_device_is_compatible(pdev->dev.of_node, - "ti,keystone-nand")) { - pdata->options |= NAND_NO_SUBPAGE_WRITE; - } } return dev_get_platdata(&pdev->dev);
This partly reverts commit 28c015a9daabe4ed3aeb0ccf669a3f1c2b8b81d5. Since f6d7c1b5598b ("mtd: nand: davinci: Reinitialize the HW ECC engine in 4bit hwctl") using subpage and 4bit HWECC works. Revert the original workaround made for this. Tested with mtd_subpagetest and UBIFS on K2E, K2HK, K2L EVMs. The addition of keystone specific compatible to davinci_nand_of_match[] contained in original commit is retained. Cc: Muralidharan Karicheri <m-karicheri2@ti.com> Cc: Franklin S Cooper Jr <fcooper@ti.com> Signed-off-by: Sekhar Nori <nsekhar@ti.com> --- drivers/mtd/nand/davinci_nand.c | 5 ----- 1 file changed, 5 deletions(-)