Hi Heiko,
On Thursday 30 June 2011 11:31 AM, Heiko Schocher wrote:
> Hello Aneesh,
>
> Aneesh V wrote:
>> Signed-off-by: Aneesh V<ane...@ti.com>
>> ---
>> arch/arm/cpu/armv7/omap-common/Makefile | 9 ++--
>> arch/arm/cpu/armv7/omap-common/spl.c | 56 ++++++++++++++++++++++++++++
>> arch/arm/cpu/armv7/omap-common/spl.lds | 62
>> +++++++++++++++++++++++++++++++
>> 3 files changed, 123 insertions(+), 4 deletions(-)
>> create mode 100644 arch/arm/cpu/armv7/omap-common/spl.c
>> create mode 100644 arch/arm/cpu/armv7/omap-common/spl.lds
>>
> [...]
>> diff --git a/arch/arm/cpu/armv7/omap-common/spl.c
>> b/arch/arm/cpu/armv7/omap-common/spl.c
>> new file mode 100644
>> index 0000000..b5a5f3c
>> --- /dev/null
>> +++ b/arch/arm/cpu/armv7/omap-common/spl.c
> [...]
>> @@ -0,0 +1,56 @@
>> +void board_init_f(ulong dummy)
>> +{
>> + debug(">>board_init_f()\n");
>> + relocate_code(CONFIG_SYS_SPL_STACK,&gdata, CONFIG_SYS_SPL_TEXT_BASE);
>> + debug("<<board_init_f()\n");
>
> This debug printf will never occur ;-)
Indeed. Thanks for pointing out.
>
> BTW: Do you really need to relocate code? You could just load the 2nd
> stage loader to ram from board_init_f, or?
I am passing the same address as I am executing from as the target for
the relocation, so the relocation will not happen, instead BSS will be
initialized. That's what I am calling it for. Initially I had my own
routine for clearing BSS. Then I decided to re-use it from start.S
BTW, this series is not complete as far as SPL support is concerned.
It doesn't even build due to many undefined references. It serves only
as a prototype for the framework part.
br,
Aneesh
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
