lkml.org 
[lkml]   [2013]   [May]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [PATCH 3/3] Thermal:core: Handle trips focused on current trip point only.
Date


> -----Original Message-----
> From: Jonghwa Lee [mailto:jonghwa3.lee@samsung.com]
> Sent: Saturday, May 18, 2013 5:51 PM
> To: linux-pm@vger.kernel.org
> Cc: linux-kernel@vger.kernel.org; Zhang, Rui; Eduardo Valentin; Amit
> Dinel Kachhap; Jonghwa Lee; MyungJoo Ham
> Subject: [PATCH 3/3] Thermal:core: Handle trips focused on current trip
> point only.
> Importance: High
>
> When thermal zone device is updated, it doesn't need to check every
> trip points and its handling mathod even current temperature doesn't
> exceed the trip's temperature. To modify those dissipatve mechanism,
> this patch introduces the way to get current thermal trip point to call
> only correspond trip point handling.
>
> Signed-off-by: Jonghwa Lee <jonghwa3.lee@samsung.com>
> Signed-off-by: MyungJoo Ham <myungjoo.ham@samsung.com>

NAK.

> ---
> drivers/thermal/thermal_core.c | 28 +++++++++++++++++-----------
> 1 file changed, 17 insertions(+), 11 deletions(-)
>
> diff --git a/drivers/thermal/thermal_core.c
> b/drivers/thermal/thermal_core.c index ce4384a..1cc4825 100644
> --- a/drivers/thermal/thermal_core.c
> +++ b/drivers/thermal/thermal_core.c
> @@ -333,14 +333,6 @@ static void handle_non_critical_trips(struct
> thermal_zone_device *tz, static void handle_critical_trips(struct
> thermal_zone_device *tz,
> int trip, enum thermal_trip_type trip_type) {
> - long trip_temp;
> -
> - tz->ops->get_trip_temp(tz, trip, &trip_temp);
> -
> - /* If we have not crossed the trip_temp, we do not care. */
> - if (tz->temperature < trip_temp)
> - return;
> -
> if (tz->ops->notify)
> tz->ops->notify(tz, trip, trip_type);
>
> @@ -437,14 +429,28 @@ static void update_temperature(struct
> thermal_zone_device *tz)
> mutex_unlock(&tz->lock);
> }
>
> +static int thermal_zone_get_current_trip(struct thermal_zone_device
> +*tz) {
> + int trip;
> + long trip_temp;
> +
> + for (trip = tz->trips - 1; trip > 0; trip--) {
> + tz->ops->get_trip_temp(tz, trip, &trip_temp);
> + if (tz->temperature > trip_temp)
> + continue;
> + }
> + return trip;
> +}
> +
> void thermal_zone_device_update(struct thermal_zone_device *tz) {
> - int count;
> + int trip;
>
> update_temperature(tz);
>
> - for (count = 0; count < tz->trips; count++)
> - handle_thermal_trip(tz, count);
> + trip = thermal_zone_get_current_trip(tz);
> +
> + handle_thermal_trip(tz, trip);

Say, trip point 1 for thermal zone 0 is 60C,
The system is running above 60C for somethime,
thus the thermal_instance for this trip point is running at upper_limit.
When the temperature suddenly drops below 60C,
we still need to handle trip point 1 to deactivate it.

Thanks,
rui
> }
> EXPORT_SYMBOL_GPL(thermal_zone_device_update);
>
> --
> 1.7.9.5



\
 
 \ /
  Last update: 2013-05-20 20:41    [W:0.043 / U:0.096 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site