[U-Boot] [PATCH v2] cmd_bdinfo.c: clean up with 2.6.38 checkpatch.pl

Detlev Zundel dzu at denx.de
Fri Apr 29 10:56:38 CEST 2011


Hi Macpaul,

> Hi Detlev,
>
> 2011/4/27 Detlev Zundel <dzu at denx.de>:
>> Hi Macpaul,
>>
>>> clean up with 2.6.38 checkpatch.pl
>>
>> There is indeed only one non-whitespace hunk in there:
>> It's ok that you do the line wrapping, but actually with such lists we
>> require alphabetical sorting of the clauses, so can you please juggle
>> the list so that the "XILINX_405" gets to the end?
>
> Since I've correct the subject of this patch v3 according to the
> "cosmetic" rule,
> it seems patch v3 is out of this mailing thread.

This change of the subject was the reason why I did not require you
changing it as I know that Wolfgang currently matches patch versions by
subject strings.

"Correct" threading however uses "References:" mail headers and can cope
with changing subjects.  Somehow you seem to know this, as v3 indeed
_has_ such a correct header and thus the threading in my mail reader
works just fine ;)

Cheers
  Detlev

-- 
0x2B | ~0x2B
--
DENX Software Engineering GmbH,      MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich,  Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-40 Fax: (+49)-8142-66989-80 Email: dzu at denx.de


More information about the U-Boot mailing list