Re: [PATCH] hwmon: (lm90) Use worker for alarm notifications

From: Dmitry Osipenko
Date: Wed Jun 29 2022 - 16:02:34 EST


On 6/29/22 18:42, Guenter Roeck wrote:
> Reporting alarms using hwmon_notify_event() may result in a callback
> from the thermal subsystem. This means that such notifications must
> not hold the update lock to avoid a deadlock. To avoid this situation,
> use a worker to handle notifications.
>
> Reported-by: Dmitry Osipenko <dmitry.osipenko@xxxxxxxxxxxxx>
> Fixes: e182cf579f0f ("hwmon: (lm90) Rework alarm/status handling")
> Signed-off-by: Guenter Roeck <linux@xxxxxxxxxxxx>
> ---
> Dmitry,
>
> can you give this patch a try in your system ?

It works, thanks!

Tested-by: Dmitry Osipenko <dmitry.osipenko@xxxxxxxxxxxxx>

--
Best regards,
Dmitry