[U-Boot] [PATCHv1] ARM: Add Altera SOCFPGA Cyclone5

Tom Rini
2012-08-27T15:43:12Z

On 08/25/2012 04:56 AM, Pavel Machek wrote:
> Hi!
>
>> [snip]
>>>> It's the omap-common/spl.c code with the bits for IH_OS_LINUX taken out
>>>> :) Can you please re-do your series on top of the SPL series I just
>>>> posted that provides a common SPL framework? Thanks!
>>>
>>> I'll take a look. OTOH, ammount of code duplication here is quite low,
>>> and redoing it on top of SPL series will mean delaying merge, right?
>>
>> My hope is that since I got the SPL patches posted in time we can get
>> them in for v2012.10 so no, this will only help your chances :)
>
> Hopefully.
>
> Some observations:
>
> Not sure what to do with CONFIG_SPL_STACK: altera was just using
> symbol from linker script. Is there way to keep it like that? I'd hate
> to invent magic constant.

Well, lets think about this. Previously, everyone has been saying "use
this spot in SRAM" or "use this spot in DRAM (which has been setup
previously)". But I think you should be able to do
#define CONFIG_SPL_STACK __linker_symbol and be done with it.

> Same issue with CONFIG_SYS_SPL_MALLOC_START... I was doing:
>
> /* setup MALLOC after clocks going faster */
> mem_malloc_init((ulong) &__malloc_start,
> (&__malloc_end -
> &__malloc_start));
>
> I guess defines allow me to keep my definition. Good.

Should be fine, yes.

> spl_parse_image_header(): We have "modern" system, we'd prefer not to
> do
> /* Signature not found - assume u-boot.bin */
> ...but to hang in that case, in order not to jump to random code.
>
> Ok, I should have something publishable, soon ;-).

The only reason I like the assumption idea is that I've already seen at
least one distribution ship new MLO (based on SPL) and u-boot.bin rather
than u-boot.img for their beagleboard images.

--
Tom