2013/5/9 Scott Wood <scottw...@freescale.com>: > On 05/07/2013 01:33:11 AM, Kuo-Jung Su wrote: >> >> From: Kuo-Jung Su <dant...@faraday-tech.com> >> >> Faraday FTNANDC021 is a integrated NAND flash controller. >> It use a build-in command table to abstract the underlying >> NAND flash control logic. >> >> For example: >> >> Issuing a command 0x10 to FTNANDC021 would result in >> a page write + a read status operation. >> >> Signed-off-by: Kuo-Jung Su <dant...@faraday-tech.com> >> CC: Scott Wood <scottw...@freescale.com> > > > Looks mostly OK, just a couple nits: > > >> diff --git a/README b/README >> index 0d37d56..883182d 100644 >> --- a/README >> +++ b/README >> @@ -3879,6 +3879,10 @@ Low Level (hardware related) configuration options: >> - drivers/mtd/nand/ndfc.c >> - drivers/mtd/nand/mxc_nand.c >> >> +- CONFIG_SYS_FTNANDC021_TIMING >> + Defined to tell the FTNANDC021 controller that the NAND >> chip is >> + using customized timing parameters. > > > This makes it sound like a boolean option, but it's actually an array -- > would be nice to mention what's actually expected. > >
Got it, thanks >> + if (mtd->writesize >= 4096) { > Got it, thanks > > Whitespace > > -Scott -- Best wishes, Kuo-Jung Su _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot