ACPI / hotplug / PCI: Drop handle field from struct acpiphp_bridge
authorRafael J. Wysocki <rafael.j.wysocki@intel.com>
Sat, 13 Jul 2013 21:27:25 +0000 (23:27 +0200)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Tue, 23 Jul 2013 02:00:24 +0000 (04:00 +0200)
The handle field in struct acpiphp_bridge is only used by
acpiphp_enumerate_slots(), but in that function the local handle
variable can be used instead, so make that happen and drop handle
from struct acpiphp_bridge.

Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Tested-by: Mika Westerberg <mika.westerberg@linux.intel.com>
drivers/pci/hotplug/acpiphp.h
drivers/pci/hotplug/acpiphp_glue.c

index dbb94258da57d02e09257acf5f2e826a29f5cd73..afd7110a1ca8e2a7cb8bcff8049ed8a51f3d0f6d 100644 (file)
@@ -77,7 +77,6 @@ struct acpiphp_bridge {
        struct list_head list;
        struct list_head slots;
        struct kref ref;
-       acpi_handle handle;
 
        struct acpiphp_context *context;
 
index 9e4ad6f37647ab7e9ec868a33f758ae8593a117c..bbbf8f4e84827032d28e140613081e036abe5aa3 100644 (file)
@@ -1084,7 +1084,6 @@ void acpiphp_enumerate_slots(struct pci_bus *bus)
 
        INIT_LIST_HEAD(&bridge->slots);
        kref_init(&bridge->ref);
-       bridge->handle = handle;
        bridge->pci_dev = pci_dev_get(bus->self);
        bridge->pci_bus = bus;
 
@@ -1124,10 +1123,10 @@ void acpiphp_enumerate_slots(struct pci_bus *bus)
        mutex_unlock(&bridge_mutex);
 
        /* register all slot objects under this bridge */
-       status = acpi_walk_namespace(ACPI_TYPE_DEVICE, bridge->handle, 1,
+       status = acpi_walk_namespace(ACPI_TYPE_DEVICE, handle, 1,
                                     register_slot, NULL, bridge, NULL);
        if (ACPI_FAILURE(status)) {
-               acpi_handle_err(bridge->handle, "failed to register slots\n");
+               acpi_handle_err(handle, "failed to register slots\n");
                cleanup_bridge(bridge);
                put_bridge(bridge);
        }