On Wed, Feb 08, 2023 at 08:03:30PM +0100, Niklas Söderlund wrote: > Hello, > > This small series fixes a window where incorrect values can be read from > the driver before it is fully initialized. The root cause is that the > thermal zone is register too early. > > Patch 1/3 is new in v2 and removes a unneeded call to set_trips() when > resuming from suspend, This call was in v1 changed as part of addressing > the initialization issue, it's nicer to get rid of it before that is > needed. > > Patch 2/3 prepares for the change while also fixing a theoretical issue > where one thermal node described in DT would describe interrupts and > another would not. Resulting in interrupt support being disabled for > both of them. I'm not aware of any case where this configuration would > be used, either the SoC supports interrupts, or it don't. > > While patch 3/3 fixes the real issue by fully initializing the device > before registering the zone. > > Niklas Söderlund (3): > drivers/thermal/rcar_gen3_thermal: Do not call set_trips() when > resuming > drivers/thermal/rcar_gen3_thermal: Create device local ops struct > drivers/thermal/rcar_gen3_thermal: Fix device initialization > > drivers/thermal/rcar_gen3_thermal.c | 36 ++++++++++++++--------------- > 1 file changed, 18 insertions(+), 18 deletions(-) Applied, thhanks -- <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
