Re: [PATCH 1/5] thermal/core: Rearm the monitoring only one time

From: Daniel Lezcano
Date: Fri Aug 05 2022 - 12:37:29 EST



Hi Rafael,

On 05/08/2022 18:29, Rafael J. Wysocki wrote:
On Fri, Aug 5, 2022 at 5:38 PM Daniel Lezcano <daniel.lezcano@xxxxxxxxxx> wrote:

The current code calls monitor_thermal_zone() inside the
handle_thermal_trip() function. But this one is called in a loop for
each trip point which means the monitoring is rearmed several times
for nothing (assuming there could be several passive and active trip
points).

Move the monitor_thermal_zone() function out of the
handle_thermal_trip() function and after the thermal trip loop, so the
timer will be disabled or rearmed one time.

Signed-off-by: Daniel Lezcano <daniel.lezcano@xxxxxxxxxx>

Does this series depend on any other? >
You've been sending quite a lot of material lately and it is not
always easy to tell what the dependencies between the different patch
series are.

Yes I understand.

This series does not depend on any other. It is not related to any pending changes.

It may have trivial conflicts with the other 26 patches series but it would be the case for any submissions posted by someone else anyway.




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