From: Gabe Black <gabeblack@google.com>
Date: Mon, 5 Mar 2012 23:49:32 +0000 (-0800)
Subject: Make libpayload parse the coreboot tables before setting up the consoles
X-Git-Url: http://wien.tomnetworks.com/gitweb/?p=coreboot.git;a=commitdiff_plain;h=b9fa1ed5e80b607df59783cdc406d85bcccf63f5

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@google.com>
Reviewed-on: http://review.coreboot.org/816
Tested-by: build bot (Jenkins)
Reviewed-by: Patrick Georgi <patrick@georgi-clan.de>
---

diff --git a/payloads/libpayload/arch/i386/main.c b/payloads/libpayload/arch/i386/main.c
index 378c6f387..36cfb69bf 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. */
 	lib_get_sysinfo();
 
+	/* Set up the consoles. */
+	console_init();
+
 	/*
 	 * Any other system init that has to happen before the
 	 * user gets control goes here.