[v1,3/6] thermal: intel: int340x: Evaluate GTSH once

Message ID 2887081.e9J7NaK4W3@kreacher
State New
Headers
Series thermal: intel: int340x: Assorted cleanup changes |

Commit Message

Rafael J. Wysocki Jan. 24, 2023, 7:16 p.m. UTC
  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(-)
  

Patch

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;