From mboxrd@z Thu Jan 1 00:00:00 1970 From: Marek Vasut Date: Thu, 13 Oct 2011 20:14:55 +0200 Subject: [U-Boot] [PATCH 01/15 V3] iMX28: Initial support for iMX28 CPU In-Reply-To: <4E970FA0.2020505@denx.de> References: <1317131135-22354-1-git-send-email-marek.vasut@gmail.com> <1317375728-23802-1-git-send-email-marek.vasut@gmail.com> <4E970FA0.2020505@denx.de> Message-ID: <201110132014.55510.marek.vasut@gmail.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Thursday, October 13, 2011 06:19:44 PM Stefano Babic wrote: > On 09/30/2011 11:42 AM, Marek Vasut wrote: > > This patch supports: > > - Timers > > - Debug UART > > - Clock > > Hi Marek, > > /opt/tools/bin/checkpatch.pl --no-tree > ~/Desktop/U-Boot-01-15-V3-iMX28-Initial-support-for-iMX28-CPU.patch > WARNING: line over 80 characters > #634: FILE: arch/arm/cpu/arm926ejs/mx28/mx28.c:150: > +int do_mx28_showclocks(cmd_tbl_t *cmdtp, int flag, int argc, char * > const argv[]) > > total: 0 errors, 1 warnings, 4527 lines checked > > Can you fix and resubmit ? > > Stefano Hi Stefano, the patch is big and will need approval to be let into ML. Removing the space in char * const argv[] shall do the trick, can you please do that by hand ? Cheers