[U-Boot] [PATCH v2 0/9] Added support for SPEAr SoCs

[Armando VISCONTI]

Thanks Tom,

Tomorrow Vipin will change it according to your inputs.
He will also do the rebase to latest uboot.

Rgds,
Arm

Tom wrote:
> 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
>
>
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
>   


-- 
-- "Every step appears to be the unavoidable consequence of the
-- preceding one." (A. Einstein) 
-- 
Armando Visconti                  Mobile: (+39) 346 8879146
Senior SW Engineer                Fax:    (+39) 02 93519290
CPG                               Work:   (+39) 02 93519683
Computer System Division          e-mail: armando.visconti at st.com
ST Microelectronics               TINA:   051  4683