[coreboot-gerrit] Patch set updated for coreboot: f381fa9 arm64: Reorganize payload entry code and related Kconfigs

Patrick Georgi (pgeorgi@google.com) gerrit at coreboot.org
Tue May 19 14:35:18 CEST 2015


Patrick Georgi (pgeorgi at google.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/10244

-gerrit

commit f381fa939d48b375421d2684762941efbc02b3b0
Author: Julius Werner <jwerner at chromium.org>
Date:   Thu May 7 16:59:31 2015 -0700

    arm64: Reorganize payload entry code and related Kconfigs
    
    This patch slightly reorganizes arm64/boot.c with the aim of being more
    readable. Improve locality of code.
    
    [pg: taken from patch linked below]
    
    Change-Id: Ie9ea6a10b9d5e5c33bf020369afcae4f10ef068e
    Signed-off-by: Patrick Georgi <pgeorgi at chromium.org>
    Original-Commit-Id: 5067e47bc03f04ad2dba044f022716e0fc62bb9e
    Original-Change-Id: I1b2038acc0d054716a3c580ce97ea8e9a45abfa2
    Original-Signed-off-by: Julius Werner <jwerner at chromium.org>
    Original-Reviewed-on: https://chromium-review.googlesource.com/270783
    Original-Reviewed-by: Aaron Durbin <adurbin at chromium.org>
---
 src/arch/arm64/boot.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/arch/arm64/boot.c b/src/arch/arm64/boot.c
index 1fdc234..367aaac 100644
--- a/src/arch/arm64/boot.c
+++ b/src/arch/arm64/boot.c
@@ -36,16 +36,16 @@ static void run_payload(struct prog *prog)
 	doit = prog_entry(prog);
 	arg = prog_entry_arg(prog);
 
-	uint8_t current_el = get_current_el();
-
-	printk(BIOS_SPEW, "entry    = %p\n", doit);
-
 	if (IS_ENABLED(CONFIG_ARM64_USE_SECURE_MONITOR))
 		secmon_run(doit, arg);
 	else {
+		uint8_t current_el = get_current_el();
+
 		/* Start the other CPUs spinning. */
 		spintable_start();
 
+		printk(BIOS_SPEW, "entry    = %p\n", doit);
+
 		/* If current EL is not EL3, jump to payload at same EL. */
 		if (current_el != EL3) {
 			cache_sync_instructions();



More information about the coreboot-gerrit mailing list