On Thu, 19 Oct 2023 at 17:59, Stephan Gerhold <stephan@xxxxxxxxxxx> 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@xxxxxxxxxx>
>
> 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@xxxxxxxxxx/
>
> 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.
>
--
With best wishes
Dmitry
