thermal: fix off-by-1 error in trip point trigger condition
This patch fixes a regression caused by commit
b1569e99c7
"ACPI: move thermal trip handling to generic thermal layer"
which accidentally changed trip point trigger condition to
temp > trip_temp
This patch changes the trigger condition back to
temp >= trip_temp
Signed-off-by: Vladimir Zajac <eightgraph@gmail.com>
Acked-by: Zhang Rui <rui.zhang@intel.com>
Acked-by: Matthew Garrett <mjg@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Len Brown <len.brown@intel.com>
This commit is contained in:
parent
091bf7624d
commit
29321357ac
|
@ -961,7 +961,7 @@ void thermal_zone_device_update(struct thermal_zone_device *tz)
|
||||||
|
|
||||||
switch (trip_type) {
|
switch (trip_type) {
|
||||||
case THERMAL_TRIP_CRITICAL:
|
case THERMAL_TRIP_CRITICAL:
|
||||||
if (temp > trip_temp) {
|
if (temp >= trip_temp) {
|
||||||
if (tz->ops->notify)
|
if (tz->ops->notify)
|
||||||
ret = tz->ops->notify(tz, count,
|
ret = tz->ops->notify(tz, count,
|
||||||
trip_type);
|
trip_type);
|
||||||
|
@ -974,7 +974,7 @@ void thermal_zone_device_update(struct thermal_zone_device *tz)
|
||||||
}
|
}
|
||||||
break;
|
break;
|
||||||
case THERMAL_TRIP_HOT:
|
case THERMAL_TRIP_HOT:
|
||||||
if (temp > trip_temp)
|
if (temp >= trip_temp)
|
||||||
if (tz->ops->notify)
|
if (tz->ops->notify)
|
||||||
tz->ops->notify(tz, count, trip_type);
|
tz->ops->notify(tz, count, trip_type);
|
||||||
break;
|
break;
|
||||||
|
@ -986,14 +986,14 @@ void thermal_zone_device_update(struct thermal_zone_device *tz)
|
||||||
|
|
||||||
cdev = instance->cdev;
|
cdev = instance->cdev;
|
||||||
|
|
||||||
if (temp > trip_temp)
|
if (temp >= trip_temp)
|
||||||
cdev->ops->set_cur_state(cdev, 1);
|
cdev->ops->set_cur_state(cdev, 1);
|
||||||
else
|
else
|
||||||
cdev->ops->set_cur_state(cdev, 0);
|
cdev->ops->set_cur_state(cdev, 0);
|
||||||
}
|
}
|
||||||
break;
|
break;
|
||||||
case THERMAL_TRIP_PASSIVE:
|
case THERMAL_TRIP_PASSIVE:
|
||||||
if (temp > trip_temp || tz->passive)
|
if (temp >= trip_temp || tz->passive)
|
||||||
thermal_zone_device_passive(tz, temp,
|
thermal_zone_device_passive(tz, temp,
|
||||||
trip_temp, count);
|
trip_temp, count);
|
||||||
break;
|
break;
|
||||||
|
|
Loading…
Reference in New Issue