[U-Boot-Users] [PATCH 1/3] Initial OneNAND support
Kyungmin Park
kmpark at infradead.org
Thu Mar 15 01:43:57 CET 2007
Dear Wolfgang Denk,
> >
> > This patch supports OneNAND device on U-Boot.
> > The most codes are based on OneNAND MTD in kernel.
>
> Your patches have a few cosing style issues (indentation not
> by TAB's, trailing white space). Please clean up and
> resubmet; see also http://www.denx.de/wiki/UBoot/CodingStyle
Thank you, I use the 'indent' programe to fix indentation
>
> Also, please restrict your line length no around 70 characters or so.
Since its code based on kernel and consistent with kernel. it has almost
same syntax.
>
> And can you please add a GPL license header to
> onenand/onenand_bbt.c, include/linux/mtd/bbm.h
I added it.
>
> > For better performance I added 32-bytes aligned memcpy32.
> > Pleae check it.
>
> Did you measure how much of performance this gains? Is it
> really worth the effort? In any case, the file needs a GPL
> license header.
we can feel that it's more faster than before. OK I added GPL license
Thank you,
Kyungmin Park
More information about the U-Boot
mailing list