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

From: Guenter Roeck
Date: Wed Jun 29 2022 - 17:18:22 EST


On 6/29/22 13:02, Dmitry Osipenko wrote:
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>

Thanks a lot for testing!

Guenter