From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?B?6aas5YWL5rOh?= Date: Fri, 15 Jul 2011 08:17:26 +0800 Subject: [U-Boot] [PATCH 1/2] arm, lib/board.c: Coding Style cleanup In-Reply-To: <4E1F177C.3050105@aribaud.net> References: <1307088697-12159-1-git-send-email-hs@denx.de> <1307088697-12159-2-git-send-email-hs@denx.de> <4E1F177C.3050105@aribaud.net> 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 Heiko and Albert, 2011/7/15 Albert ARIBAUD > > Hi Heiko, > > Sorry for having kept this patch unanswered so long. I am now on > holiday, which means *a bit* more time for U-Boot, so here comes: > > Re: patch title, can you make it "arm: libboard.c: ..." rather than > "arm, libboard.c" ? Just remind both of you here, about the clean up patches, we have an other prefix for the patch title (subject). http://www.denx.de/wiki/U-Boot/Patches General Patch Submission Rules "Non-functional changes, i.e. whitespace and reformatting changes, should be done in separate patches marked as cosmetic. This separation of functional and cosmetic changes greatly facilitates the review process." So, please use "cosmetic:" as the beginning prefix of your clear up patch. > > Also, make sure you rebase onto latest u-boot-arm/master for next > version: this one does not apply cleanly ATM. [deleted] > > > - ? ? ? ? ? ? ? ? ? ? printf (" ?CRC: %08X", > > - ? ? ? ? ? ? ? ? ? ? ? ? ? ? crc32 (0, (const unsigned char *) CONFIG_SYS_FLASH_BASE, flash_size) > > + ? ? ? ? ? ? ? ? ? ? printf(" ?CRC: %08X", > > + ? ? ? ? ? ? ? ? ? ? ? ? ? ? crc32(0, > > + ? ? ? ? ? ? ? ? ? ? ? ? ? ? (const unsigned char *) CONFIG_SYS_FLASH_BASE, > > + ? ? ? ? ? ? ? ? ? ? ? ? ? ? flash_size) > > Pleas indent deeper for the last two lines that belong to the call to crc32. > > Amicalement, -- Best regards, Macpaul Lin