On Fri, Aug 12, 2022 at 3:54 PM Daniel Lezcano <daniel.lezcano@xxxxxxxxxx> wrote: > > On 12/08/2022 15:34, Marek Szyprowski wrote: > > On 12.08.2022 15:12, Daniel Lezcano wrote: > >> The function thermal_zone_device_is_enabled() must be called with the > >> thermal zone lock held. In the resume path, it is called without. > >> > >> As the thermal_zone_device_is_enabled() is also checked in > >> thermal_zone_device_update(), do the check in resume() function is > >> pointless, except for saving an extra initialization which does not > >> hurt if it is done in all the cases. > >> > >> Signed-off-by: Daniel Lezcano <daniel.lezcano@xxxxxxxxxx> > > > > This fixes the warning I've reported. Feel free to add: > > > > Reported-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx> > > > > Tested-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx> > > Great, thanks for testing Do you want me to apply this for -rc3?
