| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101 |
- From 7805365fee582056b32c69cf35aafbb94b14a8ca Mon Sep 17 00:00:00 2001
- From: Robert Marko <[email protected]>
- Date: Fri, 19 Aug 2022 00:02:43 +0200
- Subject: [PATCH] thermal/drivers/tsens: Allow configuring min and max trips
- IPQ8074 and IPQ6018 dont support negative trip temperatures and support
- up to 204 degrees C as the max trip temperature.
- So, instead of always setting the -40 as min and 120 degrees C as max
- allow it to be configured as part of the features.
- Signed-off-by: Robert Marko <[email protected]>
- Reviewed-by: Bjorn Andersson <[email protected]>
- Link: https://lore.kernel.org/r/[email protected]
- Signed-off-by: Daniel Lezcano <[email protected]>
- ---
- drivers/thermal/qcom/tsens-8960.c | 2 ++
- drivers/thermal/qcom/tsens-v0_1.c | 2 ++
- drivers/thermal/qcom/tsens-v1.c | 2 ++
- drivers/thermal/qcom/tsens-v2.c | 2 ++
- drivers/thermal/qcom/tsens.c | 4 ++--
- drivers/thermal/qcom/tsens.h | 4 ++++
- 6 files changed, 14 insertions(+), 2 deletions(-)
- --- a/drivers/thermal/qcom/tsens-8960.c
- +++ b/drivers/thermal/qcom/tsens-8960.c
- @@ -273,6 +273,8 @@ static struct tsens_features tsens_8960_
- .adc = 1,
- .srot_split = 0,
- .max_sensors = 11,
- + .trip_min_temp = -40000,
- + .trip_max_temp = 120000,
- };
-
- struct tsens_plat_data data_8960 = {
- --- a/drivers/thermal/qcom/tsens-v0_1.c
- +++ b/drivers/thermal/qcom/tsens-v0_1.c
- @@ -553,6 +553,8 @@ static struct tsens_features tsens_v0_1_
- .adc = 1,
- .srot_split = 1,
- .max_sensors = 11,
- + .trip_min_temp = -40000,
- + .trip_max_temp = 120000,
- };
-
- static const struct reg_field tsens_v0_1_regfields[MAX_REGFIELDS] = {
- --- a/drivers/thermal/qcom/tsens-v1.c
- +++ b/drivers/thermal/qcom/tsens-v1.c
- @@ -277,6 +277,8 @@ static struct tsens_features tsens_v1_fe
- .adc = 1,
- .srot_split = 1,
- .max_sensors = 11,
- + .trip_min_temp = -40000,
- + .trip_max_temp = 120000,
- };
-
- static const struct reg_field tsens_v1_regfields[MAX_REGFIELDS] = {
- --- a/drivers/thermal/qcom/tsens-v2.c
- +++ b/drivers/thermal/qcom/tsens-v2.c
- @@ -35,6 +35,8 @@ static struct tsens_features tsens_v2_fe
- .adc = 0,
- .srot_split = 1,
- .max_sensors = 16,
- + .trip_min_temp = -40000,
- + .trip_max_temp = 120000,
- };
-
- static const struct reg_field tsens_v2_regfields[MAX_REGFIELDS] = {
- --- a/drivers/thermal/qcom/tsens.c
- +++ b/drivers/thermal/qcom/tsens.c
- @@ -573,8 +573,8 @@ static int tsens_set_trips(struct therma
- dev_dbg(dev, "[%u] %s: proposed thresholds: (%d:%d)\n",
- hw_id, __func__, low, high);
-
- - cl_high = clamp_val(high, -40000, 120000);
- - cl_low = clamp_val(low, -40000, 120000);
- + cl_high = clamp_val(high, priv->feat->trip_min_temp, priv->feat->trip_max_temp);
- + cl_low = clamp_val(low, priv->feat->trip_min_temp, priv->feat->trip_max_temp);
-
- high_val = tsens_mC_to_hw(s, cl_high);
- low_val = tsens_mC_to_hw(s, cl_low);
- --- a/drivers/thermal/qcom/tsens.h
- +++ b/drivers/thermal/qcom/tsens.h
- @@ -499,6 +499,8 @@ enum regfield_ids {
- * with SROT only being available to secure boot firmware?
- * @has_watchdog: does this IP support watchdog functionality?
- * @max_sensors: maximum sensors supported by this version of the IP
- + * @trip_min_temp: minimum trip temperature supported by this version of the IP
- + * @trip_max_temp: maximum trip temperature supported by this version of the IP
- */
- struct tsens_features {
- unsigned int ver_major;
- @@ -508,6 +510,8 @@ struct tsens_features {
- unsigned int srot_split:1;
- unsigned int has_watchdog:1;
- unsigned int max_sensors;
- + int trip_min_temp;
- + int trip_max_temp;
- };
-
- /**
|
