Merge tag 'pci-v3.9-changes' of git://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci

Pull PCI changes from Bjorn Helgaas:
 "Host bridge hotplug
    - Major overhaul of ACPI host bridge add/start (Rafael Wysocki, Yinghai Lu)
    - Major overhaul of PCI/ACPI binding (Rafael Wysocki, Yinghai Lu)
    - Split out ACPI host bridge and ACPI PCI device hotplug (Yinghai Lu)
    - Stop caching _PRT and make independent of bus numbers (Yinghai Lu)

  PCI device hotplug
    - Clean up cpqphp dead code (Sasha Levin)
    - Disable ARI unless device and upstream bridge support it (Yijing Wang)
    - Initialize all hot-added devices (not functions 0-7) (Yijing Wang)

  Power management
    - Don't touch ASPM if disabled (Joe Lawrence)
    - Fix ASPM link state management (Myron Stowe)

  Miscellaneous
    - Fix PCI_EXP_FLAGS accessor (Alex Williamson)
    - Disable Bus Master in pci_device_shutdown (Konstantin Khlebnikov)
    - Document hotplug resource and MPS parameters (Yijing Wang)
    - Add accessor for PCIe capabilities (Myron Stowe)
    - Drop pciehp suspend/resume messages (Paul Bolle)
    - Make pci_slot built-in only (not a module) (Jiang Liu)
    - Remove unused PCI/ACPI bind ops (Jiang Liu)
    - Removed used pci_root_bus (Bjorn Helgaas)"

* tag 'pci-v3.9-changes' of git://git.kernel.org/pub/scm/linux/kernel/git/helgaas/pci: (51 commits)
  PCI/ACPI: Don't cache _PRT, and don't associate them with bus numbers
  PCI: Fix PCI Express Capability accessors for PCI_EXP_FLAGS
  ACPI / PCI: Make pci_slot built-in only, not a module
  PCI/PM: Clear state_saved during suspend
  PCI: Use atomic_inc_return() rather than atomic_add_return()
  PCI: Catch attempts to disable already-disabled devices
  PCI: Disable Bus Master unconditionally in pci_device_shutdown()
  PCI: acpiphp: Remove dead code for PCI host bridge hotplug
  PCI: acpiphp: Create companion ACPI devices before creating PCI devices
  PCI: Remove unused "rc" in virtfn_add_bus()
  PCI: pciehp: Drop suspend/resume ENTRY messages
  PCI/ASPM: Don't touch ASPM if forcibly disabled
  PCI/ASPM: Deallocate upstream link state even if device is not PCIe
  PCI: Document MPS parameters pci=pcie_bus_safe, pci=pcie_bus_perf, etc
  PCI: Document hpiosize= and hpmemsize= resource reservation parameters
  PCI: Use PCI Express Capability accessor
  PCI: Introduce accessor to retrieve PCIe Capabilities Register
  PCI: Put pci_dev in device tree as early as possible
  PCI: Skip attaching driver in device_add()
  PCI: acpiphp: Keep driver loaded even if no slots found
  ...

7 files changed, 135 insertions, 68 deletions

diff --git a/arch/x86/include/asm/pci.h b/arch/x86/include/asm/pci.h
index c28fd02f4bf7..d9e9e6c7ed32 100644
--- a/arch/x86/include/asm/pci.h
+++ b/arch/x86/include/asm/pci.h
@@ -14,6 +14,9 @@
 struct pci_sysdata {
         int             domain;         /* PCI domain */
         int             node;           /* NUMA node */
+#ifdef CONFIG_ACPI
+        void            *acpi;          /* ACPI-specific data */
+#endif
 #ifdef CONFIG_X86_64
         void            *iommu;         /* IOMMU private data */
 #endif
diff --git a/arch/x86/include/asm/pci_x86.h b/arch/x86/include/asm/pci_x86.h
index 747e5a38b590..fa1195dae425 100644
--- a/arch/x86/include/asm/pci_x86.h
+++ b/arch/x86/include/asm/pci_x86.h
@@ -54,7 +54,6 @@ void pcibios_set_cache_line_size(void);
 /* pci-pc.c */
 
 extern int pcibios_last_bus;
-extern struct pci_bus *pci_root_bus;
 extern struct pci_ops pci_root_ops;
 
 void pcibios_scan_specific_bus(int busn);
diff --git a/arch/x86/pci/acpi.c b/arch/x86/pci/acpi.c
index 53ea60458e01..3e724256dbee 100644
--- a/arch/x86/pci/acpi.c
+++ b/arch/x86/pci/acpi.c
@@ -521,6 +521,7 @@ struct pci_bus *pci_acpi_scan_root(struct acpi_pci_root *root)
         sd = &info->sd;
         sd->domain = domain;
         sd->node = node;
+        sd->acpi = device->handle;
         /*
          * Maybe the desired pci bus has been already scanned. In such case
          * it is unnecessary to scan the pci bus with the given domain,busnum.
@@ -592,6 +593,14 @@ struct pci_bus *pci_acpi_scan_root(struct acpi_pci_root *root)
         return bus;
 }
 
+int pcibios_root_bridge_prepare(struct pci_host_bridge *bridge)
+{
+        struct pci_sysdata *sd = bridge->bus->sysdata;
+
+        ACPI_HANDLE_SET(&bridge->dev, sd->acpi);
+        return 0;
+}
+
 int __init pci_acpi_init(void)
 {
         struct pci_dev *dev = NULL;
diff --git a/arch/x86/pci/common.c b/arch/x86/pci/common.c
index ccd0ab3ab899..901177d75ff5 100644
--- a/arch/x86/pci/common.c
+++ b/arch/x86/pci/common.c
@@ -34,7 +34,6 @@ int noioapicreroute = 1;
 #endif
 int pcibios_last_bus = -1;
 unsigned long pirq_table_addr;
-struct pci_bus *pci_root_bus;
 const struct pci_raw_ops *__read_mostly raw_pci_ops;
 const struct pci_raw_ops *__read_mostly raw_pci_ext_ops;
 
diff --git a/arch/x86/pci/i386.c b/arch/x86/pci/i386.c
index dd8ca6f7223b..94919e307f8e 100644
--- a/arch/x86/pci/i386.c
+++ b/arch/x86/pci/i386.c
@@ -51,6 +51,7 @@ struct pcibios_fwaddrmap {
 
 static LIST_HEAD(pcibios_fwaddrmappings);
 static DEFINE_SPINLOCK(pcibios_fwaddrmap_lock);
+static bool pcibios_fw_addr_done;
 
 /* Must be called with 'pcibios_fwaddrmap_lock' lock held. */
 static struct pcibios_fwaddrmap *pcibios_fwaddrmap_lookup(struct pci_dev *dev)
@@ -72,6 +73,9 @@ pcibios_save_fw_addr(struct pci_dev *dev, int idx, resource_size_t fw_addr)
         unsigned long flags;
         struct pcibios_fwaddrmap *map;
 
+        if (pcibios_fw_addr_done)
+                return;
+
         spin_lock_irqsave(&pcibios_fwaddrmap_lock, flags);
         map = pcibios_fwaddrmap_lookup(dev);
         if (!map) {
@@ -97,6 +101,9 @@ resource_size_t pcibios_retrieve_fw_addr(struct pci_dev *dev, int idx)
         struct pcibios_fwaddrmap *map;
         resource_size_t fw_addr = 0;
 
+        if (pcibios_fw_addr_done)
+                return 0;
+
         spin_lock_irqsave(&pcibios_fwaddrmap_lock, flags);
         map = pcibios_fwaddrmap_lookup(dev);
         if (map)
@@ -106,7 +113,7 @@ resource_size_t pcibios_retrieve_fw_addr(struct pci_dev *dev, int idx)
         return fw_addr;
 }
 
-static void pcibios_fw_addr_list_del(void)
+static void __init pcibios_fw_addr_list_del(void)
 {
         unsigned long flags;
         struct pcibios_fwaddrmap *entry, *next;
@@ -118,6 +125,7 @@ static void pcibios_fw_addr_list_del(void)
                 kfree(entry);
         }
         spin_unlock_irqrestore(&pcibios_fwaddrmap_lock, flags);
+        pcibios_fw_addr_done = true;
 }
 
 static int
@@ -193,46 +201,46 @@ EXPORT_SYMBOL(pcibios_align_resource);
  *          as well.
  */
 
-static void __init pcibios_allocate_bus_resources(struct list_head *bus_list)
+static void pcibios_allocate_bridge_resources(struct pci_dev *dev)
 {
-        struct pci_bus *bus;
-        struct pci_dev *dev;
         int idx;
         struct resource *r;
 
-        /* Depth-First Search on bus tree */
-        list_for_each_entry(bus, bus_list, node) {
-                if ((dev = bus->self)) {
-                        for (idx = PCI_BRIDGE_RESOURCES;
-                            idx < PCI_NUM_RESOURCES; idx++) {
-                                r = &dev->resource[idx];
-                                if (!r->flags)
-                                        continue;
-                                if (!r->start ||
-                                    pci_claim_resource(dev, idx) < 0) {
-                                        /*
-                                         * Something is wrong with the region.
-                                         * Invalidate the resource to prevent
-                                         * child resource allocations in this
-                                         * range.
-                                         */
-                                        r->start = r->end = 0;
-                                        r->flags = 0;
-                                }
-                        }
+        for (idx = PCI_BRIDGE_RESOURCES; idx < PCI_NUM_RESOURCES; idx++) {
+                r = &dev->resource[idx];
+                if (!r->flags)
+                        continue;
+                if (!r->start || pci_claim_resource(dev, idx) < 0) {
+                        /*
+                         * Something is wrong with the region.
+                         * Invalidate the resource to prevent
+                         * child resource allocations in this
+                         * range.
+                         */
+                        r->start = r->end = 0;
+                        r->flags = 0;
                 }
-                pcibios_allocate_bus_resources(&bus->children);
         }
 }
 
+static void pcibios_allocate_bus_resources(struct pci_bus *bus)
+{
+        struct pci_bus *child;
+
+        /* Depth-First Search on bus tree */
+        if (bus->self)
+                pcibios_allocate_bridge_resources(bus->self);
+        list_for_each_entry(child, &bus->children, node)
+                pcibios_allocate_bus_resources(child);
+}
+
 struct pci_check_idx_range {
         int start;
         int end;
 };
 
-static void __init pcibios_allocate_resources(int pass)
+static void pcibios_allocate_dev_resources(struct pci_dev *dev, int pass)
 {
-        struct pci_dev *dev = NULL;
         int idx, disabled, i;
         u16 command;
         struct resource *r;
@@ -244,14 +252,13 @@ static void __init pcibios_allocate_resources(int pass)
 #endif
         };
 
-        for_each_pci_dev(dev) {
-                pci_read_config_word(dev, PCI_COMMAND, &command);
-                for (i = 0; i < ARRAY_SIZE(idx_range); i++)
+        pci_read_config_word(dev, PCI_COMMAND, &command);
+        for (i = 0; i < ARRAY_SIZE(idx_range); i++)
                 for (idx = idx_range[i].start; idx <= idx_range[i].end; idx++) {
                         r = &dev->resource[idx];
-                        if (r->parent)          /* Already allocated */
+                        if (r->parent)  /* Already allocated */
                                 continue;
-                        if (!r->start)          /* Address not assigned at all */
+                        if (!r->start)  /* Address not assigned at all */
                                 continue;
                         if (r->flags & IORESOURCE_IO)
                                 disabled = !(command & PCI_COMMAND_IO);
@@ -270,44 +277,74 @@ static void __init pcibios_allocate_resources(int pass)
                                 }
                         }
                 }
-                if (!pass) {
-                        r = &dev->resource[PCI_ROM_RESOURCE];
-                        if (r->flags & IORESOURCE_ROM_ENABLE) {
-                                /* Turn the ROM off, leave the resource region,
-                                 * but keep it unregistered. */
-                                u32 reg;
-                                dev_dbg(&dev->dev, "disabling ROM %pR\n", r);
-                                r->flags &= ~IORESOURCE_ROM_ENABLE;
-                                pci_read_config_dword(dev,
-                                                dev->rom_base_reg, &reg);
-                                pci_write_config_dword(dev, dev->rom_base_reg,
+        if (!pass) {
+                r = &dev->resource[PCI_ROM_RESOURCE];
+                if (r->flags & IORESOURCE_ROM_ENABLE) {
+                        /* Turn the ROM off, leave the resource region,
+                         * but keep it unregistered. */
+                        u32 reg;
+                        dev_dbg(&dev->dev, "disabling ROM %pR\n", r);
+                        r->flags &= ~IORESOURCE_ROM_ENABLE;
+                        pci_read_config_dword(dev, dev->rom_base_reg, &reg);
+                        pci_write_config_dword(dev, dev->rom_base_reg,
                                                 reg & ~PCI_ROM_ADDRESS_ENABLE);
-                        }
                 }
         }
 }
 
-static int __init pcibios_assign_resources(void)
+static void pcibios_allocate_resources(struct pci_bus *bus, int pass)
+{
+        struct pci_dev *dev;
+        struct pci_bus *child;
+
+        list_for_each_entry(dev, &bus->devices, bus_list) {
+                pcibios_allocate_dev_resources(dev, pass);
+
+                child = dev->subordinate;
+                if (child)
+                        pcibios_allocate_resources(child, pass);
+        }
+}
+
+static void pcibios_allocate_dev_rom_resource(struct pci_dev *dev)
 {
-        struct pci_dev *dev = NULL;
         struct resource *r;
 
-        if (!(pci_probe & PCI_ASSIGN_ROMS)) {
-                /*
-                 * Try to use BIOS settings for ROMs, otherwise let
-                 * pci_assign_unassigned_resources() allocate the new
-                 * addresses.
-                 */
-                for_each_pci_dev(dev) {
-                        r = &dev->resource[PCI_ROM_RESOURCE];
-                        if (!r->flags || !r->start)
-                                continue;
-                        if (pci_claim_resource(dev, PCI_ROM_RESOURCE) < 0) {
-                                r->end -= r->start;
-                                r->start = 0;
-                        }
-                }
+        /*
+         * Try to use BIOS settings for ROMs, otherwise let
+         * pci_assign_unassigned_resources() allocate the new
+         * addresses.
+         */
+        r = &dev->resource[PCI_ROM_RESOURCE];
+        if (!r->flags || !r->start)
+                return;
+
+        if (pci_claim_resource(dev, PCI_ROM_RESOURCE) < 0) {
+                r->end -= r->start;
+                r->start = 0;
         }
+}
+static void pcibios_allocate_rom_resources(struct pci_bus *bus)
+{
+        struct pci_dev *dev;
+        struct pci_bus *child;
+
+        list_for_each_entry(dev, &bus->devices, bus_list) {
+                pcibios_allocate_dev_rom_resource(dev);
+
+                child = dev->subordinate;
+                if (child)
+                        pcibios_allocate_rom_resources(child);
+        }
+}
+
+static int __init pcibios_assign_resources(void)
+{
+        struct pci_bus *bus;
+
+        if (!(pci_probe & PCI_ASSIGN_ROMS))
+                list_for_each_entry(bus, &pci_root_buses, node)
+                        pcibios_allocate_rom_resources(bus);
 
         pci_assign_unassigned_resources();
         pcibios_fw_addr_list_del();
@@ -315,12 +352,32 @@ static int __init pcibios_assign_resources(void)
         return 0;
 }
 
+void pcibios_resource_survey_bus(struct pci_bus *bus)
+{
+        dev_printk(KERN_DEBUG, &bus->dev, "Allocating resources\n");
+
+        pcibios_allocate_bus_resources(bus);
+
+        pcibios_allocate_resources(bus, 0);
+        pcibios_allocate_resources(bus, 1);
+
+        if (!(pci_probe & PCI_ASSIGN_ROMS))
+                pcibios_allocate_rom_resources(bus);
+}
+
 void __init pcibios_resource_survey(void)
 {
+        struct pci_bus *bus;
+
         DBG("PCI: Allocating resources\n");
-        pcibios_allocate_bus_resources(&pci_root_buses);
-        pcibios_allocate_resources(0);
-        pcibios_allocate_resources(1);
+
+        list_for_each_entry(bus, &pci_root_buses, node)
+                pcibios_allocate_bus_resources(bus);
+
+        list_for_each_entry(bus, &pci_root_buses, node)
+                pcibios_allocate_resources(bus, 0);
+        list_for_each_entry(bus, &pci_root_buses, node)
+                pcibios_allocate_resources(bus, 1);
 
         e820_reserve_resources_late();
         /*
diff --git a/arch/x86/pci/legacy.c b/arch/x86/pci/legacy.c
index 4a2ab9cb3659..4db96fb1c232 100644
--- a/arch/x86/pci/legacy.c
+++ b/arch/x86/pci/legacy.c
@@ -30,7 +30,7 @@ int __init pci_legacy_init(void)
         }
 
         printk("PCI: Probing PCI hardware\n");
-        pci_root_bus = pcibios_scan_root(0);
+        pcibios_scan_root(0);
         return 0;
 }
 
diff --git a/arch/x86/pci/numaq_32.c b/arch/x86/pci/numaq_32.c
index b96b14c250b6..72c229f9ebcf 100644
--- a/arch/x86/pci/numaq_32.c
+++ b/arch/x86/pci/numaq_32.c
@@ -152,7 +152,7 @@ int __init pci_numaq_init(void)
 
         raw_pci_ops = &pci_direct_conf1_mq;
 
-        pci_root_bus = pcibios_scan_root(0);
+        pcibios_scan_root(0);
         if (num_online_nodes() > 1)
                 for_each_online_node(quad) {
                         if (quad == 0)