Skip to content

Commit 2bc4eb9

Browse files
committed
ACPI: power: Drop name from struct acpi_power_resource
Drop the name field (that only is used in diagnostic messages) from struct acpi_power_resource and use the name of the power resource device object instead of it. Signed-off-by: Rafael J. Wysocki <[email protected]>
1 parent fad40a6 commit 2bc4eb9

File tree

1 file changed

+9
-5
lines changed

1 file changed

+9
-5
lines changed

drivers/acpi/power.c

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,6 @@ struct acpi_power_dependent_device {
4848
struct acpi_power_resource {
4949
struct acpi_device device;
5050
struct list_head list_node;
51-
char *name;
5251
u32 system_level;
5352
u32 order;
5453
unsigned int ref_count;
@@ -70,6 +69,11 @@ static DEFINE_MUTEX(power_resource_list_lock);
7069
Power Resource Management
7170
-------------------------------------------------------------------------- */
7271

72+
static inline const char *resource_dev_name(struct acpi_power_resource *pr)
73+
{
74+
return dev_name(&pr->device.dev);
75+
}
76+
7377
static inline
7478
struct acpi_power_resource *to_power_resource(struct acpi_device *device)
7579
{
@@ -264,7 +268,8 @@ acpi_power_resource_add_dependent(struct acpi_power_resource *resource,
264268

265269
dep->dev = dev;
266270
list_add_tail(&dep->node, &resource->dependents);
267-
dev_dbg(dev, "added power dependency to [%s]\n", resource->name);
271+
dev_dbg(dev, "added power dependency to [%s]\n",
272+
resource_dev_name(resource));
268273

269274
unlock:
270275
mutex_unlock(&resource->resource_lock);
@@ -283,7 +288,7 @@ acpi_power_resource_remove_dependent(struct acpi_power_resource *resource,
283288
list_del(&dep->node);
284289
kfree(dep);
285290
dev_dbg(dev, "removed power dependency to [%s]\n",
286-
resource->name);
291+
resource_dev_name(resource));
287292
break;
288293
}
289294
}
@@ -381,7 +386,7 @@ static int __acpi_power_on(struct acpi_power_resource *resource)
381386

382387
list_for_each_entry(dep, &resource->dependents, node) {
383388
dev_dbg(dep->dev, "runtime resuming because [%s] turned on\n",
384-
resource->name);
389+
resource_dev_name(resource));
385390
pm_request_resume(dep->dev);
386391
}
387392

@@ -953,7 +958,6 @@ struct acpi_device *acpi_add_power_resource(acpi_handle handle)
953958
mutex_init(&resource->resource_lock);
954959
INIT_LIST_HEAD(&resource->list_node);
955960
INIT_LIST_HEAD(&resource->dependents);
956-
resource->name = device->pnp.bus_id;
957961
strcpy(acpi_device_name(device), ACPI_POWER_DEVICE_NAME);
958962
strcpy(acpi_device_class(device), ACPI_POWER_CLASS);
959963
device->power.state = ACPI_STATE_UNKNOWN;

0 commit comments

Comments
 (0)