From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Date: Thu, 07 Jan 2010 10:42:00 -0600 Subject: [U-Boot] [PATCH v2 0/9] Added support for SPEAr SoCs In-Reply-To: <1262865390-2016-1-git-send-email-vipin.kumar@st.com> References: <1262865390-2016-1-git-send-email-vipin.kumar@st.com> Message-ID: <4B460ED8.5030101@windriver.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Vipin KUMAR wrote: > This patch set is a reworked patch which incorporates all review feedbacks from > earlier earlier mails > > This patch set contains the support for 4 SoCs > SPEAr600 > SPEAr300 > SPEAr310 > SPEAr320 > > SPEAr is an ARM based SoC which supports rich set of peripherals like Ethernet, > USB Host, USB Device etc to support various general applications > > For further info on SPEAr SoC, please see README.spear also contained in the > patch set. > > Vipin (9): > Added README.spear > SPEAr600 SoC support added > SPEAr300 SoC support added > SPEAr310 SoC support added > SPEAr320 SoC support added > i2c driver support for SPEAr SoCs > smi driver support for SPEAr SoCs > nand(fsmc) driver support for SPEAr SoCs > usbd driver support for SPEAr SoCs This review is mechanical. Whitespace/formatting checking with checkpatch.pl is in general ok. Some warnings about braces, fix if you think approprate On patch 9, ERROR: "foo * bar" should be "foo *bar" #283: FILE: drivers/usb/gadget/spr_udc.c:229: +static void usbputpcktofifo(int epNum, u8 * bufp, u32 len) These should fixed The readme has trailing whitespace issues that were caught in the 'git am ' Please fix. On patch 9, usb. There are some merge warnings. Applying: usbd driver support for SPEAr SoCs error: patch failed: drivers/serial/usbtty.h:31 error: drivers/serial/usbtty.h: patch does not apply Using index info to reconstruct a base tree... Falling back to patching base and 3-way merge... Auto-merged drivers/serial/usbtty.h There are some recent updates for usb. Please rebase you source. On MAKEALL arm Spear boards fail to build Errors similar to this Assembler messages: Fatal error: can't create build/board/spear/spear300/../common/spr_misc.o: No such file or directory Assembler messages: Fatal error: can't create build/board/spear/spear300/../common/spr_lowlevel_init.o: No such file or directory The way i test MAKEALL is to define the BUILD_DIR to be something that is not the source directory. Please rerun MAKEALL this way and resolve these errors. I will do a more detailed review soon. Tom