Re: [PATCH v3 29/30] thermal/intel/int340x: Replace parameter to simplify

From: Daniel Lezcano
Date: Sat Sep 24 2022 - 13:45:09 EST


On 24/09/2022 19:36, Rafael J. Wysocki wrote:
On Tue, Sep 6, 2022 at 6:48 PM Daniel Lezcano <daniel.lezcano@xxxxxxxxxx> wrote:

In the process of replacing the get_trip_* ops by the generic trip
points, the current code has an 'override' property to add another
indirection to a different ops.

Rework this approach to prevent this indirection and make the code
ready for the generic trip points conversion.

Actually the get_temp() is different regarding the platform, so it is
pointless to add a new set of ops but just create dynamically the ops
at init time.

Signed-off-by: Daniel Lezcano <daniel.lezcano@xxxxxxxxxx>
Reviewed-by: Srinivas Pandruvada <srinivas.pandruvada@xxxxxxxxxxxxxxx>

A fix has been posted for this one already:

https://patchwork.kernel.org/project/linux-pm/patch/20220923152009.1721739-1-nathan@xxxxxxxxxx/

Thanks I'll pick it up in the series.

I'm currently fixing a locking issue in the thermal linux-next branch


--
<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