[coreboot] New patch to review for coreboot: 7b1da0f Make libpayload parse the coreboot tables before setting up the consoles

Stefan Reinauer (stefan.reinauer@coreboot.org) gerrit at coreboot.org
Fri Mar 30 22:56:01 CEST 2012

Stefan Reinauer (stefan.reinauer at coreboot.org) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/816


commit 7b1da0fb2b5dbc8cc7269811ac36bf52f3635b15
Author: Gabe Black <gabeblack at google.com>
Date:   Mon Mar 5 15:49:32 2012 -0800

    Make libpayload parse the coreboot tables before setting up the consoles
    At least one of the console drivers, coreboot fb, uses information in the
    sysinfo structure to set itself up. If that structure hasn't been populated,
    the driver decides that there is no framebuffer and disables itself. Reversing
    the order these are set up fixes that problem.
    Change-Id: Idd8b5518980dfdd82fd4359dd0133ab7736fc428
    Signed-off-by: Gabe Black <gabeblack at google.com>
 payloads/libpayload/arch/i386/main.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/payloads/libpayload/arch/i386/main.c b/payloads/libpayload/arch/i386/main.c
index 378c6f3..36cfb69 100644
--- a/payloads/libpayload/arch/i386/main.c
+++ b/payloads/libpayload/arch/i386/main.c
@@ -46,12 +46,12 @@ void start_main(void)
 	extern int main(int argc, char **argv);
-	/* Set up the consoles. */
-	console_init();
 	/* Gather system information. */
+	/* Set up the consoles. */
+	console_init();
 	 * Any other system init that has to happen before the
 	 * user gets control goes here.

More information about the coreboot mailing list