[U-Boot] [PATCH v2] scb9328: Add ARM relocation support
Torsten Koschorrek
koschorrek at synertronixx.de
Sat Jul 9 17:36:07 CEST 2011
Stefano Babic wrote:
> On 07/04/2011 01:06 PM, Torsten Koschorrek wrote:
>> This patch fixes compiler errors due to missing definitions of
>> CONFIG_SYS_SDRAM_BASE and CONFIG_SYS_INIT_SP_ADDR.
>> It also does some cleanup: CONFIG_SYS_TEXT_BASE was moved to scb9328.h,
>> obsolete config.mk was removed. The scb9328 board has 1 DRAM bank, so don't
>> ask for more banks. CONFIG_NR_DRAM_BANKS will ever be 1.
>>
>> Signed-off-by: Torsten Koschorrek<koschorrek at synertronixx.de>
>> ---
>
> Torsten,
>
> checkpatch reports a couple of errors and your patch seems corrupted.
> Have you used "git send-email" to post your patch ?
I used git format-patch plus mozilla/iceape. Hmm, maybe I choose another
mailer for sending patches the next time...
>
> WARNING: space prohibited between function name and open parenthesis '('
> #60: FILE: board/scb9328/scb9328.c:48:
> +void dram_init_banksize (void)
>
fixed in v3
> ERROR: patch seems to be corrupt (line wrapped?)
> #80: FILE: board/scb9328/scb9328.c:53:
> diff --git a/include/configs/scb9328.h b/include/configs/scb9328.h
>
hopefully ok in v3, this most probably came from mozilla
> ERROR: Macros with complex values should be enclosed in parenthesis
> #90: FILE: board/scb9328/scb9328.c:131:
> +#define CONFIG_SYS_INIT_SP_ADDR SCB9328_SDRAM_1 + 0xf00000
>
fixed in v3
> Best regards,
> Stefano Babic
>
Torsten
More information about the U-Boot
mailing list