[coreboot-gerrit] Patch set updated for coreboot: soc/apollolake/romstage: Call console_init before any printk()

Alexandru Gagniuc (alexandrux.gagniuc@intel.com) gerrit at coreboot.org
Tue May 17 03:29:20 CEST 2016


Alexandru Gagniuc (alexandrux.gagniuc at intel.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/14864

-gerrit

commit 9d989791fae24c80e929158f04f3b84c7fa8c168
Author: Alexandru Gagniuc <alexandrux.gagniuc at intel.com>
Date:   Mon May 16 16:52:54 2016 -0700

    soc/apollolake/romstage: Call console_init before any printk()
    
    Do not assume that the bootblock will have initialized the console.
    Follow the convention used on all other platforms and explicitly call
    console_init() before any printk().
    BONUS: We can get rid of the silly "Staring romstage..." message, as
    console_init() will print a standardized message.
    
    Change-Id: Id91f0fc15ecbd3635d67a261907f4c6af9a499ab
    Signed-off-by: Alexandru Gagniuc <alexandrux.gagniuc at intel.com>
---
 src/soc/intel/apollolake/romstage.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/soc/intel/apollolake/romstage.c b/src/soc/intel/apollolake/romstage.c
index 8ec94a7..7699b85 100644
--- a/src/soc/intel/apollolake/romstage.c
+++ b/src/soc/intel/apollolake/romstage.c
@@ -127,12 +127,12 @@ asmlinkage void car_stage_entry(void)
 	struct romstage_handoff *handoff;
 	struct chipset_power_state *ps = car_get_var_ptr(&power_state);
 
-	printk(BIOS_DEBUG, "Starting romstage...\n");
 
 	soc_early_romstage_init();
-
 	disable_watchdog();
 
+	console_init();
+
 	prev_sleep_state = fill_power_state(ps);
 
 	/* Make sure the blob does not override our data in CAR */



More information about the coreboot-gerrit mailing list