Re: [PATCH 2/2] thermal/drivers/mellanox: Use generic thermal_zone_get_trip() function

[Daniel Lezcano]

Hi Ido,

On 18/10/2022 08:28, Ido Schimmel wrote:
> + Vadim
>
> On Fri, Oct 14, 2022 at 09:32:51AM +0200, Daniel Lezcano wrote:
> > The thermal framework gives the possibility to register the trip
> > points with the thermal zone. When that is done, no get_trip_* ops are
> > needed and they can be removed.
> >
> > Convert ops content logic into generic trip points and register them with the
> > thermal zone.
> >
> > Signed-off-by: Daniel Lezcano <daniel.lezcano@xxxxxxxxxx>
>
> Vadim, can you please review and test?
>
> Daniel, I saw that you wrote to Kalle that you want to take it via the
> thermal tree. Any reason not to take it via net-next? I'm asking because
> it will be the second release in a row where we need to try to avoid
> conflicts in this file.

Because I hope I can remove the ops->get_trip_ ops from thermal_ops 
structure before the end of this cycle.

May be you can consider moving the thermal driver into drivers/thermal?


--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog