[RFC PATCH 4/6] Thermal: fix step_wise handling of THERMAL_TREND_DROP_EFULL

[Zhang Rui <rui.zhang@intel.com>]

Change the step_wise cooling algorithm to handle
THERMAL_TREND_DROP_FULL a bit differently.

When the temperature is higher than the trip point,
we should always use the instance->lower as the next target state.
When the temperature is lower than the trip point,
we should always deactive the thermal instance.

Signed-off-by: Zhang Rui <rui.zhang@intel.com>
---
 drivers/thermal/step_wise.c |   12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/drivers/thermal/step_wise.c b/drivers/thermal/step_wise.c
index f0cc5e5..84d6e90 100644
--- a/drivers/thermal/step_wise.c
+++ b/drivers/thermal/step_wise.c
@@ -42,9 +42,8 @@
  *       state for this trip point, if the cooling state already
  *       equals lower limit, deactivate the thermal instance
  *    c. if the trend is THERMAL_TREND_RAISE_FULL, do nothing
- *    d. if the trend is THERMAL_TREND_DROP_FULL, use lower limit,
- *       if the cooling state already equals lower limit,
- *       deactive the thermal instance
+ *    d. if the trend is THERMAL_TREND_DROP_FULL, deactive
+ *	 the thermal instance
  */
 static unsigned long get_target_state(struct thermal_instance *instance,
 				enum thermal_trend trend, bool throttle)
@@ -91,11 +90,10 @@ static unsigned long get_target_state(struct thermal_instance *instance,
 		}
 		break;
 	case THERMAL_TREND_DROP_FULL:
-		if (cur_state == instance->lower) {
-			if (!throttle)
-				next_target = THERMAL_NO_TARGET;
-		} else
+		if (throttle)
 			next_target = instance->lower;
+		else
+			next_target = THERMAL_NO_TARGET;
 		break;
 	default:
 		break;
-- 
1.7.9.5