dany/u-boot-megous
1
0
Fork 0
mirror of https://xff.cz/git/u-boot/ synced 2025-10-26 16:13:55 +01:00
Code Issues Packages Projects Releases Wiki Activity

armv8: spl: Call board_init_r from crt0_64 in SPL

Browse Source 
As part of the startup process for boards using the SPL, the
meaning of board_init_f changed such that it should return normally
rather than calling board_init_r directly. (see
db910353a1 )
This was fixed in 32-bit arm, but broke when SPL was added to
64 bit arm. This fixes crt0_64 so that it calls board_init_r
during the SPL and removes the direct call from board_init_f
from the arm SPL example.

Signed-off-by: Jeremy Hunt <Jeremy.Hunt@DEShawResearch.com>

Acked-by: Simon Glass <sjg@chromium.org>
This commit is contained in:
Jeremy Hunt
2016-07-18 12:01:02 -04:00
committed by Tom Rini
parent c933ed94bc
commit b8cb51d0de
2 changed files with 12 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
arch/arm/lib/crt0_64.S
View File

@@ -108,6 +108,7 @@ relocation_return:
* Set up final (full) environment * Set up final (full) environment
*/ */
bl c_runtime_cpu_setup /* still call old routine */ bl c_runtime_cpu_setup /* still call old routine */
#endif /* !CONFIG_SPL_BUILD */
/* TODO: For SPL, call spl_relocate_stack_gd() to alloc stack relocation */ /* TODO: For SPL, call spl_relocate_stack_gd() to alloc stack relocation */
@@ -130,6 +131,4 @@ clear_loop:
/* NOTREACHED - board_init_r() does not return */ /* NOTREACHED - board_init_r() does not return */
#endif /* !CONFIG_SPL_BUILD */
ENDPROC(_main) ENDPROC(_main)

24
arch/arm/lib/spl.c
View File

@@ -25,22 +25,20 @@ gd_t gdata __attribute__ ((section(".data")));
#endif #endif
/* /*
* In the context of SPL, board_init_f must ensure that any clocks/etc for * In the context of SPL, board_init_f() prepares the hardware for execution
* DDR are enabled, ensure that the stack pointer is valid, clear the BSS * from system RAM (DRAM, DDR...). As system RAM may not be available yet,
* and call board_init_r. We provide this version by default but mark it * board_init_f() must use the current GD to store any data which must be
* as __weak to allow for platforms to do this in their own way if needed. * passed on to later stages. These data include the relocation destination,
* the future stack, and the future GD location. BSS is cleared after this
* function (and therefore must be accessible).
*
* We provide this version by default but mark it as __weak to allow for
* platforms to do this in their own way if needed. Please see the top
* level U-Boot README "Board Initialization Flow" section for info on what
* to put in this function.
*/ */
void __weak board_init_f(ulong dummy) void __weak board_init_f(ulong dummy)
{ {
/* Clear the BSS. */
memset(__bss_start, 0, __bss_end - __bss_start);
#ifndef CONFIG_SPL_DM
/* TODO: Remove settings of the global data pointer here */
gd = &gdata;
#endif
board_init_r(NULL, 0);
} }
/* /*