2023-08-10 19:16:30

by Rafael J. Wysocki

[permalink] [raw]
Subject: [PATCH v1] ACPI: thermal: Fold acpi_thermal_get_info() into its caller

From: Rafael J. Wysocki <[email protected]>

Commit dabc621a3110 ("ACPI: thermal: Drop enabled flag from struct
acpi_thermal_active") left behind a variable that is only assigned to
and never read, so drop it now.

Fixes: dabc621a3110 ("ACPI: thermal: Drop enabled flag from struct acpi_thermal_active")
Signed-off-by: Rafael J. Wysocki <[email protected]>
---

The commit mentioned above is a linux-next one.

---
drivers/acpi/thermal.c | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)

Index: linux-pm/drivers/acpi/thermal.c
===================================================================
--- linux-pm.orig/drivers/acpi/thermal.c
+++ linux-pm/drivers/acpi/thermal.c
@@ -1039,7 +1039,7 @@ static int acpi_thermal_suspend(struct d
static int acpi_thermal_resume(struct device *dev)
{
struct acpi_thermal *tz;
- int i, j, power_state, result;
+ int i, j, power_state;

if (!dev)
return -EINVAL;
@@ -1053,9 +1053,8 @@ static int acpi_thermal_resume(struct de
break;

for (j = 0; j < tz->trips.active[i].devices.count; j++) {
- result = acpi_bus_update_power(
- tz->trips.active[i].devices.handles[j],
- &power_state);
+ acpi_bus_update_power(tz->trips.active[i].devices.handles[j],
+ &power_state);
}
}