From mboxrd@z Thu Jan 1 00:00:00 1970 From: Detlev Zundel Date: Fri, 29 Apr 2011 10:56:38 +0200 Subject: [U-Boot] [PATCH v2] cmd_bdinfo.c: clean up with 2.6.38 checkpatch.pl In-Reply-To: (Macpaul Lin's message of "Thu, 28 Apr 2011 10:35:31 +0800") References: <1303872765-13825-1-git-send-email-macpaul@andestech.com> <1303873107-14082-1-git-send-email-macpaul@andestech.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi Macpaul, > Hi Detlev, > > 2011/4/27 Detlev Zundel : >> 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