From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753782AbeDPMz7 (ORCPT ); Mon, 16 Apr 2018 08:55:59 -0400 Received: from mail-wr0-f196.google.com ([209.85.128.196]:46401 "EHLO mail-wr0-f196.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754073AbeDPMz5 (ORCPT ); Mon, 16 Apr 2018 08:55:57 -0400 X-Google-Smtp-Source: AIpwx490Q/RAAWYtM3KWWo1eQ9324BLmMMReBNAS5lmn+Zdz0cNxF/fBu/AcgS1Uj2WIy3uB3vi56A== Subject: Re: [PATCH 05/14] thermal: exynos: remove parsing of samsung,tmu_default_temp_offset property To: Bartlomiej Zolnierkiewicz , Eduardo Valentin Cc: Zhang Rui , Krzysztof Kozlowski , Kukjin Kim , linux-samsung-soc@vger.kernel.org, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org References: <1523873525-23718-1-git-send-email-b.zolnierkie@samsung.com> <1523873525-23718-6-git-send-email-b.zolnierkie@samsung.com> From: Daniel Lezcano Message-ID: <86b33c49-d6a3-0db7-5c95-78619f0f29fe@linaro.org> Date: Mon, 16 Apr 2018 14:55:54 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.7.0 MIME-Version: 1.0 In-Reply-To: <1523873525-23718-6-git-send-email-b.zolnierkie@samsung.com> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 16/04/2018 12:11, Bartlomiej Zolnierkiewicz wrote: > Trimming (one point based or two points based) is always used for > the temperature calibration and the default non-trimming code is > never reached. Remove it and then remove no longer needed parsing > of samsung,tmu_default_temp_offset property. > > There should be no functional changes caused by this patch. > > Signed-off-by: Bartlomiej Zolnierkiewicz > --- Reviewed-by: Daniel Lezcano This one was easy :) > drivers/thermal/samsung/exynos_tmu.c | 48 ++++++++++-------------------------- > drivers/thermal/samsung/exynos_tmu.h | 2 -- > 2 files changed, 13 insertions(+), 37 deletions(-) > > diff --git a/drivers/thermal/samsung/exynos_tmu.c b/drivers/thermal/samsung/exynos_tmu.c > index d7a3c3c..958a7c4 100644 > --- a/drivers/thermal/samsung/exynos_tmu.c > +++ b/drivers/thermal/samsung/exynos_tmu.c > @@ -249,24 +249,14 @@ static void exynos_report_trigger(struct exynos_tmu_data *p) > static int temp_to_code(struct exynos_tmu_data *data, u8 temp) > { > struct exynos_tmu_platform_data *pdata = data->pdata; > - int temp_code; > - > - switch (pdata->cal_type) { > - case TYPE_TWO_POINT_TRIMMING: > - temp_code = (temp - pdata->first_point_trim) * > - (data->temp_error2 - data->temp_error1) / > - (pdata->second_point_trim - pdata->first_point_trim) + > - data->temp_error1; > - break; > - case TYPE_ONE_POINT_TRIMMING: > - temp_code = temp + data->temp_error1 - pdata->first_point_trim; > - break; > - default: > - temp_code = temp + pdata->default_temp_offset; > - break; > - } > > - return temp_code; > + if (pdata->cal_type == TYPE_ONE_POINT_TRIMMING) > + return temp + data->temp_error1 - pdata->first_point_trim; > + > + return (temp - pdata->first_point_trim) * > + (data->temp_error2 - data->temp_error1) / > + (pdata->second_point_trim - pdata->first_point_trim) + > + data->temp_error1; > } > > /* > @@ -276,24 +266,14 @@ static int temp_to_code(struct exynos_tmu_data *data, u8 temp) > static int code_to_temp(struct exynos_tmu_data *data, u16 temp_code) > { > struct exynos_tmu_platform_data *pdata = data->pdata; > - int temp; > > - switch (pdata->cal_type) { > - case TYPE_TWO_POINT_TRIMMING: > - temp = (temp_code - data->temp_error1) * > - (pdata->second_point_trim - pdata->first_point_trim) / > - (data->temp_error2 - data->temp_error1) + > - pdata->first_point_trim; > - break; > - case TYPE_ONE_POINT_TRIMMING: > - temp = temp_code - data->temp_error1 + pdata->first_point_trim; > - break; > - default: > - temp = temp_code - pdata->default_temp_offset; > - break; > - } > + if (pdata->cal_type == TYPE_ONE_POINT_TRIMMING) > + return temp_code - data->temp_error1 + pdata->first_point_trim; > > - return temp; > + return (temp_code - data->temp_error1) * > + (pdata->second_point_trim - pdata->first_point_trim) / > + (data->temp_error2 - data->temp_error1) + > + pdata->first_point_trim; > } > > static void sanitize_temp_error(struct exynos_tmu_data *data, u32 trim_info) > @@ -1166,8 +1146,6 @@ static int exynos_of_sensor_conf(struct device_node *np, > pdata->first_point_trim = (u8)value; > of_property_read_u32(np, "samsung,tmu_second_point_trim", &value); > pdata->second_point_trim = (u8)value; > - of_property_read_u32(np, "samsung,tmu_default_temp_offset", &value); > - pdata->default_temp_offset = (u8)value; > > of_property_read_u32(np, "samsung,tmu_cal_type", &pdata->cal_type); > > diff --git a/drivers/thermal/samsung/exynos_tmu.h b/drivers/thermal/samsung/exynos_tmu.h > index 8c468b6..a7e81b4 100644 > --- a/drivers/thermal/samsung/exynos_tmu.h > +++ b/drivers/thermal/samsung/exynos_tmu.h > @@ -50,7 +50,6 @@ enum soc_type { > * @efuse_value: platform defined fuse value > * @min_efuse_value: minimum valid trimming data > * @max_efuse_value: maximum valid trimming data > - * @default_temp_offset: default temperature offset in case of no trimming > * @cal_type: calibration type for temperature > * > * This structure is required for configuration of exynos_tmu driver. > @@ -65,7 +64,6 @@ struct exynos_tmu_platform_data { > u32 max_efuse_value; > u8 first_point_trim; > u8 second_point_trim; > - u8 default_temp_offset; > > u32 cal_type; > }; > -- Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog