diff options
author | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2013-07-13 17:27:25 -0400 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2013-07-22 22:00:24 -0400 |
commit | 89373a55d294b53e85792dbc636015b83d492f67 (patch) | |
tree | 79eb8fbe65ddaf50fd4952c18ab4a9840327d4dd | |
parent | 5a3bc573ae32a71bb9e307812d4de1bdcab6b9fb (diff) |
ACPI / hotplug / PCI: Drop handle field from struct acpiphp_bridge
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>
-rw-r--r-- | drivers/pci/hotplug/acpiphp.h | 1 | ||||
-rw-r--r-- | drivers/pci/hotplug/acpiphp_glue.c | 5 |
2 files changed, 2 insertions, 4 deletions
diff --git a/drivers/pci/hotplug/acpiphp.h b/drivers/pci/hotplug/acpiphp.h index dbb94258da57..afd7110a1ca8 100644 --- a/drivers/pci/hotplug/acpiphp.h +++ b/drivers/pci/hotplug/acpiphp.h | |||
@@ -77,7 +77,6 @@ struct acpiphp_bridge { | |||
77 | struct list_head list; | 77 | struct list_head list; |
78 | struct list_head slots; | 78 | struct list_head slots; |
79 | struct kref ref; | 79 | struct kref ref; |
80 | acpi_handle handle; | ||
81 | 80 | ||
82 | struct acpiphp_context *context; | 81 | struct acpiphp_context *context; |
83 | 82 | ||
diff --git a/drivers/pci/hotplug/acpiphp_glue.c b/drivers/pci/hotplug/acpiphp_glue.c index 9e4ad6f37647..bbbf8f4e8482 100644 --- a/drivers/pci/hotplug/acpiphp_glue.c +++ b/drivers/pci/hotplug/acpiphp_glue.c | |||
@@ -1084,7 +1084,6 @@ void acpiphp_enumerate_slots(struct pci_bus *bus) | |||
1084 | 1084 | ||
1085 | INIT_LIST_HEAD(&bridge->slots); | 1085 | INIT_LIST_HEAD(&bridge->slots); |
1086 | kref_init(&bridge->ref); | 1086 | kref_init(&bridge->ref); |
1087 | bridge->handle = handle; | ||
1088 | bridge->pci_dev = pci_dev_get(bus->self); | 1087 | bridge->pci_dev = pci_dev_get(bus->self); |
1089 | bridge->pci_bus = bus; | 1088 | bridge->pci_bus = bus; |
1090 | 1089 | ||
@@ -1124,10 +1123,10 @@ void acpiphp_enumerate_slots(struct pci_bus *bus) | |||
1124 | mutex_unlock(&bridge_mutex); | 1123 | mutex_unlock(&bridge_mutex); |
1125 | 1124 | ||
1126 | /* register all slot objects under this bridge */ | 1125 | /* register all slot objects under this bridge */ |
1127 | status = acpi_walk_namespace(ACPI_TYPE_DEVICE, bridge->handle, 1, | 1126 | status = acpi_walk_namespace(ACPI_TYPE_DEVICE, handle, 1, |
1128 | register_slot, NULL, bridge, NULL); | 1127 | register_slot, NULL, bridge, NULL); |
1129 | if (ACPI_FAILURE(status)) { | 1128 | if (ACPI_FAILURE(status)) { |
1130 | acpi_handle_err(bridge->handle, "failed to register slots\n"); | 1129 | acpi_handle_err(handle, "failed to register slots\n"); |
1131 | cleanup_bridge(bridge); | 1130 | cleanup_bridge(bridge); |
1132 | put_bridge(bridge); | 1131 | put_bridge(bridge); |
1133 | } | 1132 | } |