[POWERPC] Merge PCI resource fixups

The PCI code in 32 and 64 bits fixes up resources differently.

32 bits uses a header quirk plus handles bridges in pcibios_fixup_bus()
while 64 bits does things in various places depending on whether you
are using OF probing, using PCI hotplug, etc...

This merges those by basically using the 32 bits approach for both,
with various tweaks to make 64 bits work with the new approach.

Signed-off-by: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Signed-off-by: Paul Mackerras <paulus@samba.org>

1 files changed, 2 insertions, 81 deletions

diff --git a/arch/powerpc/kernel/pci_32.c b/arch/powerpc/kernel/pci_32.c
index f05ef5b17892..717c554d4658 100644
--- a/arch/powerpc/kernel/pci_32.c
+++ b/arch/powerpc/kernel/pci_32.c
@@ -40,7 +40,6 @@ unsigned int ppc_pci_flags;
 
 void pcibios_make_OF_bus_map(void);
 
-static void pcibios_fixup_resources(struct pci_dev* dev);
 static void fixup_broken_pcnet32(struct pci_dev* dev);
 static int reparent_resources(struct resource *parent, struct resource *res);
 static void fixup_cpc710_pci64(struct pci_dev* dev);
@@ -98,53 +97,6 @@ fixup_cpc710_pci64(struct pci_dev* dev)
 }
 DECLARE_PCI_FIXUP_HEADER(PCI_VENDOR_ID_IBM,     PCI_DEVICE_ID_IBM_CPC710_PCI64, fixup_cpc710_pci64);
 
-static void
-pcibios_fixup_resources(struct pci_dev *dev)
-{
-        struct pci_controller* hose = (struct pci_controller *)dev->sysdata;
-        int i;
-        resource_size_t offset, mask;
-
-        if (!hose) {
-                printk(KERN_ERR "No hose for PCI dev %s!\n", pci_name(dev));
-                return;
-        }
-        for (i = 0; i < DEVICE_COUNT_RESOURCE; i++) {
-                struct resource *res = dev->resource + i;
-                if (!res->flags)
-                        continue;
-                if (res->end == 0xffffffff) {
-                        DBG("PCI:%s Resource %d [%016llx-%016llx] is unassigned\n",
-                            pci_name(dev), i, (u64)res->start, (u64)res->end);
-                        res->end -= res->start;
-                        res->start = 0;
-                        res->flags |= IORESOURCE_UNSET;
-                        continue;
-                }
-                offset = 0;
-                mask = (resource_size_t)-1;
-                if (res->flags & IORESOURCE_MEM) {
-                        offset = hose->pci_mem_offset;
-                } else if (res->flags & IORESOURCE_IO) {
-                        offset = (unsigned long) hose->io_base_virt
-                                - isa_io_base;
-                        mask = 0xffffffffu;
-                }
-                if (offset != 0) {
-                        res->start = (res->start + offset) & mask;
-                        res->end = (res->end + offset) & mask;
-                        DBG("PCI: Fixup res %d (0x%lx) of dev %s: %llx -> %llx\n",
-                            i, res->flags, pci_name(dev),
-                            (u64)res->start - offset, (u64)res->start);
-                }
-        }
-
-        /* Call machine specific resource fixup */
-        if (ppc_md.pcibios_fixup_resources)
-                ppc_md.pcibios_fixup_resources(dev);
-}
-DECLARE_PCI_FIXUP_HEADER(PCI_ANY_ID,            PCI_ANY_ID,                     pcibios_fixup_resources);
-
 static int skip_isa_ioresource_align(struct pci_dev *dev)
 {
         if ((ppc_pci_flags & PPC_PCI_CAN_SKIP_ISA_ALIGN) &&
@@ -757,14 +709,14 @@ pcibios_init(void)
 
 subsys_initcall(pcibios_init);
 
-void pcibios_fixup_bus(struct pci_bus *bus)
+void __devinit pcibios_do_bus_setup(struct pci_bus *bus)
 {
         struct pci_controller *hose = (struct pci_controller *) bus->sysdata;
         unsigned long io_offset;
         struct resource *res;
-        struct pci_dev *dev;
         int i;
 
+        /* Hookup PHB resources */
         io_offset = (unsigned long)hose->io_base_virt - isa_io_base;
         if (bus->parent == NULL) {
                 /* This is a host bridge - fill in its resources */
@@ -795,37 +747,6 @@ void pcibios_fixup_bus(struct pci_bus *bus)
                         }
                         bus->resource[i+1] = res;
                 }
-        } else {
-                /* This is a subordinate bridge */
-                pci_read_bridge_bases(bus);
-
-                for (i = 0; i < 4; ++i) {
-                        if ((res = bus->resource[i]) == NULL)
-                                continue;
-                        if (!res->flags || bus->self->transparent)
-                                continue;
-                        if (io_offset && (res->flags & IORESOURCE_IO)) {
-                                res->start = (res->start + io_offset) &
-                                        0xffffffffu;
-                                res->end = (res->end + io_offset) &
-                                        0xffffffffu;
-                        } else if (hose->pci_mem_offset
-                                   && (res->flags & IORESOURCE_MEM)) {
-                                res->start += hose->pci_mem_offset;
-                                res->end += hose->pci_mem_offset;
-                        }
-                }
-        }
-
-        /* Platform specific bus fixups */
-        if (ppc_md.pcibios_fixup_bus)
-                ppc_md.pcibios_fixup_bus(bus);
-
-        /* Read default IRQs and fixup if necessary */
-        list_for_each_entry(dev, &bus->devices, bus_list) {
-                pci_read_irq_line(dev);
-                if (ppc_md.pci_irq_fixup)
-                        ppc_md.pci_irq_fixup(dev);
         }
 }