diff options
author | Shawn Guo <shawn.guo@linaro.org> | 2013-06-17 21:24:23 +0800 |
---|---|---|
committer | MyungJoo Ham <myungjoo.ham@samsung.com> | 2013-11-15 13:52:52 +0900 |
commit | 0ff220df7d93fdd06775f5b39af87757973bc415 (patch) | |
tree | cb28525af0f7c80c02973852b77630b7e07ddc1f | |
parent | a4e537485cd11c8645b403db3853bbc10fffce60 (diff) | |
download | linux-3.10-0ff220df7d93fdd06775f5b39af87757973bc415.tar.gz linux-3.10-0ff220df7d93fdd06775f5b39af87757973bc415.tar.bz2 linux-3.10-0ff220df7d93fdd06775f5b39af87757973bc415.zip |
thermal: step_wise: cdev only needs update on a new target state
The cooling device only needs update on a new target state. Since we
already check old target in thermal_zone_trip_update(), we can do one
more check to see if it's a new target state. If not, we can reasonably
save some uncecesary code execution.
Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
Acked-by: Eduardo Valentin <eduardo.valentin@ti.com>
Signed-off-by: Zhang Rui <rui.zhang@intel.com>
-rw-r--r-- | drivers/thermal/step_wise.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/drivers/thermal/step_wise.c b/drivers/thermal/step_wise.c index 4d4ddae1a99..0afbd86e496 100644 --- a/drivers/thermal/step_wise.c +++ b/drivers/thermal/step_wise.c @@ -133,6 +133,9 @@ static void thermal_zone_trip_update(struct thermal_zone_device *tz, int trip) old_target = instance->target; instance->target = get_target_state(instance, trend, throttle); + if (old_target == instance->target) + continue; + /* Activate a passive thermal instance */ if (old_target == THERMAL_NO_TARGET && instance->target != THERMAL_NO_TARGET) |