Wolfgang Denk wrote:
> Dear Alessandro Rubini,
> 
> In message 
> <a14c1bde15838a59c60e177a43a4ebd35f28adea.1263851321.git.rub...@unipv.it> you 
> wrote:
>> Signed-off-by: Alessandro Rubini <[email protected]>
>> Cc: Gregory Hermant <[email protected]>
>> ---
>>  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
[email protected]
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to