On Tue, 2014-01-28 at 23:19 +0000, Karicheri, Muralidharan wrote: > >-----Original Message----- > >From: Scott Wood [mailto:scottw...@freescale.com] > >Sent: Wednesday, January 22, 2014 3:48 PM > >To: Karicheri, Muralidharan > >Cc: u-boot@lists.denx.de; Rini, Tom; Andrianov, Vitaly > >Subject: Re: [U-Boot] [PATCH 5/5] NAND: DaVinci: allow forced disable of > >subpage writes > > > >On Mon, 2014-01-20 at 17:10 -0500, Murali Karicheri wrote: > >> This patch introduces a configurable mechanism to disable subpage > >> writes in the DaVinci NAND driver. > >> > >> Signed-off-by: Vitaly Andrianov <vita...@ti.com> > >> Signed-off-by: Murali Karicheri <m-kariche...@ti.com> > >> --- > >> drivers/mtd/nand/davinci_nand.c | 3 +++ > >> 1 file changed, 3 insertions(+) > >> > >> diff --git a/drivers/mtd/nand/davinci_nand.c > >> b/drivers/mtd/nand/davinci_nand.c index 5b17d7b..75b03a7 100644 > >> --- a/drivers/mtd/nand/davinci_nand.c > >> +++ b/drivers/mtd/nand/davinci_nand.c > >> @@ -609,6 +609,9 @@ void davinci_nand_init(struct nand_chip *nand) > >> #ifdef CONFIG_SYS_NAND_USE_FLASH_BBT > >> nand->bbt_options |= NAND_BBT_USE_FLASH; > >> #endif > >> +#ifdef CONFIG_SYS_NAND_NO_SUBPAGE_WRITE > >> + nand->options |= NAND_NO_SUBPAGE_WRITE; > >> +#endif > >> #ifdef CONFIG_SYS_NAND_HW_ECC > >> nand->ecc.mode = NAND_ECC_HW; > >> nand->ecc.size = 512; > > > >Why does it need to be configurable? > > > > Same driver used in multiple platforms, and on some platforms like the one > I work with doesn't support subpage writes and has to be disabled.
The option needs to be documented, and it should be either named in a davinci-specific manner, or the documentation should clearly list which drivers pay attention to it. It seems odd though, that the ability to support subpage writes is an independent thing rather than a consequence of some other variable such as controller version. -Scott _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
