[U-Boot] [PATCH ARM 3/3] s3c24x0 code style changes
Wolfgang Denk
wd at denx.de
Tue Dec 15 18:00:18 CET 2009
Dear Kevin,
in message <4B2796B7.3000300 at fearnside-systems.co.uk> you wrote:
>
> All I really wanted to do was change the sc324x0 register struct members
> to lower case, but if I do that without cleaning up the white space I
> get checkpatch.pl errors, and if I don't change the code to use the
> accessor functions the patch will get NAK-ed anyway...and all I really
> want to do is get the code into a state where I can add support for the
> Embest SBC244-II Board!
I understand and appreciate your efforts.
In such a situation it is actually not really important in which order
the changes are performend, but they shouldbe separated into
orthogonal patches, and these patches submitted as a series.
> I think I'll split the patches into a white-space change followed by an
> accessor function/s3c24x0 register change patch.
This is indeed the optimal sequence in this situation.
> > Please also re-check the resulting code. I think there are some way
> > too long lines, etc.
>
> checkpatch.pl didn't report any line length errors before I emailed the
> patches. I'll check it again.
Thanks.
Best regards,
Wolfgang Denk
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Humans do claim a great deal for that particular emotion (love).
-- Spock, "The Lights of Zetar", stardate 5725.6
More information about the U-Boot
mailing list