On 20.12.2022 03:47, Dmitry Baryshkov wrote:
> Drop msm8976-specific defines, which duplicate generic ones.
>
> Fixes: 0e580290170d ("thermal: qcom: tsens-v1: Add support for MSM8956 and MSM8976")
> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@xxxxxxxxxxxxx>
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>
> ---
Reviewed-by: Konrad Dybcio <konrad.dybcio@xxxxxxxxxx>
Konrad
> drivers/thermal/qcom/tsens-v1.c | 9 ++-------
> 1 file changed, 2 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/thermal/qcom/tsens-v1.c b/drivers/thermal/qcom/tsens-v1.c
> index 96ef12d47bff..a7f53966156b 100644
> --- a/drivers/thermal/qcom/tsens-v1.c
> +++ b/drivers/thermal/qcom/tsens-v1.c
> @@ -78,11 +78,6 @@
>
> #define MSM8976_CAL_SEL_MASK 0x3
>
> -#define MSM8976_CAL_DEGC_PT1 30
> -#define MSM8976_CAL_DEGC_PT2 120
> -#define MSM8976_SLOPE_FACTOR 1000
> -#define MSM8976_SLOPE_DEFAULT 3200
> -
> /* eeprom layout data for qcs404/405 (v1) */
> #define BASE0_MASK 0x000007f8
> #define BASE1_MASK 0x0007f800
> @@ -160,8 +155,8 @@ static void compute_intercept_slope_8976(struct tsens_priv *priv,
> priv->sensor[10].slope = 3286;
>
> for (i = 0; i < priv->num_sensors; i++) {
> - priv->sensor[i].offset = (p1[i] * MSM8976_SLOPE_FACTOR) -
> - (MSM8976_CAL_DEGC_PT1 *
> + priv->sensor[i].offset = (p1[i] * SLOPE_FACTOR) -
> + (CAL_DEGC_PT1 *
> priv->sensor[i].slope);
> }
> }
