diff mbox series

thermal/qcom/tsens: drop ops_v0_1

Message ID 20231019144311.1035181-1-dmitry.baryshkov@linaro.org (mailing list archive)
State Not Applicable
Headers show
Series thermal/qcom/tsens: drop ops_v0_1 | expand

Commit Message

Dmitry Baryshkov Oct. 19, 2023, 2:43 p.m. UTC
Since the commit 6812d1dfbca9 ("thermal/drivers/qcom/tsens-v0_1: Fix
mdm9607 slope values") the default v0.1 implementation of tsens options
is unused by the driver. Drop it now to stop compiler complaining about
the unused static const. If the need for the default v0.1 ops struct
arives, this commit can be easily reverted without further
considerations.

Fixes: 6812d1dfbca9 ("thermal/drivers/qcom/tsens-v0_1: Fix mdm9607 slope values")
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
---
 drivers/thermal/qcom/tsens-v0_1.c | 6 ------
 1 file changed, 6 deletions(-)

Comments

Stephan Gerhold Oct. 19, 2023, 2:59 p.m. UTC | #1
On Thu, Oct 19, 2023 at 05:43:11PM +0300, Dmitry Baryshkov wrote:
> Since the commit 6812d1dfbca9 ("thermal/drivers/qcom/tsens-v0_1: Fix
> mdm9607 slope values") the default v0.1 implementation of tsens options
> is unused by the driver. Drop it now to stop compiler complaining about
> the unused static const. If the need for the default v0.1 ops struct
> arives, this commit can be easily reverted without further
> considerations.
> 
> Fixes: 6812d1dfbca9 ("thermal/drivers/qcom/tsens-v0_1: Fix mdm9607 slope values")
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>

Seems like I remember correctly that there was a patch for this already
that wasn't picked up yet(?):
https://lore.kernel.org/linux-arm-msm/20230617113837.3224912-1-trix@redhat.com/

It doesn't have the Fixes tag though. I think both patches are fine.
Thanks for fixing this. :-)

Stephan

> ---
>  drivers/thermal/qcom/tsens-v0_1.c | 6 ------
>  1 file changed, 6 deletions(-)
> 
> diff --git a/drivers/thermal/qcom/tsens-v0_1.c b/drivers/thermal/qcom/tsens-v0_1.c
> index 87c09f62ee81..32d2d3e33287 100644
> --- a/drivers/thermal/qcom/tsens-v0_1.c
> +++ b/drivers/thermal/qcom/tsens-v0_1.c
> @@ -325,12 +325,6 @@ static const struct reg_field tsens_v0_1_regfields[MAX_REGFIELDS] = {
>  	[TRDY] = REG_FIELD(TM_TRDY_OFF, 0, 0),
>  };
>  
> -static const struct tsens_ops ops_v0_1 = {
> -	.init		= init_common,
> -	.calibrate	= tsens_calibrate_common,
> -	.get_temp	= get_temp_common,
> -};
> -
>  static const struct tsens_ops ops_8226 = {
>  	.init		= init_8226,
>  	.calibrate	= tsens_calibrate_common,
> -- 
> 2.39.2
>
Dmitry Baryshkov Oct. 19, 2023, 3:24 p.m. UTC | #2
On Thu, 19 Oct 2023 at 17:59, Stephan Gerhold <stephan@gerhold.net> wrote:
>
> On Thu, Oct 19, 2023 at 05:43:11PM +0300, Dmitry Baryshkov wrote:
> > Since the commit 6812d1dfbca9 ("thermal/drivers/qcom/tsens-v0_1: Fix
> > mdm9607 slope values") the default v0.1 implementation of tsens options
> > is unused by the driver. Drop it now to stop compiler complaining about
> > the unused static const. If the need for the default v0.1 ops struct
> > arives, this commit can be easily reverted without further
> > considerations.
> >
> > Fixes: 6812d1dfbca9 ("thermal/drivers/qcom/tsens-v0_1: Fix mdm9607 slope values")
> > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
>
> Seems like I remember correctly that there was a patch for this already
> that wasn't picked up yet(?):
> https://lore.kernel.org/linux-arm-msm/20230617113837.3224912-1-trix@redhat.com/
>
> It doesn't have the Fixes tag though. I think both patches are fine.
> Thanks for fixing this. :-)

I'm perfectly fine with either of the patches being picked up.

>
Daniel Lezcano Oct. 19, 2023, 4:11 p.m. UTC | #3
On 19/10/2023 16:43, Dmitry Baryshkov wrote:
> Since the commit 6812d1dfbca9 ("thermal/drivers/qcom/tsens-v0_1: Fix
> mdm9607 slope values") the default v0.1 implementation of tsens options
> is unused by the driver. Drop it now to stop compiler complaining about
> the unused static const. If the need for the default v0.1 ops struct
> arives, this commit can be easily reverted without further
> considerations.
> 
> Fixes: 6812d1dfbca9 ("thermal/drivers/qcom/tsens-v0_1: Fix mdm9607 slope values")
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> ---

Applied, thanks
diff mbox series

Patch

diff --git a/drivers/thermal/qcom/tsens-v0_1.c b/drivers/thermal/qcom/tsens-v0_1.c
index 87c09f62ee81..32d2d3e33287 100644
--- a/drivers/thermal/qcom/tsens-v0_1.c
+++ b/drivers/thermal/qcom/tsens-v0_1.c
@@ -325,12 +325,6 @@  static const struct reg_field tsens_v0_1_regfields[MAX_REGFIELDS] = {
 	[TRDY] = REG_FIELD(TM_TRDY_OFF, 0, 0),
 };
 
-static const struct tsens_ops ops_v0_1 = {
-	.init		= init_common,
-	.calibrate	= tsens_calibrate_common,
-	.get_temp	= get_temp_common,
-};
-
 static const struct tsens_ops ops_8226 = {
 	.init		= init_8226,
 	.calibrate	= tsens_calibrate_common,