Wolfgang Denk wrote: > Dear Alessandro Rubini, > > In message > <a14c1bde15838a59c60e177a43a4ebd35f28adea.1263851321.git.rub...@unipv.it> you > wrote: >> Signed-off-by: Alessandro Rubini <rub...@gnudd.com> >> Cc: Gregory Hermant <gregory.herm...@calao-systems.com> >> --- >> drivers/mtd/nand/nomadik.c | 15 +++++++++++---- >> 1 files changed, 11 insertions(+), 4 deletions(-) > > New config options should be documented. If not in the REDME, so at > last in the commit message. You may know what > CONFIG_NAND_NOMADIK_16BIT is supposed to mean, others may not. > > Also, I suggest to change the name into CONFIG_NOMADIK_NAND_16BIT > which seems more logical to me.
It's an attribute of the hardware, so it should be CONFIG_SYS_NOMADIK_NAND_16BIT. -Scott _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot