[coreboot-gerrit] Patch set updated for coreboot: b8fa838 Intel 82801gx: Use 2 << 24 to clarify that I/O APIC ID is 2
Paul Menzel (paulepanter@users.sourceforge.net)
gerrit at coreboot.org
Tue Apr 23 13:22:19 CEST 2013
Paul Menzel (paulepanter at users.sourceforge.net) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/3122
-gerrit
commit b8fa8380d68125dcb892fc1fa61156e72760be3b
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date: Tue Apr 23 13:00:34 2013 +0200
Intel 82801gx: Use 2 << 24 to clarify that I/O APIC ID is 2
Commit »Support for the Intel ICH7 southbridge.« (debb11fc) [1] used
`1 << 25` to set the I/O APIC ID of 2. Instead using `2 << 24`, which
is the same value, makes it clear, that the I/O APIC ID is 2.
Commit »Intel Panther Point PCH: Use 2 << 24 to clarify that APIC ID
is 2« (8c937c7e) [2] is used as a template.
[1] http://review.coreboot.org/gitweb?p=coreboot.git;a=commit;h=debb11fc1fe5f5560015ab9905f1ccc2e08c73e0
[2] http://review.coreboot.org/3100
Change-Id: Ib688500944cd78a1cc1c8082bb138fa9468bdbfb
Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
---
src/southbridge/intel/i82801gx/lpc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/southbridge/intel/i82801gx/lpc.c b/src/southbridge/intel/i82801gx/lpc.c
index 3a94e84..a4dd523 100644
--- a/src/southbridge/intel/i82801gx/lpc.c
+++ b/src/southbridge/intel/i82801gx/lpc.c
@@ -52,7 +52,7 @@ static void i82801gx_enable_apic(struct device *dev)
pci_write_config8(dev, ACPI_CNTL, ACPI_EN);
*ioapic_index = 0;
- *ioapic_data = (1 << 25);
+ *ioapic_data = (2 << 24);
*ioapic_index = 0;
reg32 = *ioapic_data;
More information about the coreboot-gerrit
mailing list