3 Nov
2012
3 Nov
'12
5:08 p.m.
Hi Tom,
On Tue, Oct 23, 2012 at 1:45 AM, Tom Rini trini@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.