ar71xx: refactor PCI code to allow registering multiple PCI controllers
[openwrt/svn-archive/archive.git] / target / linux / ar71xx / patches-3.3 / 153-MIPS-pci-ar71xx-use-dynamically-allocated-PCI-contro.patch
diff --git a/target/linux/ar71xx/patches-3.3/153-MIPS-pci-ar71xx-use-dynamically-allocated-PCI-contro.patch b/target/linux/ar71xx/patches-3.3/153-MIPS-pci-ar71xx-use-dynamically-allocated-PCI-contro.patch
new file mode 100644 (file)
index 0000000..4db1fd9
--- /dev/null
@@ -0,0 +1,228 @@
+From 6c3ef689e4364dca74eaaecd72384be09e5a6bc8 Mon Sep 17 00:00:00 2001
+From: Gabor Juhos <juhosg@openwrt.org>
+Date: Mon, 25 Jun 2012 09:19:08 +0200
+Subject: [PATCH 14/34] MIPS: pci-ar71xx: use dynamically allocated PCI controller structure
+
+Signed-off-by: Gabor Juhos <juhosg@openwrt.org>
+---
+ arch/mips/pci/pci-ar71xx.c |   84 +++++++++++++++++++++++++++----------------
+ 1 files changed, 53 insertions(+), 31 deletions(-)
+
+--- a/arch/mips/pci/pci-ar71xx.c
++++ b/arch/mips/pci/pci-ar71xx.c
+@@ -20,6 +20,7 @@
+ #include <linux/interrupt.h>
+ #include <linux/module.h>
+ #include <linux/platform_device.h>
++#include <linux/slab.h>
+ #include <asm/mach-ath79/ar71xx_regs.h>
+ #include <asm/mach-ath79/ath79.h>
+@@ -48,8 +49,12 @@
+ #define AR71XX_PCI_IRQ_COUNT          5
+-static DEFINE_SPINLOCK(ar71xx_pci_lock);
+-static void __iomem *ar71xx_pcicfg_base;
++struct ar71xx_pci_controller {
++      void __iomem *cfg_base;
++      spinlock_t lock;
++      int irq;
++      struct pci_controller pci_ctrl;
++};
+ /* Byte lane enable bits */
+ static const u8 ar71xx_pci_ble_table[4][4] = {
+@@ -92,9 +97,18 @@ static inline u32 ar71xx_pci_bus_addr(st
+       return ret;
+ }
+-static int ar71xx_pci_check_error(int quiet)
++static inline struct ar71xx_pci_controller *
++pci_bus_to_ar71xx_controller(struct pci_bus *bus)
+ {
+-      void __iomem *base = ar71xx_pcicfg_base;
++      struct pci_controller *hose;
++
++      hose = (struct pci_controller *) bus->sysdata;
++      return container_of(hose, struct ar71xx_pci_controller, pci_ctrl);
++}
++
++static int ar71xx_pci_check_error(struct ar71xx_pci_controller *apc, int quiet)
++{
++      void __iomem *base = apc->cfg_base;
+       u32 pci_err;
+       u32 ahb_err;
+@@ -129,9 +143,10 @@ static int ar71xx_pci_check_error(int qu
+       return !!(ahb_err | pci_err);
+ }
+-static inline void ar71xx_pci_local_write(int where, int size, u32 value)
++static inline void ar71xx_pci_local_write(struct ar71xx_pci_controller *apc,
++                                        int where, int size, u32 value)
+ {
+-      void __iomem *base = ar71xx_pcicfg_base;
++      void __iomem *base = apc->cfg_base;
+       u32 ad_cbe;
+       value = value << (8 * (where & 3));
+@@ -147,7 +162,8 @@ static inline int ar71xx_pci_set_cfgaddr
+                                        unsigned int devfn,
+                                        int where, int size, u32 cmd)
+ {
+-      void __iomem *base = ar71xx_pcicfg_base;
++      struct ar71xx_pci_controller *apc = pci_bus_to_ar71xx_controller(bus);
++      void __iomem *base = apc->cfg_base;
+       u32 addr;
+       addr = ar71xx_pci_bus_addr(bus, devfn, where);
+@@ -156,13 +172,14 @@ static inline int ar71xx_pci_set_cfgaddr
+       __raw_writel(cmd | ar71xx_pci_get_ble(where, size, 0),
+                    base + AR71XX_PCI_REG_CFG_CBE);
+-      return ar71xx_pci_check_error(1);
++      return ar71xx_pci_check_error(apc, 1);
+ }
+ static int ar71xx_pci_read_config(struct pci_bus *bus, unsigned int devfn,
+                                 int where, int size, u32 *value)
+ {
+-      void __iomem *base = ar71xx_pcicfg_base;
++      struct ar71xx_pci_controller *apc = pci_bus_to_ar71xx_controller(bus);
++      void __iomem *base = apc->cfg_base;
+       unsigned long flags;
+       u32 data;
+       int err;
+@@ -171,7 +188,7 @@ static int ar71xx_pci_read_config(struct
+       ret = PCIBIOS_SUCCESSFUL;
+       data = ~0;
+-      spin_lock_irqsave(&ar71xx_pci_lock, flags);
++      spin_lock_irqsave(&apc->lock, flags);
+       err = ar71xx_pci_set_cfgaddr(bus, devfn, where, size,
+                                    AR71XX_PCI_CFG_CMD_READ);
+@@ -180,7 +197,7 @@ static int ar71xx_pci_read_config(struct
+       else
+               data = __raw_readl(base + AR71XX_PCI_REG_CFG_RDDATA);
+-      spin_unlock_irqrestore(&ar71xx_pci_lock, flags);
++      spin_unlock_irqrestore(&apc->lock, flags);
+       *value = (data >> (8 * (where & 3))) & ar71xx_pci_read_mask[size & 7];
+@@ -190,7 +207,8 @@ static int ar71xx_pci_read_config(struct
+ static int ar71xx_pci_write_config(struct pci_bus *bus, unsigned int devfn,
+                                  int where, int size, u32 value)
+ {
+-      void __iomem *base = ar71xx_pcicfg_base;
++      struct ar71xx_pci_controller *apc = pci_bus_to_ar71xx_controller(bus);
++      void __iomem *base = apc->cfg_base;
+       unsigned long flags;
+       int err;
+       int ret;
+@@ -198,7 +216,7 @@ static int ar71xx_pci_write_config(struc
+       value = value << (8 * (where & 3));
+       ret = PCIBIOS_SUCCESSFUL;
+-      spin_lock_irqsave(&ar71xx_pci_lock, flags);
++      spin_lock_irqsave(&apc->lock, flags);
+       err = ar71xx_pci_set_cfgaddr(bus, devfn, where, size,
+                                    AR71XX_PCI_CFG_CMD_WRITE);
+@@ -207,7 +225,7 @@ static int ar71xx_pci_write_config(struc
+       else
+               __raw_writel(value, base + AR71XX_PCI_REG_CFG_WRDATA);
+-      spin_unlock_irqrestore(&ar71xx_pci_lock, flags);
++      spin_unlock_irqrestore(&apc->lock, flags);
+       return ret;
+ }
+@@ -231,12 +249,6 @@ static struct resource ar71xx_pci_mem_re
+       .flags          = IORESOURCE_MEM
+ };
+-static struct pci_controller ar71xx_pci_controller = {
+-      .pci_ops        = &ar71xx_pci_ops,
+-      .mem_resource   = &ar71xx_pci_mem_resource,
+-      .io_resource    = &ar71xx_pci_io_resource,
+-};
+-
+ static void ar71xx_pci_irq_handler(unsigned int irq, struct irq_desc *desc)
+ {
+       void __iomem *base = ath79_reset_base;
+@@ -294,7 +306,7 @@ static struct irq_chip ar71xx_pci_irq_ch
+       .irq_mask_ack   = ar71xx_pci_irq_mask,
+ };
+-static __devinit void ar71xx_pci_irq_init(int irq)
++static __devinit void ar71xx_pci_irq_init(struct ar71xx_pci_controller *apc)
+ {
+       void __iomem *base = ath79_reset_base;
+       int i;
+@@ -309,7 +321,7 @@ static __devinit void ar71xx_pci_irq_ini
+               irq_set_chip_and_handler(i, &ar71xx_pci_irq_chip,
+                                        handle_level_irq);
+-      irq_set_chained_handler(irq, ar71xx_pci_irq_handler);
++      irq_set_chained_handler(apc->irq, ar71xx_pci_irq_handler);
+ }
+ static __devinit void ar71xx_pci_reset(void)
+@@ -336,20 +348,26 @@ static __devinit void ar71xx_pci_reset(v
+ static int __devinit ar71xx_pci_probe(struct platform_device *pdev)
+ {
++      struct ar71xx_pci_controller *apc;
+       struct resource *res;
+-      int irq;
+       u32 t;
++      apc = kzalloc(sizeof(struct ar71xx_pci_controller), GFP_KERNEL);
++      if (!apc)
++              return -ENOMEM;
++
++      spin_lock_init(&apc->lock);
++
+       res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "cfg_base");
+       if (!res)
+               return -EINVAL;
+-      ar71xx_pcicfg_base = devm_request_and_ioremap(&pdev->dev, res);
+-      if (!ar71xx_pcicfg_base)
++      apc->cfg_base = devm_request_and_ioremap(&pdev->dev, res);
++      if (!apc->cfg_base)
+               return -ENOMEM;
+-      irq = platform_get_irq(pdev, 0);
+-      if (irq < 0)
++      apc->irq = platform_get_irq(pdev, 0);
++      if (apc->irq < 0)
+               return -EINVAL;
+       ar71xx_pci_reset();
+@@ -357,14 +375,18 @@ static int __devinit ar71xx_pci_probe(st
+       /* setup COMMAND register */
+       t = PCI_COMMAND_MEMORY | PCI_COMMAND_MASTER | PCI_COMMAND_INVALIDATE
+         | PCI_COMMAND_PARITY | PCI_COMMAND_SERR | PCI_COMMAND_FAST_BACK;
+-      ar71xx_pci_local_write(PCI_COMMAND, 4, t);
++      ar71xx_pci_local_write(apc, PCI_COMMAND, 4, t);
+       /* clear bus errors */
+-      ar71xx_pci_check_error(1);
++      ar71xx_pci_check_error(apc, 1);
++
++      ar71xx_pci_irq_init(apc);
+-      ar71xx_pci_irq_init(irq);
++      apc->pci_ctrl.pci_ops = &ar71xx_pci_ops;
++      apc->pci_ctrl.mem_resource = &ar71xx_pci_mem_resource;
++      apc->pci_ctrl.io_resource = &ar71xx_pci_io_resource;
+-      register_pci_controller(&ar71xx_pci_controller);
++      register_pci_controller(&apc->pci_ctrl);
+       return 0;
+ }