[PATCH v1 13/13] ACPI: thermal: Rename structure fields holding temperature in deci-Kelvin

From: Rafael J. Wysocki
Date: Thu Sep 21 2023 - 14:36:50 EST


From: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
Subject: [PATCH v1]

Rename structure fields holding temperature values in deci-Kelvin so as
to avoid temperature units confusion.

No intentional functional impact.

Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
---
drivers/acpi/thermal.c | 44 ++++++++++++++++++++++----------------------
1 file changed, 22 insertions(+), 22 deletions(-)

Index: linux-pm/drivers/acpi/thermal.c
===================================================================
--- linux-pm.orig/drivers/acpi/thermal.c
+++ linux-pm/drivers/acpi/thermal.c
@@ -82,7 +82,7 @@ MODULE_PARM_DESC(psv, "Disable or overri
static struct workqueue_struct *acpi_thermal_pm_queue;

struct acpi_thermal_trip {
- unsigned long temperature;
+ unsigned long temp_dk;
struct acpi_handle_list devices;
};

@@ -105,8 +105,8 @@ struct acpi_thermal_trips {
struct acpi_thermal {
struct acpi_device *device;
acpi_bus_id name;
- unsigned long temperature;
- unsigned long last_temperature;
+ unsigned long temp_dk;
+ unsigned long last_temp_dk;
unsigned long polling_frequency;
volatile u8 zombie;
struct acpi_thermal_trips trips;
@@ -131,16 +131,16 @@ static int acpi_thermal_get_temperature(
if (!tz)
return -EINVAL;

- tz->last_temperature = tz->temperature;
+ tz->last_temp_dk = tz->temp_dk;

status = acpi_evaluate_integer(tz->device->handle, "_TMP", NULL, &tmp);
if (ACPI_FAILURE(status))
return -ENODEV;

- tz->temperature = tmp;
+ tz->temp_dk = tmp;

acpi_handle_debug(tz->device->handle, "Temperature is %lu dK\n",
- tz->temperature);
+ tz->temp_dk);

return 0;
}
@@ -175,7 +175,7 @@ static int acpi_thermal_temp(struct acpi

static bool acpi_thermal_trip_valid(struct acpi_thermal_trip *acpi_trip)
{
- return acpi_trip->temperature != THERMAL_TEMP_INVALID;
+ return acpi_trip->temp_dk != THERMAL_TEMP_INVALID;
}

static long get_passive_temp(struct acpi_thermal *tz)
@@ -197,7 +197,7 @@ static void acpi_thermal_update_passive_
if (!acpi_thermal_trip_valid(acpi_trip) || psv > 0)
return;

- acpi_trip->temperature = get_passive_temp(tz);
+ acpi_trip->temp_dk = get_passive_temp(tz);
if (!acpi_thermal_trip_valid(acpi_trip))
ACPI_THERMAL_TRIPS_EXCEPTION(tz, "state");
}
@@ -244,7 +244,7 @@ static void acpi_thermal_update_trip_dev
return;
}

- acpi_trip->temperature = THERMAL_TEMP_INVALID;
+ acpi_trip->temp_dk = THERMAL_TEMP_INVALID;
ACPI_THERMAL_TRIPS_EXCEPTION(tz, "state");
}

@@ -278,7 +278,7 @@ static void acpi_thermal_update_active_t
if (!acpi_thermal_trip_valid(acpi_trip))
return;

- acpi_trip->temperature = get_active_temp(tz, index);
+ acpi_trip->temp_dk = get_active_temp(tz, index);
if (!acpi_thermal_trip_valid(acpi_trip))
ACPI_THERMAL_TRIPS_EXCEPTION(tz, "state");
}
@@ -292,7 +292,7 @@ static int acpi_thermal_adjust_trip(stru
return 0;

if (acpi_thermal_trip_valid(acpi_trip))
- trip->temperature = acpi_thermal_temp(tz, acpi_trip->temperature);
+ trip->temperature = acpi_thermal_temp(tz, acpi_trip->temp_dk);
else
trip->temperature = THERMAL_TEMP_INVALID;

@@ -447,11 +447,11 @@ static bool acpi_thermal_init_trip(struc
if (!update_trip_devices(tz, acpi_trip, index, false))
goto fail;

- acpi_trip->temperature = temp;
+ acpi_trip->temp_dk = temp;
return true;

fail:
- acpi_trip->temperature = THERMAL_TEMP_INVALID;
+ acpi_trip->temp_dk = THERMAL_TEMP_INVALID;
return false;
}

@@ -488,7 +488,7 @@ static int thermal_get_temp(struct therm
if (result)
return result;

- *temp = deci_kelvin_to_millicelsius_with_offset(tz->temperature,
+ *temp = deci_kelvin_to_millicelsius_with_offset(tz->temp_dk,
tz->kelvin_offset);
return 0;
}
@@ -510,10 +510,10 @@ static int thermal_get_trend(struct ther

switch (trip->type) {
case THERMAL_TRIP_PASSIVE:
- t = tz->trips.passive.tc1 * (tz->temperature -
- tz->last_temperature) +
- tz->trips.passive.tc2 * (tz->temperature -
- acpi_trip->temperature);
+ t = tz->trips.passive.tc1 * (tz->temp_dk -
+ tz->last_temp_dk) +
+ tz->trips.passive.tc2 * (tz->temp_dk -
+ acpi_trip->temp_dk);
if (t > 0)
*trend = THERMAL_TREND_RAISING;
else if (t < 0)
@@ -524,7 +524,7 @@ static int thermal_get_trend(struct ther
return 0;

case THERMAL_TRIP_ACTIVE:
- t = acpi_thermal_temp(tz, tz->temperature);
+ t = acpi_thermal_temp(tz, tz->temp_dk);
if (t <= trip->temperature)
break;

@@ -894,7 +894,7 @@ static int acpi_thermal_add(struct acpi_
passive_delay = tz->trips.passive.tsp * 100;

trip->type = THERMAL_TRIP_PASSIVE;
- trip->temperature = acpi_thermal_temp(tz, acpi_trip->temperature);
+ trip->temperature = acpi_thermal_temp(tz, acpi_trip->temp_dk);
trip->priv = acpi_trip;
trip++;
}
@@ -906,7 +906,7 @@ static int acpi_thermal_add(struct acpi_
break;

trip->type = THERMAL_TRIP_ACTIVE;
- trip->temperature = acpi_thermal_temp(tz, acpi_trip->temperature);
+ trip->temperature = acpi_thermal_temp(tz, acpi_trip->temp_dk);
trip->priv = acpi_trip;
trip++;
}
@@ -920,7 +920,7 @@ static int acpi_thermal_add(struct acpi_
INIT_WORK(&tz->thermal_check_work, acpi_thermal_check_fn);

pr_info("%s [%s] (%ld C)\n", acpi_device_name(device),
- acpi_device_bid(device), deci_kelvin_to_celsius(tz->temperature));
+ acpi_device_bid(device), deci_kelvin_to_celsius(tz->temp_dk));

result = acpi_dev_install_notify_handler(device, ACPI_DEVICE_NOTIFY,
acpi_thermal_notify);