Hi Tom,

On Tue, Oct 23, 2012 at 1:45 AM, Tom Rini <tr...@ti.com> wrote:

> In general, things look OK but please run it through checkpatch.pl, use
> just one Signed-off-by line and fixup if (...) { ... one line ... } in
> the glue code you add that's not synced up from the kernel (I saw one in
> the am335x bits).  Thanks!  And lets move this out of RFC...
>

Hm.. checkpatch produces tons of messages (mostly over 80 character lines)
but I can't see anything about if ( ... ) { one-line } in the code I've
added by hand... I've fixed some other minor style problems in the code
that was not taken from the kernel directly.

Regards, Ilya.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to