2023-06-04 12:22:12

by Rafael J. Wysocki

Subject: [PATCH v2 6/7] ACPI: thermal: Drop struct acpi_thermal_state

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

Drop struct acpi_thermal_state which is not really used.

No functional impact.

Signed-off-by: Rafael J. Wysocki <[email protected]>
---

v1 -> v2: New patch

---
drivers/acpi/thermal.c | 11 -----------
1 file changed, 11 deletions(-)

Index: linux-pm/drivers/acpi/thermal.c
===================================================================
--- linux-pm.orig/drivers/acpi/thermal.c
+++ linux-pm/drivers/acpi/thermal.c
@@ -96,15 +96,6 @@ MODULE_PARM_DESC(psv, "Disable or overri

static struct workqueue_struct *acpi_thermal_pm_queue;

-struct acpi_thermal_state {
- u8 critical:1;
- u8 hot:1;
- u8 passive:1;
- u8 active:1;
- u8 reserved:4;
- int active_index;
-};
-
struct acpi_thermal_critical {
unsigned long temperature;
bool valid;
@@ -152,7 +143,6 @@ struct acpi_thermal {
unsigned long polling_frequency;
volatile u8 zombie;
struct acpi_thermal_flags flags;
- struct acpi_thermal_state state;
struct acpi_thermal_trips trips;
struct acpi_handle_list devices;
struct thermal_zone_device *thermal_zone;
@@ -1083,7 +1073,6 @@ static int acpi_thermal_resume(struct de
break;
}
}
- tz->state.active |= tz->trips.active[i].enabled;
}

acpi_queue_thermal_check(tz);

2023-06-06 17:05:02

by Daniel Lezcano

Subject: Re: [PATCH v2 6/7] ACPI: thermal: Drop struct acpi_thermal_state

On 04/06/2023 14:19, Rafael J. Wysocki wrote:
> rom: Rafael J. Wysocki <[email protected]>
>
> Drop struct acpi_thermal_state which is not really used.
>
> No functional impact.
>
> Signed-off-by: Rafael J. Wysocki <[email protected]>

Reviewed-by: Daniel Lezcano <[email protected]>

--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog