[coreboot] New patch to review for coreboot: 3652eb5 Fix technexion tim5690 board VGA handler

Stefan Reinauer (stefan.reinauer@coreboot.org) gerrit at coreboot.org
Tue Nov 27 21:47:48 CET 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/1929

-gerrit

commit 3652eb5ae14a561967e51d17fff53e4a4ded9fe5
Author: Stefan Reinauer <reinauer at chromium.org>
Date:   Tue Nov 27 12:44:18 2012 -0800

    Fix technexion tim5690 board VGA handler
    
    When dropping ramstage.a, unused functions with unresolved
    symbols are not silently dropped anymore. This makes the
    tim5690 compilation fail.
    This fix makes sure we don't compile in the int15 handler code
    when we don't set CONFIG_VGA_ROM_RUN
    
    Change-Id: If6872c983d9fd811eb33259421f94b551f3b9b34
    Signed-off-by: Stefan Reinauer <reinauer at google.com>
---
 src/mainboard/technexion/tim5690/Makefile.inc | 2 +-
 src/mainboard/technexion/tim5690/mainboard.c  | 8 +++++++-
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/src/mainboard/technexion/tim5690/Makefile.inc b/src/mainboard/technexion/tim5690/Makefile.inc
index d9e8046..878dbab 100644
--- a/src/mainboard/technexion/tim5690/Makefile.inc
+++ b/src/mainboard/technexion/tim5690/Makefile.inc
@@ -23,4 +23,4 @@
 ramstage-y += tn_post_code.c
 
 ramstage-y += speaker.c
-ramstage-y += vgabios.c
+ramstage-$(CONFIG_VGA_ROM_RUN) += vgabios.c
diff --git a/src/mainboard/technexion/tim5690/mainboard.c b/src/mainboard/technexion/tim5690/mainboard.c
index 878daae..7974ce9 100644
--- a/src/mainboard/technexion/tim5690/mainboard.c
+++ b/src/mainboard/technexion/tim5690/mainboard.c
@@ -188,6 +188,7 @@ static void mb_gpio_init(u16 *iobase)
         it8712f_exit_conf();
 }
 
+#if CONFIG_VGA_ROM_RUN
 /* The LCD's panel id seletion. */
 static void lcd_panel_id(rs690_vbios_regs *vbios_regs, u8 num_id)
 {
@@ -215,6 +216,7 @@ static void lcd_panel_id(rs690_vbios_regs *vbios_regs, u8 num_id)
 		break;
 	}
 }
+#endif
 
 /*************************************************
 * enable the dedicated function in tim5690 board.
@@ -222,14 +224,17 @@ static void lcd_panel_id(rs690_vbios_regs *vbios_regs, u8 num_id)
 *************************************************/
 static void tim5690_enable(device_t dev)
 {
-	rs690_vbios_regs vbios_regs;
 	u16 gpio_base = IT8712F_SIMPLE_IO_BASE;
+#if CONFIG_VGA_ROM_RUN
+	rs690_vbios_regs vbios_regs;
 	u8 port2;
+#endif
 
 	printk(BIOS_INFO, "Mainboard tim5690 Enable. dev=0x%p\n", dev);
 
 	mb_gpio_init(&gpio_base);
 
+#if CONFIG_VGA_ROM_RUN
 	/* The LCD's panel id seletion by switch. */
 	port2 = inb(gpio_base+1);
 	lcd_panel_id(&vbios_regs, ((~port2) & 0xf));
@@ -237,6 +242,7 @@ static void tim5690_enable(device_t dev)
 	/* No support TV */
 	vbios_regs.int15_regs.fun05_tv_standard = TV_MODE_NO;
 	vgabios_init(&vbios_regs);
+#endif
 
 	set_thermal_config();
 }




More information about the coreboot mailing list