PCI: Add pci_assign_irq() function and have pci_fixup_irqs() use it
authorMatthew Minter <matt@masarand.com>
Wed, 28 Jun 2017 20:14:02 +0000 (15:14 -0500)
committerBjorn Helgaas <bhelgaas@google.com>
Sun, 2 Jul 2017 21:14:28 +0000 (16:14 -0500)
Here we delete the static pdev_fixup_irq() function which is currently what
pci_fixup_irqs() uses to actually assign the IRQs and replace it with the
pci_assign_irq() function which changes the interface and uses the new
function pointers stored in the host bridge structure.

Eventually this will allow pci_fixup_irqs() to be removed entirely and the
new deferred assignment code path will call pci_assign_irq() directly.
However to ensure current users continue to work, a new implementation of
pci_fixup_irqs() is introduced which simply wraps the functionality of
pci_assign_irq().

Signed-off-by: Matthew Minter <matt@masarand.com>
[lorenzo.pieralisi@arm.com: reworked comments/log]
Signed-off-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Signed-off-by: Bjorn Helgaas <bhelgaas@google.com>
drivers/pci/setup-irq.c
include/linux/pci.h

index 95c225be49d17c82483ce12aef9aca2cddcdb7d1..81eda3d93a5dca78b3342a75c312af0ddfb128e3 100644 (file)
@@ -15,6 +15,7 @@
 #include <linux/errno.h>
 #include <linux/ioport.h>
 #include <linux/cache.h>
+#include "pci.h"
 
 void __weak pcibios_update_irq(struct pci_dev *dev, int irq)
 {
@@ -22,12 +23,17 @@ void __weak pcibios_update_irq(struct pci_dev *dev, int irq)
        pci_write_config_byte(dev, PCI_INTERRUPT_LINE, irq);
 }
 
-static void pdev_fixup_irq(struct pci_dev *dev,
-                          u8 (*swizzle)(struct pci_dev *, u8 *),
-                          int (*map_irq)(const struct pci_dev *, u8, u8))
+void pci_assign_irq(struct pci_dev *dev)
 {
-       u8 pin, slot;
+       u8 pin;
+       u8 slot = -1;
        int irq = 0;
+       struct pci_host_bridge *hbrg = pci_find_host_bridge(dev->bus);
+
+       if (!(hbrg->map_irq)) {
+               dev_dbg(&dev->dev, "runtime IRQ mapping not provided by arch\n");
+               return;
+       }
 
        /* If this device is not on the primary bus, we need to figure out
           which interrupt pin it will come in on.   We know which slot it
@@ -40,17 +46,22 @@ static void pdev_fixup_irq(struct pci_dev *dev,
        if (pin > 4)
                pin = 1;
 
-       if (pin != 0) {
+       if (pin) {
                /* Follow the chain of bridges, swizzling as we go.  */
-               slot = (*swizzle)(dev, &pin);
+               if (hbrg->swizzle_irq)
+                       slot = (*(hbrg->swizzle_irq))(dev, &pin);
 
-               irq = (*map_irq)(dev, slot, pin);
+               /*
+                * If a swizzling function is not used map_irq must
+                * ignore slot
+                */
+               irq = (*(hbrg->map_irq))(dev, slot, pin);
                if (irq == -1)
                        irq = 0;
        }
        dev->irq = irq;
 
-       dev_dbg(&dev->dev, "fixup irq: got %d\n", dev->irq);
+       dev_dbg(&dev->dev, "assign IRQ: got %d\n", dev->irq);
 
        /* Always tell the device, so the driver knows what is
           the real IRQ to use; the device does not use it. */
@@ -60,9 +71,23 @@ static void pdev_fixup_irq(struct pci_dev *dev,
 void pci_fixup_irqs(u8 (*swizzle)(struct pci_dev *, u8 *),
                    int (*map_irq)(const struct pci_dev *, u8, u8))
 {
+       /*
+        * Implement pci_fixup_irqs() through pci_assign_irq().
+        * This code should be remove eventually, it is a wrapper
+        * around pci_assign_irq() interface to keep current
+        * pci_fixup_irqs() behaviour unchanged on architecture
+        * code still relying on its interface.
+        */
        struct pci_dev *dev = NULL;
+       struct pci_host_bridge *hbrg = NULL;
 
-       for_each_pci_dev(dev)
-               pdev_fixup_irq(dev, swizzle, map_irq);
+       for_each_pci_dev(dev) {
+               hbrg = pci_find_host_bridge(dev->bus);
+               hbrg->swizzle_irq = swizzle;
+               hbrg->map_irq = map_irq;
+               pci_assign_irq(dev);
+               hbrg->swizzle_irq = NULL;
+               hbrg->map_irq = NULL;
+       }
 }
 EXPORT_SYMBOL_GPL(pci_fixup_irqs);
index 3c5d8b026d6e6392d5c44f720300d3c14a723454..5c1c0ae38dd3440d19105a1e49852912b1af16c7 100644 (file)
@@ -1147,6 +1147,7 @@ void pdev_enable_device(struct pci_dev *);
 int pci_enable_resources(struct pci_dev *, int mask);
 void pci_fixup_irqs(u8 (*)(struct pci_dev *, u8 *),
                    int (*)(const struct pci_dev *, u8, u8));
+void pci_assign_irq(struct pci_dev *dev);
 struct resource *pci_find_resource(struct pci_dev *dev, struct resource *res);
 #define HAVE_PCI_REQ_REGIONS   2
 int __must_check pci_request_regions(struct pci_dev *, const char *);