Make libpayload parse the coreboot tables before setting up the consoles
authorGabe Black <gabeblack@google.com>
Mon, 5 Mar 2012 23:49:32 +0000 (15:49 -0800)
committerPatrick Georgi <patrick@georgi-clan.de>
Sat, 31 Mar 2012 10:10:07 +0000 (12:10 +0200)
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>
payloads/libpayload/arch/i386/main.c

index 378c6f387f237241273a9162c810cfb75c86f827..36cfb69bff307f6838b382318fce37589199405d 100644 (file)
@@ -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.