[coreboot-gerrit] Change in coreboot[master]: sb/ti/pci7420: Drop dead code

[Angel Pons (Code Review)]

Angel Pons has submitted this change. ( https://review.coreboot.org/c/coreboot/+/43219 )

Change subject: sb/ti/pci7420: Drop dead code
......................................................................

sb/ti/pci7420: Drop dead code

This code is not even being build-tested. Drop it before it grows moss.

Change-Id: I51a2b71abc7762b550f69f2980dd34f0e4947ab5
Signed-off-by: Angel Pons 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/43219
Reviewed-by: Michael Niewöhner
Tested-by: build bot (Jenkins) 
---
M src/southbridge/ti/pci7420/cardbus.c
M src/southbridge/ti/pci7420/firewire.c
2 files changed, 0 insertions(+), 27 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Michael Niewöhner: Looks good to me, approved

diff --git a/src/southbridge/ti/pci7420/cardbus.c b/src/southbridge/ti/pci7420/cardbus.c
index 02c4a01..6ef08a3 100644
--- a/src/southbridge/ti/pci7420/cardbus.c
+++ b/src/southbridge/ti/pci7420/cardbus.c
@@ -8,10 +8,6 @@
 #include "pci7420.h"
 #include "chip.h"

-#ifdef ODD_IRQ_FIXUP
-static int cardbus_count = 0;
-#endif
-
 static void pci7420_cardbus_init(struct device *dev)
 {
    u8 reg8;
@@ -47,20 +43,6 @@

/* Multifunction routing status */
    pci_write_config32(dev, MFUNC, 0x018a1b22);
-
-#ifdef ODD_IRQ_FIXUP
-	/* This is a workaround for buggy kernels. This should
-	 * probably be read from the device tree, but as long
-	 * as only one mainboard is using this bridge it does
-	 * not matter.
-	 *
-	 * Basically what we do here is assign INTA to the first
-	 * cardbus controller, and INTB to the second one. We know
-	 * there are only two of them.
-	 */
-	pci_write_config8(dev, PCI_INTERRUPT_PIN, cardbus_count);
-	cardbus_count++;
-#endif
 }

static void pci7420_cardbus_read_resources(struct device *dev)
diff --git a/src/southbridge/ti/pci7420/firewire.c b/src/southbridge/ti/pci7420/firewire.c
index ef06ea6..a327745 100644
--- a/src/southbridge/ti/pci7420/firewire.c
+++ b/src/southbridge/ti/pci7420/firewire.c
@@ -11,15 +11,6 @@
 static void pci7420_firewire_init(struct device *dev)
 {
    printk(BIOS_DEBUG, "TI PCI7420/7620 FireWire init\n");
-
-#ifdef ODD_IRQ_FIXUP
-	/* This is a workaround for buggy kernels. This should
-	 * probably be read from the device tree, but as long
-	 * as only one mainboard is using this bridge it does
-	 * not matter
-	 */
-	pci_write_config8(dev, PCI_INTERRUPT_PIN, INTC);
-#endif
 }

static struct device_operations ti_pci7420_firewire_ops = {