lkml.org 
[lkml]   [2023]   [Jan]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH v1 3/6] thermal: intel: int340x: Evaluate GTSH once
Date
From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>

It is not necessary to evaluate the GTSH ACPI object for every trip
point in int340x_thermal_zone_add() and the function used for that
is not really necessary either, because it only has one caller.

Accordingly, fold the evaluation of GTSH directly into
int340x_thermal_zone_add(), evaluate it only once and populate the
hysteresis for all of the trip points with the value produced by it.

No expected functional impact.

Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
---
drivers/thermal/intel/int340x_thermal/int340x_thermal_zone.c | 24 +++--------
1 file changed, 8 insertions(+), 16 deletions(-)

Index: linux-pm/drivers/thermal/intel/int340x_thermal/int340x_thermal_zone.c
===================================================================
--- linux-pm.orig/drivers/thermal/intel/int340x_thermal/int340x_thermal_zone.c
+++ linux-pm/drivers/thermal/intel/int340x_thermal/int340x_thermal_zone.c
@@ -53,21 +53,6 @@ static int int340x_thermal_set_trip_temp
return 0;
}

-static int int340x_thermal_get_global_hyst(struct acpi_device *adev, int *temp)
-{
- acpi_status status;
- unsigned long long hyst;
-
- status = acpi_evaluate_integer(adev->handle, "GTSH", NULL, &hyst);
- if (ACPI_FAILURE(status))
- *temp = 0;
- else
- *temp = hyst * 100;
-
- return 0;
-}
-
-
static void int340x_thermal_critical(struct thermal_zone_device *zone)
{
dev_dbg(&zone->device, "%s: critical temperature reached\n", zone->type);
@@ -120,6 +105,7 @@ struct int34x_thermal_zone *int340x_ther
{
struct int34x_thermal_zone *int34x_zone;
unsigned long long trip_cnt = 0;
+ unsigned long long hyst;
int trip_mask = 0;
acpi_status status;
int i, ret;
@@ -156,8 +142,14 @@ struct int34x_thermal_zone *int340x_ther

trip_cnt = int340x_thermal_read_trips(int34x_zone);

+ status = acpi_evaluate_integer(adev->handle, "GTSH", NULL, &hyst);
+ if (ACPI_SUCCESS(status))
+ hyst *= 100;
+ else
+ hyst = 0;
+
for (i = 0; i < trip_cnt; ++i)
- int340x_thermal_get_global_hyst(adev, &int34x_zone->trips[i].hysteresis);
+ int34x_zone->trips[i].hysteresis = hyst;

for (i = 0; i < int34x_zone->aux_trip_nr; i++) {
int34x_zone->trips[i].type = THERMAL_TRIP_PASSIVE;


\
 
 \ /
  Last update: 2023-03-26 23:54    [W:0.050 / U:0.712 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site