Hi Macpaul,
Hi Detlev,
2011/4/27 Detlev Zundel dzu@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