thermal/drivers/tsens: Don't hardcode sensor slope
authorAnsuel Smith <ansuelsmth@gmail.com>
Tue, 20 Apr 2021 18:33:35 +0000 (20:33 +0200)
committerDaniel Lezcano <daniel.lezcano@linaro.org>
Thu, 22 Apr 2021 12:08:50 +0000 (14:08 +0200)
Function compute_intercept_slope hardcode the sensor slope to
SLOPE_DEFAULT. Change this and use the default value only if a slope is
not defined. This is needed for tsens VER_0 that has a hardcoded slope
table.

Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com>
Reviewed-by: Thara Gopinath <thara.gopinath@linaro.org>
Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Link: https://lore.kernel.org/r/20210420183343.2272-2-ansuelsmth@gmail.com
drivers/thermal/qcom/tsens.c

index a5da6a7..d6f8021 100644 (file)
@@ -85,7 +85,8 @@ void compute_intercept_slope(struct tsens_priv *priv, u32 *p1,
                        "%s: sensor%d - data_point1:%#x data_point2:%#x\n",
                        __func__, i, p1[i], p2[i]);
 
-               priv->sensor[i].slope = SLOPE_DEFAULT;
+               if (!priv->sensor[i].slope)
+                       priv->sensor[i].slope = SLOPE_DEFAULT;
                if (mode == TWO_PT_CALIB) {
                        /*
                         * slope (m) = adc_code2 - adc_code1 (y2 - y1)/