From: Shawn Guo Date: Mon, 17 Jun 2013 13:24:23 +0000 (+0800) Subject: thermal: step_wise: cdev only needs update on a new target state X-Git-Tag: v3.12-rc1~45^2~11^2~1 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=178c2490b99f898efc06d1ad75cadc84f13021a6;p=profile%2Fivi%2Fkernel-x86-ivi.git 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 Acked-by: Eduardo Valentin Signed-off-by: Zhang Rui --- diff --git a/drivers/thermal/step_wise.c b/drivers/thermal/step_wise.c index 4d4ddae..0afbd86 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)