summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorArthur Heymans <arthur@aheymans.xyz>2017-04-12 10:53:30 +0200
committerMartin Roth <martinroth@google.com>2017-07-25 15:15:28 +0000
commit87af36ac172b0ec48b078453ed4f2fd8d2a62a22 (patch)
treeb0756abcf0e6d5c9add3360fca6f030fe47931bd
parent7d96565190c7afbca6eb1ed69b1191597b6e0644 (diff)
downloadcoreboot-87af36ac172b0ec48b078453ed4f2fd8d2a62a22.tar.xz
sb/intel/i82801jx: Route all PIRQ to INT11
Interrupt 11 is not used by legacy devices and so can always be used for PCI interrupts. Full legacy IRQ routing is complicated and hard to get right. Change-Id: I6c718f4b9fb91ffcc4a136120581a4fcd7ec7231 Signed-off-by: Arthur Heymans <arthur@aheymans.xyz> Reviewed-on: https://review.coreboot.org/19254 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Martin Roth <martinroth@google.com>
-rw-r--r--src/southbridge/intel/i82801jx/chip.h13
-rw-r--r--src/southbridge/intel/i82801jx/lpc.c41
2 files changed, 21 insertions, 33 deletions
diff --git a/src/southbridge/intel/i82801jx/chip.h b/src/southbridge/intel/i82801jx/chip.h
index 95d0d75cc1..533254a872 100644
--- a/src/southbridge/intel/i82801jx/chip.h
+++ b/src/southbridge/intel/i82801jx/chip.h
@@ -24,19 +24,6 @@ enum {
struct southbridge_intel_i82801jx_config {
/**
- * Interrupt Routing configuration
- * If bit7 is 1, the interrupt is disabled.
- */
- uint8_t pirqa_routing;
- uint8_t pirqb_routing;
- uint8_t pirqc_routing;
- uint8_t pirqd_routing;
- uint8_t pirqe_routing;
- uint8_t pirqf_routing;
- uint8_t pirqg_routing;
- uint8_t pirqh_routing;
-
- /**
* GPI Routing configuration
*
* Only the lower two bits have a meaning:
diff --git a/src/southbridge/intel/i82801jx/lpc.c b/src/southbridge/intel/i82801jx/lpc.c
index bd50366751..18025ff5dc 100644
--- a/src/southbridge/intel/i82801jx/lpc.c
+++ b/src/southbridge/intel/i82801jx/lpc.c
@@ -94,42 +94,43 @@ static void i82801jx_enable_serial_irqs(struct device *dev)
static void i82801jx_pirq_init(device_t dev)
{
device_t irq_dev;
- /* Get the chip configuration */
- config_t *config = dev->chip_info;
- pci_write_config8(dev, D31F0_PIRQA_ROUT, config->pirqa_routing);
- pci_write_config8(dev, D31F0_PIRQB_ROUT, config->pirqb_routing);
- pci_write_config8(dev, D31F0_PIRQC_ROUT, config->pirqc_routing);
- pci_write_config8(dev, D31F0_PIRQD_ROUT, config->pirqd_routing);
+ /* Interrupt 11 is not used by legacy devices and so can always be used
+ * for PCI interrupts. Full legacy IRQ routing is complicated and hard
+ * to get right. Fortunately all modern OS use MSI and so it's not that
+ * big of an issue anyway. Still we have to provide a reasonable
+ * default. Using interrupt 11 for it everywhere is a working default.
+ * ACPI-aware OS can move it to any interrupt and others will just leave
+ * them at default.
+ */
+ const u8 pirq_routing = 11;
+
+ pci_write_config8(dev, D31F0_PIRQA_ROUT, pirq_routing);
+ pci_write_config8(dev, D31F0_PIRQB_ROUT, pirq_routing);
+ pci_write_config8(dev, D31F0_PIRQC_ROUT, pirq_routing);
+ pci_write_config8(dev, D31F0_PIRQD_ROUT, pirq_routing);
- pci_write_config8(dev, D31F0_PIRQE_ROUT, config->pirqe_routing);
- pci_write_config8(dev, D31F0_PIRQF_ROUT, config->pirqf_routing);
- pci_write_config8(dev, D31F0_PIRQG_ROUT, config->pirqg_routing);
- pci_write_config8(dev, D31F0_PIRQH_ROUT, config->pirqh_routing);
+ pci_write_config8(dev, D31F0_PIRQE_ROUT, pirq_routing);
+ pci_write_config8(dev, D31F0_PIRQF_ROUT, pirq_routing);
+ pci_write_config8(dev, D31F0_PIRQG_ROUT, pirq_routing);
+ pci_write_config8(dev, D31F0_PIRQH_ROUT, pirq_routing);
/* Eric Biederman once said we should let the OS do this.
* I am not so sure anymore he was right.
*/
for (irq_dev = all_devices; irq_dev; irq_dev = irq_dev->next) {
- u8 int_pin=0, int_line=0;
+ u8 int_pin = 0;
if (!irq_dev->enabled || irq_dev->path.type != DEVICE_PATH_PCI)
continue;
int_pin = pci_read_config8(irq_dev, PCI_INTERRUPT_PIN);
- switch (int_pin) {
- case 1: /* INTA# */ int_line = config->pirqa_routing; break;
- case 2: /* INTB# */ int_line = config->pirqb_routing; break;
- case 3: /* INTC# */ int_line = config->pirqc_routing; break;
- case 4: /* INTD# */ int_line = config->pirqd_routing; break;
- }
-
- if (!int_line)
+ if (int_pin == 0)
continue;
- pci_write_config8(irq_dev, PCI_INTERRUPT_LINE, int_line);
+ pci_write_config8(irq_dev, PCI_INTERRUPT_LINE, pirq_routing);
}
}