Commit d6aa484c authored by Alexander Chiang's avatar Alexander Chiang Committed by Len Brown

PCI Hotplug: acpiphp: convert to acpi_get_pci_dev

Now that acpi_get_pci_dev is available, let's use it instead of
acpi_get_pci_id.
Signed-off-by: default avatarAlex Chiang <achiang@hp.com>
Acked-by: default avatarBjorn Helgaas <bjorn.helgaas@hp.com>
Acked-by: default avatarJesse Barnes <jbarnes@virtuousgeek.org>
Signed-off-by: default avatarLen Brown <len.brown@intel.com>
parent 97719a87
...@@ -678,18 +678,9 @@ static void remove_bridge(acpi_handle handle) ...@@ -678,18 +678,9 @@ static void remove_bridge(acpi_handle handle)
static struct pci_dev * get_apic_pci_info(acpi_handle handle) static struct pci_dev * get_apic_pci_info(acpi_handle handle)
{ {
struct acpi_pci_id id;
struct pci_bus *bus;
struct pci_dev *dev; struct pci_dev *dev;
if (ACPI_FAILURE(acpi_get_pci_id(handle, &id))) dev = acpi_get_pci_dev(handle);
return NULL;
bus = pci_find_bus(id.segment, id.bus);
if (!bus)
return NULL;
dev = pci_get_slot(bus, PCI_DEVFN(id.device, id.function));
if (!dev) if (!dev)
return NULL; return NULL;
...@@ -1396,19 +1387,16 @@ static void acpiphp_sanitize_bus(struct pci_bus *bus) ...@@ -1396,19 +1387,16 @@ static void acpiphp_sanitize_bus(struct pci_bus *bus)
/* Program resources in newly inserted bridge */ /* Program resources in newly inserted bridge */
static int acpiphp_configure_bridge (acpi_handle handle) static int acpiphp_configure_bridge (acpi_handle handle)
{ {
struct acpi_pci_id pci_id; struct pci_dev *dev;
struct pci_bus *bus; struct pci_bus *bus;
if (ACPI_FAILURE(acpi_get_pci_id(handle, &pci_id))) { dev = acpi_get_pci_dev(handle);
if (!dev) {
err("cannot get PCI domain and bus number for bridge\n"); err("cannot get PCI domain and bus number for bridge\n");
return -EINVAL; return -EINVAL;
} }
bus = pci_find_bus(pci_id.segment, pci_id.bus);
if (!bus) { bus = dev->bus;
err("cannot find bus %d:%d\n",
pci_id.segment, pci_id.bus);
return -EINVAL;
}
pci_bus_size_bridges(bus); pci_bus_size_bridges(bus);
pci_bus_assign_resources(bus); pci_bus_assign_resources(bus);
...@@ -1416,6 +1404,7 @@ static int acpiphp_configure_bridge (acpi_handle handle) ...@@ -1416,6 +1404,7 @@ static int acpiphp_configure_bridge (acpi_handle handle)
acpiphp_set_hpp_values(handle, bus); acpiphp_set_hpp_values(handle, bus);
pci_enable_bridges(bus); pci_enable_bridges(bus);
acpiphp_configure_ioapics(handle); acpiphp_configure_ioapics(handle);
pci_dev_put(dev);
return 0; return 0;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment