[U-Boot] [PATCH] igep00x0: Use all SRAM available for SPL.

[Enric Balletbo Serra]

Hi Heiko,

2016-04-27 6:25 GMT+02:00 Heiko Schocher <hs at denx.de>:
> Hello Enric,
>
> Am 26.04.2016 um 17:05 schrieb Enric Balletbo i Serra:
>>
>> Internal SRAM starts at 0x40200000 and ends at 0x4020FFFF, so there
>> are 64KB available to be used for SPL. This will also help some
>> compilers to fit all the code into the allocated space.
>>
>> Signed-off-by: Enric Balletbo i Serra <enric.balletbo at collabora.com>
>> ---
>>   include/configs/omap3_igep00x0.h | 7 +++++++
>>   1 file changed, 7 insertions(+)
>>
>> diff --git a/include/configs/omap3_igep00x0.h
>> b/include/configs/omap3_igep00x0.h
>> index 5da50a5..2459064 100644
>> --- a/include/configs/omap3_igep00x0.h
>> +++ b/include/configs/omap3_igep00x0.h
>> @@ -19,6 +19,13 @@
>>   #include <configs/ti_omap3_common.h>
>>   #include <asm/mach-types.h>
>>
>> +/* SRAM starts at 0x40200000 and ends at 0x4020FFFF (64KB) */
>> +#undef CONFIG_SPL_MAX_SIZE
>> +#undef CONFIG_SPL_TEXT_BASE
>> +
>> +#define CONFIG_SPL_MAX_SIZE            (64*1024)
>> +#define CONFIG_SPL_TEXT_BASE           0x40200000
>
>
> Hmm... risky, as the SPL needs at last some bytes for stack, until
> RAM is initialized and stack moved to it ... or do I miss something?
>

This is what I thought, orginally I thought

  CONFIG_SPL_MAX_SIZE            (60*1024) /* 4KB for stack */

But then I saw that overo and logic boards set

  CONFIG_SPL_MAX_SIZE    (64 * 1024)

I send this version just to have some discussion. So, can we say that overo
and logic boards are incorrect too (or at least risky)? And, Tom proposed a
4KB stack, do you think it's enough?

Regards,
 Enric

> bye,
> Heiko
> --
> DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot