diff options
author | Subrata Banik <subrata.banik@intel.com> | 2021-11-15 12:10:56 +0530 |
---|---|---|
committer | Subrata Banik <subrata.banik@intel.com> | 2021-11-16 10:24:10 +0000 |
commit | 94030cfa558257e5453402867feb443cd7505a91 (patch) | |
tree | df19bcf93bc7e0eb3d14930314cc346edbbf2492 /src/soc/intel | |
parent | 4ab77addc51e256ed3f6a360ecec06ca7afd5e9c (diff) |
soc/intel/../thermal: Drop `ltt_value` local variable
Using the `GET_LTT_VALUE` macro directly instead of 'ltt_value' local
variable.
BUG=b:193774296
Change-Id: I791766bf2a78fa30dbba8cf4ad8a50e44f0e73ed
Signed-off-by: Subrata Banik <subrata.banik@intel.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/59309
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Maulik V Vaghela <maulik.v.vaghela@intel.com>
Reviewed-by: EricR Lai <ericr_lai@compal.corp-partner.google.com>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Diffstat (limited to 'src/soc/intel')
-rw-r--r-- | src/soc/intel/common/block/thermal/thermal.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/src/soc/intel/common/block/thermal/thermal.c b/src/soc/intel/common/block/thermal/thermal.c index f88699531e..6211f35bd1 100644 --- a/src/soc/intel/common/block/thermal/thermal.c +++ b/src/soc/intel/common/block/thermal/thermal.c @@ -11,6 +11,7 @@ #define MAX_TRIP_TEMP 205 /* This is the safest default Trip Temp value */ #define DEFAULT_TRIP_TEMP 50 +/* Trip Point Temp = (LTT / 2 - 50 degree C) */ #define GET_LTT_VALUE(x) (((x) + 50) * (2)) static uint8_t get_thermal_trip_temp(void) @@ -24,7 +25,6 @@ static uint8_t get_thermal_trip_temp(void) /* PCH Low Temp Threshold (LTT) */ static uint16_t pch_get_ltt_value(struct device *dev) { - uint16_t ltt_value; uint8_t thermal_config; thermal_config = get_thermal_trip_temp(); @@ -34,10 +34,7 @@ static uint16_t pch_get_ltt_value(struct device *dev) if (thermal_config > MAX_TRIP_TEMP) die("Input PCH temp trip is higher than allowed range!"); - /* Trip Point Temp = (LTT / 2 - 50 degree C) */ - ltt_value = GET_LTT_VALUE(thermal_config); - - return ltt_value; + return GET_LTT_VALUE(thermal_config); } /* Enable thermal sensor power management */ |