RE: [PATCH 01/30] thermal: cpu_cooling: remove unused headers

2013-04-24 Thread Zhang, Rui
> -Original Message- > From: linux-pm-ow...@vger.kernel.org [mailto:linux-pm- > ow...@vger.kernel.org] On Behalf Of Zhang, Rui > Sent: Thursday, April 25, 2013 12:32 AM > To: Eduardo Valentin; amit.kach...@linaro.org > Cc: linux...@vger.kernel.org; linux-kernel@vger.ke

RE: [PATCH 01/30] thermal: cpu_cooling: remove unused headers

2013-04-24 Thread Zhang, Rui
> -Original Message- > From: linux-pm-ow...@vger.kernel.org [mailto:linux-pm- > ow...@vger.kernel.org] On Behalf Of Eduardo Valentin > Sent: Thursday, April 18, 2013 1:12 AM > To: amit.kach...@linaro.org > Cc: Eduardo Valentin; Zhang, Rui; linux...@vger.kernel

RE: [PATCH 02/30] thermal: cpu_cooling: remove trailing white spaces

2013-04-24 Thread Zhang, Rui
> -Original Message- > From: linux-pm-ow...@vger.kernel.org [mailto:linux-pm- > ow...@vger.kernel.org] On Behalf Of Zhang, Rui > Sent: Wednesday, April 24, 2013 10:49 PM > To: Eduardo Valentin; amit.kach...@linaro.org > Cc: linux...@vger.kernel.org; linux-kernel@vger.ke

RE: [PATCH 26/30] thermal: cpu_cooling: remove unused symbols

2013-04-24 Thread Zhang, Rui
> -Original Message- > From: Eduardo Valentin [mailto:eduardo.valen...@ti.com] > Sent: Thursday, April 18, 2013 1:12 AM > To: amit.kach...@linaro.org > Cc: Eduardo Valentin; Zhang, Rui; linux...@vger.kernel.org; linux- > ker...@vger.kernel.org > Subject:

RE: [PATCH 03/30] thermal: cpu_cooling: fix kernel_doc for cpufreq_cooling_device

2013-04-24 Thread Zhang, Rui
> -Original Message- > From: Eduardo Valentin [mailto:eduardo.valen...@ti.com] > Sent: Thursday, April 18, 2013 1:12 AM > To: amit.kach...@linaro.org > Cc: Eduardo Valentin; Zhang, Rui; linux...@vger.kernel.org; linux- > ker...@vger.kernel.org > Subject:

RE: [PATCH 04/30] thermal: cpu_cooling: use EXPORT_SYMBOL_GPL

2013-04-24 Thread Zhang, Rui
Applied. Thanks, rui > -Original Message- > From: Eduardo Valentin [mailto:eduardo.valen...@ti.com] > Sent: Thursday, April 18, 2013 1:12 AM > To: amit.kach...@linaro.org > Cc: Eduardo Valentin; Zhang, Rui; linux...@vger.kernel.org; linux- > ker...@vger.kernel.org >

RE: [PATCH 02/30] thermal: cpu_cooling: remove trailing white spaces

2013-04-24 Thread Zhang, Rui
> -Original Message- > From: Eduardo Valentin [mailto:eduardo.valen...@ti.com] > Sent: Thursday, April 18, 2013 1:12 AM > To: amit.kach...@linaro.org > Cc: Eduardo Valentin; Zhang, Rui; linux...@vger.kernel.org; linux- > ker...@vger.kernel.org > Subject:

RE: [PATCH 01/30] thermal: cpu_cooling: remove unused headers

2013-04-24 Thread Zhang, Rui
> -Original Message- > From: linux-pm-ow...@vger.kernel.org [mailto:linux-pm- > ow...@vger.kernel.org] On Behalf Of Eduardo Valentin > Sent: Thursday, April 18, 2013 1:12 AM > To: amit.kach...@linaro.org > Cc: Eduardo Valentin; Zhang, Rui; linux...@vger.kernel

RE: [PATCH 01/30] thermal: cpu_cooling: remove unused headers

2013-04-24 Thread Zhang, Rui
-Original Message- From: linux-pm-ow...@vger.kernel.org [mailto:linux-pm- ow...@vger.kernel.org] On Behalf Of Eduardo Valentin Sent: Thursday, April 18, 2013 1:12 AM To: amit.kach...@linaro.org Cc: Eduardo Valentin; Zhang, Rui; linux...@vger.kernel.org; linux- ker

RE: [PATCH 02/30] thermal: cpu_cooling: remove trailing white spaces

2013-04-24 Thread Zhang, Rui
-Original Message- From: Eduardo Valentin [mailto:eduardo.valen...@ti.com] Sent: Thursday, April 18, 2013 1:12 AM To: amit.kach...@linaro.org Cc: Eduardo Valentin; Zhang, Rui; linux...@vger.kernel.org; linux- ker...@vger.kernel.org Subject: [PATCH 02/30] thermal: cpu_cooling

RE: [PATCH 04/30] thermal: cpu_cooling: use EXPORT_SYMBOL_GPL

2013-04-24 Thread Zhang, Rui
Applied. Thanks, rui -Original Message- From: Eduardo Valentin [mailto:eduardo.valen...@ti.com] Sent: Thursday, April 18, 2013 1:12 AM To: amit.kach...@linaro.org Cc: Eduardo Valentin; Zhang, Rui; linux...@vger.kernel.org; linux- ker...@vger.kernel.org Subject: [PATCH 04/30

RE: [PATCH 03/30] thermal: cpu_cooling: fix kernel_doc for cpufreq_cooling_device

2013-04-24 Thread Zhang, Rui
-Original Message- From: Eduardo Valentin [mailto:eduardo.valen...@ti.com] Sent: Thursday, April 18, 2013 1:12 AM To: amit.kach...@linaro.org Cc: Eduardo Valentin; Zhang, Rui; linux...@vger.kernel.org; linux- ker...@vger.kernel.org Subject: [PATCH 03/30] thermal: cpu_cooling: fix

RE: [PATCH 26/30] thermal: cpu_cooling: remove unused symbols

2013-04-24 Thread Zhang, Rui
-Original Message- From: Eduardo Valentin [mailto:eduardo.valen...@ti.com] Sent: Thursday, April 18, 2013 1:12 AM To: amit.kach...@linaro.org Cc: Eduardo Valentin; Zhang, Rui; linux...@vger.kernel.org; linux- ker...@vger.kernel.org Subject: [PATCH 26/30] thermal: cpu_cooling

RE: [PATCH 02/30] thermal: cpu_cooling: remove trailing white spaces

2013-04-24 Thread Zhang, Rui
-Original Message- From: linux-pm-ow...@vger.kernel.org [mailto:linux-pm- ow...@vger.kernel.org] On Behalf Of Zhang, Rui Sent: Wednesday, April 24, 2013 10:49 PM To: Eduardo Valentin; amit.kach...@linaro.org Cc: linux...@vger.kernel.org; linux-kernel@vger.kernel.org Subject: RE

RE: [PATCH 01/30] thermal: cpu_cooling: remove unused headers

2013-04-24 Thread Zhang, Rui
-Original Message- From: linux-pm-ow...@vger.kernel.org [mailto:linux-pm- ow...@vger.kernel.org] On Behalf Of Eduardo Valentin Sent: Thursday, April 18, 2013 1:12 AM To: amit.kach...@linaro.org Cc: Eduardo Valentin; Zhang, Rui; linux...@vger.kernel.org; linux- ker

RE: [PATCH 01/30] thermal: cpu_cooling: remove unused headers

2013-04-24 Thread Zhang, Rui
-Original Message- From: linux-pm-ow...@vger.kernel.org [mailto:linux-pm- ow...@vger.kernel.org] On Behalf Of Zhang, Rui Sent: Thursday, April 25, 2013 12:32 AM To: Eduardo Valentin; amit.kach...@linaro.org Cc: linux...@vger.kernel.org; linux-kernel@vger.kernel.org Subject: RE

RE: [PATCH RESEND 11/11] thermal: update kernel-doc for thermal_zone_device_register

2013-04-24 Thread Zhang, Rui
-Original Message- From: Eduardo Valentin [mailto:eduardo.valen...@ti.com] Sent: Thursday, April 18, 2013 3:30 AM To: Zhang, Rui Cc: Eduardo Valentin; linux...@vger.kernel.org; linux- ker...@vger.kernel.org Subject: Re: [PATCH RESEND 11/11] thermal: update kernel-doc

RE: [PATCHv2 0/9] thermal: clean ups on thermal code base

2013-04-24 Thread Zhang, Rui
Hi, Eduardo, -Original Message- From: linux-pm-ow...@vger.kernel.org [mailto:linux-pm- ow...@vger.kernel.org] On Behalf Of Eduardo Valentin Sent: Wednesday, April 24, 2013 5:48 AM To: Zhang, Rui Cc: linux...@vger.kernel.org; linux-kernel@vger.kernel.org; Eduardo Valentin Subject

Re: [PATCH 28/30] MAINTAINERS: update thermal entry by adding file cpu_cooling.h

2013-04-24 Thread Zhang Rui
On Wed, 2013-04-24 at 13:34 -0400, Eduardo Valentin wrote: Rui, On 17-04-2013 13:12, Eduardo Valentin wrote: Add cpu_cooling.h to thermal entry in MAINTAINERS. Cc: Zhang Rui rui.zh...@intel.com Cc: linux...@vger.kernel.org Cc: linux-kernel@vger.kernel.org Signed-off-by: Eduardo

Re: [PATCH RESEND 03/11] thermal: rename notify_thermal_framework to thermal_notify_framework

2013-04-17 Thread Zhang Rui
On Mon, 2013-04-08 at 12:07 -0400, Eduardo Valentin wrote: > To follow the prefix names used by the thermal exported functions, > this patch renames notify_thermal_framework to thermal_notify_framework. > > Signed-off-by: Eduardo Valentin Hmm, I'm okay with this one. thanks, rui > --- >

Re: [PATCH RESEND 01/11] thermal: use strlcpy instead of strcpy

2013-04-17 Thread Zhang Rui
On Mon, 2013-04-08 at 12:07 -0400, Eduardo Valentin wrote: > For memory boundaries safety, use strlcpy instead of strcpy. > > Signed-off-by: Eduardo Valentin ACK. thanks, rui > --- > drivers/thermal/thermal_sys.c |4 ++-- > 1 files changed, 2 insertions(+), 2 deletions(-) > > diff --git

Re: [PATCH RESEND 10/11] thermal: update kernel-doc for create_trip_attrs

2013-04-17 Thread Zhang Rui
On Mon, 2013-04-08 at 12:07 -0400, Eduardo Valentin wrote: > This patch updates the documentation for create_trip_attrs > and removes the warnings generated by scripts/kernel-doc -v. > > Signed-off-by: Eduardo Valentin this one looks okay to me. thanks, rui > --- >

Re: [PATCH RESEND 09/11] thermal: update kernel-doc for thermal_cooling_device_register

2013-04-17 Thread Zhang Rui
On Mon, 2013-04-08 at 12:07 -0400, Eduardo Valentin wrote: > This patch updates the documentation for thermal_cooling_device_register > and removes the warnings generated by scripts/kernel-doc -v. > > Signed-off-by: Eduardo Valentin > --- > drivers/thermal/thermal_sys.c |9 - > 1

Re: [PATCH RESEND 08/11] thermal: update kernel-doc for thermal_zone_unbind_cooling_device

2013-04-17 Thread Zhang Rui
On Mon, 2013-04-08 at 12:07 -0400, Eduardo Valentin wrote: > This patch updates the documentation for thermal_zone_unbind_cooling_device > and removes the warnings generated by scripts/kernel-doc -v. > > Signed-off-by: Eduardo Valentin I'm fine with this one. thanks, rui > --- >

Re: [PATCH RESEND 07/11] thermal: update kernel-doc for thermal_zone_bind_cooling_device

2013-04-17 Thread Zhang Rui
On Wed, 2013-04-17 at 23:26 +0800, Zhang Rui wrote: > On Mon, 2013-04-08 at 12:07 -0400, Eduardo Valentin wrote: > > This patch updates the documentation for thermal_zone_bind_cooling_device > > and removes the warnings generated by scripts/kernel-doc -v. > > > > Sign

Re: [PATCH RESEND 07/11] thermal: update kernel-doc for thermal_zone_bind_cooling_device

2013-04-17 Thread Zhang Rui
On Mon, 2013-04-08 at 12:07 -0400, Eduardo Valentin wrote: > This patch updates the documentation for thermal_zone_bind_cooling_device > and removes the warnings generated by scripts/kernel-doc -v. > > Signed-off-by: Eduardo Valentin > --- > drivers/thermal/thermal_sys.c | 16 +---

Re: [PATCH RESEND 11/11] thermal: update kernel-doc for thermal_zone_device_register

2013-04-17 Thread Zhang Rui
On Mon, 2013-04-08 at 12:07 -0400, Eduardo Valentin wrote: > This patch updates the documentation for thermal_zone_device_register it is not enough. parameters including const struct thermal_zone_params *tzp, int passive_delay, and int polling_delay are missing in the documentation. thanks, rui

Re: [PATCH RESEND 06/11] thermal: use EXPORT_SYMBOL_GPL

2013-04-17 Thread Zhang Rui
On Mon, 2013-04-08 at 12:07 -0400, Eduardo Valentin wrote: > Restrict usage of GPL modules. > > Signed-off-by: Eduardo Valentin NACK for thermal_zone_trend_get/thermal_instance_get/thermal_cdev_update, for the reasons stated earlier. ACK for the others. thanks, rui > --- >

Re: [PATCH RESEND 05/11] thermal: rename get_tz_trend to thermal_zone_trend_get

2013-04-17 Thread Zhang Rui
On Mon, 2013-04-08 at 12:07 -0400, Eduardo Valentin wrote: > To follow the prefix names used by the thermal exported functions, > this patch renames get_tz_trend to thermal_zone_trend_get. > > Signed-off-by: Eduardo Valentin NACK. get_tz_trend will not be exported as API any more. thanks, rui

Re: [PATCH RESEND 04/11] thermal: rename get_thermal_instance to thermal_instance_get

2013-04-17 Thread Zhang Rui
On Mon, 2013-04-08 at 12:07 -0400, Eduardo Valentin wrote: > To follow the prefix names used by the thermal exported functions, > this patch renames get_thermal_instance to thermal_instance_get. > > Signed-off-by: Eduardo Valentin NACK. with the change of building all thermal governors into

Re: [PATCH RESEND 04/11] thermal: rename get_thermal_instance to thermal_instance_get

2013-04-17 Thread Zhang Rui
On Mon, 2013-04-08 at 12:07 -0400, Eduardo Valentin wrote: To follow the prefix names used by the thermal exported functions, this patch renames get_thermal_instance to thermal_instance_get. Signed-off-by: Eduardo Valentin eduardo.valen...@ti.com NACK. with the change of building all

Re: [PATCH RESEND 05/11] thermal: rename get_tz_trend to thermal_zone_trend_get

2013-04-17 Thread Zhang Rui
On Mon, 2013-04-08 at 12:07 -0400, Eduardo Valentin wrote: To follow the prefix names used by the thermal exported functions, this patch renames get_tz_trend to thermal_zone_trend_get. Signed-off-by: Eduardo Valentin eduardo.valen...@ti.com NACK. get_tz_trend will not be exported as API any

Re: [PATCH RESEND 06/11] thermal: use EXPORT_SYMBOL_GPL

2013-04-17 Thread Zhang Rui
On Mon, 2013-04-08 at 12:07 -0400, Eduardo Valentin wrote: Restrict usage of GPL modules. Signed-off-by: Eduardo Valentin eduardo.valen...@ti.com NACK for thermal_zone_trend_get/thermal_instance_get/thermal_cdev_update, for the reasons stated earlier. ACK for the others. thanks, rui ---

Re: [PATCH RESEND 11/11] thermal: update kernel-doc for thermal_zone_device_register

2013-04-17 Thread Zhang Rui
On Mon, 2013-04-08 at 12:07 -0400, Eduardo Valentin wrote: This patch updates the documentation for thermal_zone_device_register it is not enough. parameters including const struct thermal_zone_params *tzp, int passive_delay, and int polling_delay are missing in the documentation. thanks, rui

Re: [PATCH RESEND 07/11] thermal: update kernel-doc for thermal_zone_bind_cooling_device

2013-04-17 Thread Zhang Rui
On Mon, 2013-04-08 at 12:07 -0400, Eduardo Valentin wrote: This patch updates the documentation for thermal_zone_bind_cooling_device and removes the warnings generated by scripts/kernel-doc -v. Signed-off-by: Eduardo Valentin eduardo.valen...@ti.com --- drivers/thermal/thermal_sys.c | 16

Re: [PATCH RESEND 07/11] thermal: update kernel-doc for thermal_zone_bind_cooling_device

2013-04-17 Thread Zhang Rui
On Wed, 2013-04-17 at 23:26 +0800, Zhang Rui wrote: On Mon, 2013-04-08 at 12:07 -0400, Eduardo Valentin wrote: This patch updates the documentation for thermal_zone_bind_cooling_device and removes the warnings generated by scripts/kernel-doc -v. Signed-off-by: Eduardo Valentin

Re: [PATCH RESEND 08/11] thermal: update kernel-doc for thermal_zone_unbind_cooling_device

2013-04-17 Thread Zhang Rui
On Mon, 2013-04-08 at 12:07 -0400, Eduardo Valentin wrote: This patch updates the documentation for thermal_zone_unbind_cooling_device and removes the warnings generated by scripts/kernel-doc -v. Signed-off-by: Eduardo Valentin eduardo.valen...@ti.com I'm fine with this one. thanks, rui

Re: [PATCH RESEND 09/11] thermal: update kernel-doc for thermal_cooling_device_register

2013-04-17 Thread Zhang Rui
On Mon, 2013-04-08 at 12:07 -0400, Eduardo Valentin wrote: This patch updates the documentation for thermal_cooling_device_register and removes the warnings generated by scripts/kernel-doc -v. Signed-off-by: Eduardo Valentin eduardo.valen...@ti.com --- drivers/thermal/thermal_sys.c |9

Re: [PATCH RESEND 10/11] thermal: update kernel-doc for create_trip_attrs

2013-04-17 Thread Zhang Rui
On Mon, 2013-04-08 at 12:07 -0400, Eduardo Valentin wrote: This patch updates the documentation for create_trip_attrs and removes the warnings generated by scripts/kernel-doc -v. Signed-off-by: Eduardo Valentin eduardo.valen...@ti.com this one looks okay to me. thanks, rui ---

Re: [PATCH RESEND 01/11] thermal: use strlcpy instead of strcpy

2013-04-17 Thread Zhang Rui
On Mon, 2013-04-08 at 12:07 -0400, Eduardo Valentin wrote: For memory boundaries safety, use strlcpy instead of strcpy. Signed-off-by: Eduardo Valentin eduardo.valen...@ti.com ACK. thanks, rui --- drivers/thermal/thermal_sys.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-)

Re: [PATCH RESEND 03/11] thermal: rename notify_thermal_framework to thermal_notify_framework

2013-04-17 Thread Zhang Rui
On Mon, 2013-04-08 at 12:07 -0400, Eduardo Valentin wrote: To follow the prefix names used by the thermal exported functions, this patch renames notify_thermal_framework to thermal_notify_framework. Signed-off-by: Eduardo Valentin eduardo.valen...@ti.com Hmm, I'm okay with this one. thanks,

Re: still in 3.9-rc6 - fan speed at 100% after suspend/resume regression

2013-04-15 Thread Zhang Rui
On Mon, 2013-04-15 at 09:30 -0400, Eduardo Valentin wrote: > On 13-04-2013 19:46, Ville Syrjälä wrote: > > On Sun, Apr 14, 2013 at 12:44:00AM +0300, Ville Syrjälä wrote: > >> On Sat, Apr 13, 2013 at 10:02:21AM -0600, Jake Edge wrote: > >>> Hi Zhang Rui, > >&g

Re: still in 3.9-rc6 - fan speed at 100% after suspend/resume regression

2013-04-15 Thread Zhang Rui
On Mon, 2013-04-15 at 09:30 -0400, Eduardo Valentin wrote: On 13-04-2013 19:46, Ville Syrjälä wrote: On Sun, Apr 14, 2013 at 12:44:00AM +0300, Ville Syrjälä wrote: On Sat, Apr 13, 2013 at 10:02:21AM -0600, Jake Edge wrote: Hi Zhang Rui, The problem reported in https://lkml.org/lkml/2012

Re: [PATCHv3 3/3] staging: ti-soc-thermal: remove external heat while extrapolating hotspot

2013-04-14 Thread Zhang Rui
On Fri, 2013-04-05 at 08:32 -0400, Eduardo Valentin wrote: > For boards that provide a PCB sensor close to SoC junction > temperature, it is possible to remove the cumulative heat > reported by the SoC temperature sensor. > > This patch changes the extrapolation computation to consider > an

Re: [PATCHv3 2/3] thermal: expose thermal_zone_get_temp API

2013-04-14 Thread Zhang Rui
On Fri, 2013-04-05 at 08:32 -0400, Eduardo Valentin wrote: > This patch exports the thermal_zone_get_temp API so that driver > writers can fetch temperature of thermal zones managed by other > drivers. > > Acked-by: Durgadoss R > Signed-off-by: Eduardo Valentin refreshed the patch to modify

Re: [PATCHv3 1/3] thermal: introduce thermal_zone_get_zone_by_name helper function

2013-04-14 Thread Zhang Rui
On Fri, 2013-04-05 at 08:32 -0400, Eduardo Valentin wrote: > This patch adds a helper function to get a reference of > a thermal zone, based on the zone type name. > > It will perform a zone name lookup and return a reference > to a thermal zone device that matches the name requested. > In case

Re: [PATCHv3 1/3] thermal: introduce thermal_zone_get_zone_by_name helper function

2013-04-14 Thread Zhang Rui
On Fri, 2013-04-05 at 08:32 -0400, Eduardo Valentin wrote: This patch adds a helper function to get a reference of a thermal zone, based on the zone type name. It will perform a zone name lookup and return a reference to a thermal zone device that matches the name requested. In case the

Re: [PATCHv3 2/3] thermal: expose thermal_zone_get_temp API

2013-04-14 Thread Zhang Rui
On Fri, 2013-04-05 at 08:32 -0400, Eduardo Valentin wrote: This patch exports the thermal_zone_get_temp API so that driver writers can fetch temperature of thermal zones managed by other drivers. Acked-by: Durgadoss R durgados...@intel.com Signed-off-by: Eduardo Valentin

Re: [PATCHv3 3/3] staging: ti-soc-thermal: remove external heat while extrapolating hotspot

2013-04-14 Thread Zhang Rui
On Fri, 2013-04-05 at 08:32 -0400, Eduardo Valentin wrote: For boards that provide a PCB sensor close to SoC junction temperature, it is possible to remove the cumulative heat reported by the SoC temperature sensor. This patch changes the extrapolation computation to consider an external

Re: [PATCH] Thermal: Don't resolve THERMAL_NO_LIMIT to max_state.

2013-04-11 Thread Zhang Rui
On Tue, 2013-03-26 at 21:29 +0800, Yuxuan Shui wrote: > max_state may change at runtime, for example, when loading/unloading > cpufreq policy. > this seems to be a problem that we have not covered yet. when loading/unloading the cpufreq policy, the cpufreq_frequency_table will be changed as

Re: [PATCH V2 1/3] Thermal: rename thermal_sys.c to thermal_core.c

2013-04-11 Thread Zhang Rui
On Thu, 2013-04-04 at 16:24 -0400, Eduardo Valentin wrote: > On 29-03-2013 10:26, Zhang Rui wrote: > > this is the preparation work to build all the thermal core framework > > source file, like governors, cpu cooling, etc, into one module. > > > > No functional change in

Re: [PATCH 14/30] thermal/exynos: remove unnecessary header inclusions

2013-04-11 Thread Zhang Rui
by only the exynos platform. Fortunately a number > > of drivers that include those headers do not actually need > > them, so we can just remove the inclusions. > > > > Signed-off-by: Arnd Bergmann > > Cc: linux...@vger.kernel.org > > Cc: Zhang Rui > > This patc

Re: [PATCH] thermal: step_wise: set throttle target within thermal instance limits

2013-04-11 Thread Zhang Rui
On Wed, 2013-04-10 at 16:22 -0400, Eduardo Valentin wrote: > On 10-04-2013 03:06, Zhang Rui wrote: > > On Tue, 2013-04-09 at 14:59 -0700, Andrew Bresticker wrote: > >> When selecting a target cooling state in get_target_state(), make sure > >> that the state is at least

Re: [PATCH] thermal: step_wise: set throttle target within thermal instance limits

2013-04-11 Thread Zhang Rui
On Wed, 2013-04-10 at 16:22 -0400, Eduardo Valentin wrote: On 10-04-2013 03:06, Zhang Rui wrote: On Tue, 2013-04-09 at 14:59 -0700, Andrew Bresticker wrote: When selecting a target cooling state in get_target_state(), make sure that the state is at least as high as the minimum when

Re: [PATCH 14/30] thermal/exynos: remove unnecessary header inclusions

2013-04-11 Thread Zhang Rui
a number of drivers that include those headers do not actually need them, so we can just remove the inclusions. Signed-off-by: Arnd Bergmann a...@arndb.de Cc: linux...@vger.kernel.org Cc: Zhang Rui rui.zh...@intel.com This patch looks good to me. You can add my: Acked-by: Eduardo

Re: [PATCH V2 1/3] Thermal: rename thermal_sys.c to thermal_core.c

2013-04-11 Thread Zhang Rui
On Thu, 2013-04-04 at 16:24 -0400, Eduardo Valentin wrote: On 29-03-2013 10:26, Zhang Rui wrote: this is the preparation work to build all the thermal core framework source file, like governors, cpu cooling, etc, into one module. No functional change in this patch. Signed-off

Re: [PATCH] Thermal: Don't resolve THERMAL_NO_LIMIT to max_state.

2013-04-11 Thread Zhang Rui
On Tue, 2013-03-26 at 21:29 +0800, Yuxuan Shui wrote: max_state may change at runtime, for example, when loading/unloading cpufreq policy. this seems to be a problem that we have not covered yet. when loading/unloading the cpufreq policy, the cpufreq_frequency_table will be changed as well,

Re: [PATCH] thermal: step_wise: set throttle target within thermal instance limits

2013-04-10 Thread Zhang Rui
On Tue, 2013-04-09 at 14:59 -0700, Andrew Bresticker wrote: > When selecting a target cooling state in get_target_state(), make sure > that the state is at least as high as the minimum when the temperature > is rising and at least as low as the maximum when the temperature is > falling. This is

Re: [PATCH] thermal: step_wise: set throttle target within thermal instance limits

2013-04-10 Thread Zhang Rui
On Tue, 2013-04-09 at 14:59 -0700, Andrew Bresticker wrote: When selecting a target cooling state in get_target_state(), make sure that the state is at least as high as the minimum when the temperature is rising and at least as low as the maximum when the temperature is falling. This is

Re: [PATCH] thermal: exynos: fix handling of invalid frequency table entries

2013-04-09 Thread Zhang Rui
Hi, Andrew, can you please verify commit fc35b35cbe24ef021ea9acfba21e54da958df747 commit 57df8106932b57427df1eaaa13871857f75b1194 at http://git.kernel.org/cgit/linux/kernel/git/rzhang/linux.git/log/?h=thermal fixes the problem for you? thanks, rui On Tue, 2013-04-09 at 14:59 -0700, Andrew

Re: [PATCH] thermal: exynos: fix handling of invalid frequency table entries

2013-04-09 Thread Zhang Rui
Hi, Andrew, can you please verify commit fc35b35cbe24ef021ea9acfba21e54da958df747 commit 57df8106932b57427df1eaaa13871857f75b1194 at http://git.kernel.org/cgit/linux/kernel/git/rzhang/linux.git/log/?h=thermal fixes the problem for you? thanks, rui On Tue, 2013-04-09 at 14:59 -0700, Andrew

[PATCH V2 2/3] Thermal: build thermal governors into thermal_sys module

2013-03-29 Thread Zhang Rui
a problem that the generic thermal layer does not work when CONFIG_THERMAL=m and CONFIG_THERMAL_GOV_XXX=y. Signed-off-by: Zhang Rui --- Documentation/thermal/sysfs-api.txt |8 - drivers/thermal/Makefile|6 ++-- drivers/thermal/fair_share.c| 15 ++--- drivers

[PATCH V2 1/3] Thermal: rename thermal_sys.c to thermal_core.c

2013-03-29 Thread Zhang Rui
this is the preparation work to build all the thermal core framework source file, like governors, cpu cooling, etc, into one module. No functional change in this patch. Signed-off-by: Zhang Rui --- drivers/thermal/Makefile |1 + drivers/thermal/{thermal_sys.c

[PATCH V2 3/3] Thermal: build cpu_cooling code into thermal_sys module

2013-03-29 Thread Zhang Rui
Signed-off-by: Zhang Rui --- drivers/thermal/Kconfig |2 +- drivers/thermal/Makefile|2 +- include/linux/cpu_cooling.h |2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig index a764f16..10014de 100644

[PATCH V2 0/3] Thermal: build all thermal framework code into thermal_sys module

2013-03-29 Thread Zhang Rui
A regression introduced by thermal governor feature is reported in http://marc.info/?l=linux-kernel=136085598604095=2 And the root cause is that all the thermal governors can not be registered successfully when the thermal framework is built as a module. Thus I made this patch set to build all

[PATCH V2 0/3] Thermal: build all thermal framework code into thermal_sys module

2013-03-29 Thread Zhang Rui
A regression introduced by thermal governor feature is reported in http://marc.info/?l=linux-kernelm=136085598604095w=2 And the root cause is that all the thermal governors can not be registered successfully when the thermal framework is built as a module. Thus I made this patch set to build all

[PATCH V2 3/3] Thermal: build cpu_cooling code into thermal_sys module

2013-03-29 Thread Zhang Rui
Signed-off-by: Zhang Rui rui.zh...@intel.com --- drivers/thermal/Kconfig |2 +- drivers/thermal/Makefile|2 +- include/linux/cpu_cooling.h |2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig index a764f16

[PATCH V2 2/3] Thermal: build thermal governors into thermal_sys module

2013-03-29 Thread Zhang Rui
a problem that the generic thermal layer does not work when CONFIG_THERMAL=m and CONFIG_THERMAL_GOV_XXX=y. Signed-off-by: Zhang Rui rui.zh...@intel.com --- Documentation/thermal/sysfs-api.txt |8 - drivers/thermal/Makefile|6 ++-- drivers/thermal/fair_share.c| 15

[PATCH V2 1/3] Thermal: rename thermal_sys.c to thermal_core.c

2013-03-29 Thread Zhang Rui
this is the preparation work to build all the thermal core framework source file, like governors, cpu cooling, etc, into one module. No functional change in this patch. Signed-off-by: Zhang Rui rui.zh...@intel.com --- drivers/thermal/Makefile |1 + drivers/thermal

RE: [RFC PATCH 3/5] Thermal: build thermal governors into thermal_sys module

2013-03-26 Thread Zhang Rui
On Tue, 2013-03-26 at 10:53 -0600, R, Durgadoss wrote: > Hi Rui, > > > -Original Message- > > From: Zhang, Rui > > Sent: Tuesday, March 26, 2013 9:56 PM > > To: linux...@vger.kernel.org; linux-kernel@vger.kernel.org > > Cc: amit.dan...@samsung.com; R,

[RFC PATCH 4/5] Thermal: governor API cleanup

2013-03-26 Thread Zhang Rui
Signed-off-by: Zhang Rui --- drivers/thermal/thermal_core.c |2 -- drivers/thermal/thermal_core.h |2 ++ include/linux/thermal.h|3 --- 3 files changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/thermal/thermal_core.c b/drivers/thermal/thermal_core.c index eac9745

[RFC PATCH 5/5] Thermal: build cpu_cooling code into thermal_sys module

2013-03-26 Thread Zhang Rui
Signed-off-by: Zhang Rui --- drivers/thermal/Kconfig |2 +- drivers/thermal/Makefile|2 +- include/linux/cpu_cooling.h |2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig index a764f16..10014de 100644

[RFC PATCH 3/5] Thermal: build thermal governors into thermal_sys module

2013-03-26 Thread Zhang Rui
Signed-off-by: Zhang Rui --- drivers/thermal/Makefile |6 +++--- drivers/thermal/fair_share.c | 15 ++- drivers/thermal/step_wise.c| 16 ++-- drivers/thermal/thermal_core.c | 36 ++-- drivers/thermal/thermal_core.h

[RFC PATCH 2/5] Thermal: thermal framework registration failure case cleanup

2013-03-26 Thread Zhang Rui
Signed-off-by: Zhang Rui --- drivers/thermal/thermal_core.c | 12 ++-- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/drivers/thermal/thermal_core.c b/drivers/thermal/thermal_core.c index 5b7863a..845ed6e 100644 --- a/drivers/thermal/thermal_core.c +++ b/drivers/thermal

[RFC PATCH 0/5] Thermal: build all thermal framework code into thermal_sys module

2013-03-26 Thread Zhang Rui
A regression introduced by thermal governor feature is reported in http://marc.info/?l=linux-kernel=136085598604095=2 And the root cause is that all the thermal governors can not be registered successfully when the thermal framework is built as a module. Thus I made this patch set to build all

[RFC PATCH 0/5] Thermal: build all thermal framework code into thermal_sys module

2013-03-26 Thread Zhang Rui
A regression introduced by thermal governor feature is reported in http://marc.info/?l=linux-kernelm=136085598604095w=2 And the root cause is that all the thermal governors can not be registered successfully when the thermal framework is built as a module. Thus I made this patch set to build all

[RFC PATCH 2/5] Thermal: thermal framework registration failure case cleanup

2013-03-26 Thread Zhang Rui
Signed-off-by: Zhang Rui rui.zh...@intel.com --- drivers/thermal/thermal_core.c | 12 ++-- 1 file changed, 2 insertions(+), 10 deletions(-) diff --git a/drivers/thermal/thermal_core.c b/drivers/thermal/thermal_core.c index 5b7863a..845ed6e 100644 --- a/drivers/thermal/thermal_core.c

[RFC PATCH 3/5] Thermal: build thermal governors into thermal_sys module

2013-03-26 Thread Zhang Rui
Signed-off-by: Zhang Rui rui.zh...@intel.com --- drivers/thermal/Makefile |6 +++--- drivers/thermal/fair_share.c | 15 ++- drivers/thermal/step_wise.c| 16 ++-- drivers/thermal/thermal_core.c | 36 ++-- drivers/thermal

[RFC PATCH 5/5] Thermal: build cpu_cooling code into thermal_sys module

2013-03-26 Thread Zhang Rui
Signed-off-by: Zhang Rui rui.zh...@intel.com --- drivers/thermal/Kconfig |2 +- drivers/thermal/Makefile|2 +- include/linux/cpu_cooling.h |2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig index a764f16

[RFC PATCH 4/5] Thermal: governor API cleanup

2013-03-26 Thread Zhang Rui
Signed-off-by: Zhang Rui rui.zh...@intel.com --- drivers/thermal/thermal_core.c |2 -- drivers/thermal/thermal_core.h |2 ++ include/linux/thermal.h|3 --- 3 files changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/thermal/thermal_core.c b/drivers/thermal

RE: [RFC PATCH 3/5] Thermal: build thermal governors into thermal_sys module

2013-03-26 Thread Zhang Rui
On Tue, 2013-03-26 at 10:53 -0600, R, Durgadoss wrote: Hi Rui, -Original Message- From: Zhang, Rui Sent: Tuesday, March 26, 2013 9:56 PM To: linux...@vger.kernel.org; linux-kernel@vger.kernel.org Cc: amit.dan...@samsung.com; R, Durgadoss; a...@lisas.de; Zhang, Rui Subject

Re: [PATCH V2] drivers: thermal: cpu_cooling: fix checkpatch warning

2013-03-25 Thread Zhang Rui
On Tue, 2013-03-12 at 11:47 +0100, Laurent Navet wrote: > - WARNING: Avoid CamelCase: > > Signed-off-by: Laurent Navet applied to thermal -next. thanks, rui > --- > drivers/thermal/cpu_cooling.c | 10 +- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git

RE: [PATCH 1/2] thermal: introduce thermal_zone_lookup_temperature helper function

2013-03-25 Thread Zhang Rui
On Mon, 2013-03-25 at 00:20 -0600, R, Durgadoss wrote: > > -Original Message- > > From: linux-pm-ow...@vger.kernel.org [mailto:linux-pm- > > ow...@vger.kernel.org] On Behalf Of Zhang Rui > > Sent: Monday, March 25, 2013 11:41 AM > > To: Eduardo Valentin &g

Re: [PATCH 2/2] staging: ti-soc-thermal: remove external heat while extrapolating hotspot

2013-03-25 Thread Zhang Rui
On Fri, 2013-03-22 at 17:13 -0400, Eduardo Valentin wrote: > For boards that provide a PCB sensor close to SoC junction > temperature, it is possible to remove the cumulative heat > reported by the SoC temperature sensor. > > This patch changes the extrapolation computation to consider > an

Re: [PATCH 1/2] thermal: introduce thermal_zone_lookup_temperature helper function

2013-03-25 Thread Zhang Rui
On Fri, 2013-03-22 at 17:13 -0400, Eduardo Valentin wrote: > This patch adds a helper function to get temperature of > a thermal zone, based on the zone type name. > > It will perform a zone name lookup and return the last > sensor temperature reading. In case the zone is not found > or if the

Re: [PATCH 1/2] thermal: introduce thermal_zone_lookup_temperature helper function

2013-03-25 Thread Zhang Rui
On Fri, 2013-03-22 at 17:13 -0400, Eduardo Valentin wrote: This patch adds a helper function to get temperature of a thermal zone, based on the zone type name. It will perform a zone name lookup and return the last sensor temperature reading. In case the zone is not found or if the required

Re: [PATCH 2/2] staging: ti-soc-thermal: remove external heat while extrapolating hotspot

2013-03-25 Thread Zhang Rui
On Fri, 2013-03-22 at 17:13 -0400, Eduardo Valentin wrote: For boards that provide a PCB sensor close to SoC junction temperature, it is possible to remove the cumulative heat reported by the SoC temperature sensor. This patch changes the extrapolation computation to consider an external

RE: [PATCH 1/2] thermal: introduce thermal_zone_lookup_temperature helper function

2013-03-25 Thread Zhang Rui
On Mon, 2013-03-25 at 00:20 -0600, R, Durgadoss wrote: -Original Message- From: linux-pm-ow...@vger.kernel.org [mailto:linux-pm- ow...@vger.kernel.org] On Behalf Of Zhang Rui Sent: Monday, March 25, 2013 11:41 AM To: Eduardo Valentin Cc: linux...@vger.kernel.org; linux-kernel

Re: [PATCH V2] drivers: thermal: cpu_cooling: fix checkpatch warning

2013-03-25 Thread Zhang Rui
On Tue, 2013-03-12 at 11:47 +0100, Laurent Navet wrote: - WARNING: Avoid CamelCase: maskPtr Signed-off-by: Laurent Navet laurent.na...@gmail.com applied to thermal -next. thanks, rui --- drivers/thermal/cpu_cooling.c | 10 +- 1 file changed, 5 insertions(+), 5 deletions(-)

Re: [PATCH] kirkwood: fix coccicheck warnings

2013-03-11 Thread Zhang Rui
On Mon, 2013-03-11 at 18:03 +0100, Rafael J. Wysocki wrote: > On Monday, March 11, 2013 10:55:00 PM Zhang Rui wrote: > > On Mon, 2013-03-11 at 09:35 +0200, Silviu-Mihai Popescu wrote: > > > Convert all uses of devm_request_and_ioremap() to the newly introduced > > > de

Re: [PATCH] kirkwood: fix coccicheck warnings

2013-03-11 Thread Zhang Rui
On Mon, 2013-03-11 at 09:35 +0200, Silviu-Mihai Popescu wrote: > Convert all uses of devm_request_and_ioremap() to the newly introduced > devm_ioremap_resource() which provides more consistent error handling. > > devm_ioremap_resource() provides its own error messages so all explicit > error

Re: [PATCH] drivers: thermal: cpu_cooling: fix checkpatch warning

2013-03-11 Thread Zhang Rui
On Tue, 2013-02-26 at 14:42 +0100, Laurent Navet wrote: > - WARNING: Avoid CamelCase: > > Signed-off-by: Laurent Navet > --- > drivers/thermal/cpu_cooling.c |6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/thermal/cpu_cooling.c

Re: [PATCH] drivers: thermal: cpu_cooling: fix checkpatch warning

2013-03-11 Thread Zhang Rui
On Tue, 2013-02-26 at 14:42 +0100, Laurent Navet wrote: - WARNING: Avoid CamelCase: maskPtr Signed-off-by: Laurent Navet laurent.na...@gmail.com --- drivers/thermal/cpu_cooling.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/thermal/cpu_cooling.c

Re: [PATCH] kirkwood: fix coccicheck warnings

2013-03-11 Thread Zhang Rui
On Mon, 2013-03-11 at 09:35 +0200, Silviu-Mihai Popescu wrote: Convert all uses of devm_request_and_ioremap() to the newly introduced devm_ioremap_resource() which provides more consistent error handling. devm_ioremap_resource() provides its own error messages so all explicit error messages

Re: [PATCH] kirkwood: fix coccicheck warnings

2013-03-11 Thread Zhang Rui
On Mon, 2013-03-11 at 18:03 +0100, Rafael J. Wysocki wrote: On Monday, March 11, 2013 10:55:00 PM Zhang Rui wrote: On Mon, 2013-03-11 at 09:35 +0200, Silviu-Mihai Popescu wrote: Convert all uses of devm_request_and_ioremap() to the newly introduced devm_ioremap_resource() which provides

Re: [PATCH] acerhdf: Fix fan activation with new thermal governor

2013-03-05 Thread Zhang Rui
On Wed, 2013-03-06 at 08:00 +0100, Borislav Petkov wrote: > On Wed, Mar 06, 2013 at 11:47:20AM +0800, Zhang Rui wrote: > > > So this now does not have any of the Schmitt trigger equivalent (?) > > > governor changes that were nicely discussed in this activity - > > &

Re: [PATCH] acerhdf: Fix fan activation with new thermal governor

2013-03-05 Thread Zhang Rui
et, though. Will do more testing. > > So this now does not have any of the Schmitt trigger equivalent (?) > governor changes that were nicely discussed in this activity - > So what do we want to have committed, and what do we not, the problem should have been fixed in 3.9-rc1 by commit

Re: [PATCH] acerhdf: Fix fan activation with new thermal governor

2013-03-05 Thread Zhang Rui
that were nicely discussed in this activity - So what do we want to have committed, and what do we not, the problem should have been fixed in 3.9-rc1 by commit b8bb6cb999858043489c1ddef08eed2127559169 Author: Zhang Rui rui.zh...@intel.com Date: Thu Nov 22 15:45:02 2012 +0800 step_wise: Unify

Re: [PATCH] acerhdf: Fix fan activation with new thermal governor

2013-03-05 Thread Zhang Rui
On Wed, 2013-03-06 at 08:00 +0100, Borislav Petkov wrote: On Wed, Mar 06, 2013 at 11:47:20AM +0800, Zhang Rui wrote: So this now does not have any of the Schmitt trigger equivalent (?) governor changes that were nicely discussed in this activity - So what do we want to have committed

[GIT PULL] thermal management updates for v3.9-rc1

2013-02-27 Thread Zhang Rui
an error on failure to register thermal class Sachin Kamat (3): thermal: exynos: Use of_match_ptr() macro thermal: db8500: Use of_match_ptr() macro in db8500_thermal.c thermal: db8500: Use of_match_ptr() macro in db8500_cpufreq_cooling.c Zhang Rui (6): Introduce

<    8   9   10   11   12   13   14   15   16   17   >