[coreboot] New patch to review for coreboot: 2f34f0c Unbreak all boards where chipset is capable of selection of FSB/serial APIC bus
Rudolf Marek (r.marek@assembler.cz)
gerrit at coreboot.org
Mon Apr 23 00:00:46 CEST 2012
Rudolf Marek (r.marek at assembler.cz) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/921
-gerrit
commit 2f34f0c19b365f2cf1ac221dd6bdb667e92f5baf
Author: Rudolf Marek <r.marek at assembler.cz>
Date: Sun Apr 22 23:52:23 2012 +0200
Unbreak all boards where chipset is capable of selection of FSB/serial APIC bus
The commit Change I73c55755e0ec1ac5756b4ee7ccdfc8eb93184e4f contains #ifdef instead
of #if making the FSB/serial bus selection for APIC always to fall to serial APIC bus
This bug is not visible on most chipsets, because this bit is RO, but it do broke
VIA K8T890 chipsets.
Change-Id: I89c4855922199eca7f921c3e4eb500656544c8e5
Signed-off-by: Rudolf Marek <r.marek at assembler.cz>
---
src/arch/x86/lib/ioapic.c | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/arch/x86/lib/ioapic.c b/src/arch/x86/lib/ioapic.c
index e974d7c..57b99e2 100644
--- a/src/arch/x86/lib/ioapic.c
+++ b/src/arch/x86/lib/ioapic.c
@@ -89,7 +89,7 @@ void setup_ioapic(u32 ioapic_base, u8 ioapic_id)
ioapic_interrupts = 24;
printk(BIOS_DEBUG, "IOAPIC: %d interrupts\n", ioapic_interrupts);
-#ifdef CONFIG_IOAPIC_INTERRUPTS_ON_FSB
+#if CONFIG_IOAPIC_INTERRUPTS_ON_FSB
/*
* For the Pentium 4 and above APICs deliver their interrupts
* on the front side bus, enable that.
@@ -98,7 +98,7 @@ void setup_ioapic(u32 ioapic_base, u8 ioapic_id)
io_apic_write(ioapic_base, 0x03,
io_apic_read(ioapic_base, 0x03) | (1 << 0));
#endif
-#ifdef CONFIG_IOAPIC_INTERRUPTS_ON_APIC_SERIAL_BUS
+#if CONFIG_IOAPIC_INTERRUPTS_ON_APIC_SERIAL_BUS
printk(BIOS_DEBUG, "IOAPIC: Enabling interrupts on APIC serial bus\n");
io_apic_write(ioapic_base, 0x03, 0);
#endif
More information about the coreboot
mailing list