Prevent build breakage without consoles enabled
authorStefan Reinauer <reinauer@chromium.org>
Wed, 1 Jun 2011 21:04:50 +0000 (14:04 -0700)
committerStefan Reinauer <stefan.reinauer@coreboot.org>
Thu, 13 Oct 2011 18:11:33 +0000 (20:11 +0200)
If all console types are disabled, coreboot will fail to compile because
static code is unused. This patch fixes the issue.

Signed-off-by: Stefan Reinauer <reinauer@google.com>
Change-Id: Ie9c8bf2a78e3aeba4c2908b06bc03f0f5af37db2
Reviewed-on: http://review.coreboot.org/260
Tested-by: build bot (Jenkins)
Reviewed-by: Marc Jones <marcj303@gmail.com>
src/arch/x86/boot/coreboot_table.c

index d53a23a9eeacb7178e6b9f5a76f25735c9cecf27..cdfc0c1d8d79d1ce6176febc290421fcf175a0f1 100644 (file)
@@ -120,6 +120,8 @@ static struct lb_serial *lb_serial(struct lb_header *header)
 #endif
 }
 
+#if CONFIG_CONSOLE_SERIAL8250 || CONFIG_CONSOLE_SERIAL8250MEM || \
+    CONFIG_CONSOLE_LOGBUF || CONFIG_USBDEBUG
 static void add_console(struct lb_header *header, u16 consoletype)
 {
        struct lb_console *console;
@@ -130,6 +132,8 @@ static void add_console(struct lb_header *header, u16 consoletype)
        console->type = consoletype;
 }
 
+#endif
+
 static void lb_console(struct lb_header *header)
 {
 #if CONFIG_CONSOLE_SERIAL8250