Re: Re: [PATCH v4 2/3] devfreq: Add suspend and resume apis

2012-10-16 Thread MyungJoo Ham
On Monday 08 of October 2012 10:48:24 MyungJoo Ham wrote: On 8 October 2012 03:31, Rafael J. Wysocki r...@sisk.pl wrote: On Thursday 04 of October 2012 14:58:33 Rajagopal Venkat wrote: Add devfreq suspend/resume apis for devfreq users. This patch supports suspend and resume of

[PATCH 3/5] Thermal: fix empty list checking method.

2012-10-16 Thread hongbo.zhang
From: hongbo.zhang hongbo.zh...@linaro.com Is is not reliable to check the list entry pointer after list_for_each_entry loop, list_empty should be used instead. Signed-off-by: hongbo.zhang hongbo.zh...@linaro.com --- drivers/thermal/cpu_cooling.c | 5 +++-- 1 file changed, 3 insertions(+), 2

[PATCH 0/5] Fix thermal bugs and Upstream ST-Ericsson thermal driver

2012-10-16 Thread hongbo.zhang
From: hongbo.zhang hongbo.zh...@linaro.com Hi all, This patch set is to upstream ST-Ericsson thermal driver and fix some bugs of thermal layer at the same time. All of these patches are based on v3.7-rc1. [PATCH 1/5] Thermal: do bind operation after thermal zone or cooling device register

[PATCH 1/5] Thermal: do bind operation after thermal zone or cooling device register returns.

2012-10-16 Thread hongbo.zhang
From: hongbo.zhang hongbo.zh...@linaro.com In the previous bind function, cdev-get_max_state(cdev, max_state) is called before the registration function finishes, but at this moment, the parameter cdev at thermal driver layer isn't ready--it will get ready only after its registration, so the the

[PATCH 4/5] Thermal: make sure cpufreq cooling register after cpufreq driver

2012-10-16 Thread hongbo.zhang
From: hongbo.zhang hongbo.zh...@linaro.com The cpufreq works as a cooling device, so the cooling layer should check and wait until the cpufreq driver is initialized. Signed-off-by: hongbo.zhang hongbo.zh...@linaro.com --- drivers/thermal/cpu_cooling.c | 12 +++- 1 file changed, 11

Re: [PATCH] cpufreq:core: Fix printing of governor and driver name

2012-10-16 Thread Rafael J. Wysocki
On Tuesday 16 of October 2012 09:09:15 Viresh Kumar wrote: On 15 October 2012 23:21, Rafael J. Wysocki r...@sisk.pl wrote: On Wednesday 10 of October 2012 10:12:11 Viresh Kumar wrote: Arrays for governer and driver name are of size CPUFREQ_NAME_LEN or 16. i.e. 15 bytes for name and 1 for

Re: [PATCH v4 2/3] devfreq: Add suspend and resume apis

2012-10-16 Thread Rafael J. Wysocki
On Tuesday 16 of October 2012 06:40:19 MyungJoo Ham wrote: On Monday 08 of October 2012 10:48:24 MyungJoo Ham wrote: On 8 October 2012 03:31, Rafael J. Wysocki r...@sisk.pl wrote: On Thursday 04 of October 2012 14:58:33 Rajagopal Venkat wrote: Add devfreq suspend/resume apis for

Agenda for 2012-10-17 posted

2012-10-16 Thread Zach Pfeffer
The agenda for tomorrow's meeting has been posted here: https://wiki.linaro.org/Platform/Android/Meetings/2012-10-17 Please take a look, add any agenda items you'd like to talk about and add your status. See you tomorrow! -- Zach Pfeffer Android Platform Team Lead, Linaro Platform Teams

[PATCH 2/2] cpufreq: Debugging options for the cpufreq subsystem

2012-10-16 Thread Viresh Kumar
This adds Kconfig options for DEBUG and VERBOSE_DEBUG to the cpufreq subsystem, This is pretty useful for developers who want to debug cpufreq subsystem and don't want to editing the Makefile manually each time they want to debug. Signed-off-by: Viresh Kumar viresh.ku...@linaro.org ---

[PATCH 1/2] cpufreq: Improve debug prints

2012-10-16 Thread Viresh Kumar
With debug options on, it is difficult to locate cpufreq core's debug prints. Fix this by prefixing debug prints with: cpufreq: Signed-off-by: Viresh Kumar viresh.ku...@linaro.org --- drivers/cpufreq/cpufreq.c | 2 ++ drivers/cpufreq/cpufreq_performance.c | 2 ++

Re: [PATCH 1/4] cpuidle - sysfs : change function parameter

2012-10-16 Thread Rafael J. Wysocki
On Thursday 11 of October 2012 00:21:28 Daniel Lezcano wrote: The function needs the cpuidle_device which is initially passed to the caller. The current code gets the struct device from the struct cpuidle_device, pass it the cpuidle_add_sysfs function. This function calls

Re: [PATCH 2/4] cpuidle - sysfs : move kobj initialization in the syfs file

2012-10-16 Thread Rafael J. Wysocki
On Thursday 11 of October 2012 00:21:29 Daniel Lezcano wrote: Move the kobj initialization and completion in the sysfs.c and encapsulate the code more. Signed-off-by: Daniel Lezcano daniel.lezc...@linaro.org I like this one too and I'm going to take it for v3.8 as well. Thanks, Rafael

Re: [PATCH 3/4] cpuidle - sysfs : change function name

2012-10-16 Thread Rafael J. Wysocki
On Thursday 11 of October 2012 00:21:30 Daniel Lezcano wrote: The next patchset providing the multiple drivers support will add to the per cpu sysfs driver. In order to prepare the place this patch creates a single entry function when a cpuidle device is added or removed. Signed-off-by:

Re: [PATCH 4/4] cpuidle - sysfs : move declaration in the sysfs file

2012-10-16 Thread Rafael J. Wysocki
On Thursday 11 of October 2012 00:21:31 Daniel Lezcano wrote: The structure cpuidle_state_kobj is not used anywhere except in the sysfs.c file. The definition of this function is not Surely structure? needed in the cpuidle header file.

Re: [PATCH 1/2] cpufreq: Improve debug prints

2012-10-16 Thread Viresh Kumar
On 17 October 2012 11:09, Joe Perches j...@perches.com wrote: On Wed, 2012-10-17 at 10:50 +0530, Viresh Kumar wrote: With debug options on, it is difficult to locate cpufreq core's debug prints. [] diff --git a/drivers/cpufreq/cpufreq.c b/drivers/cpufreq/cpufreq.c [] @@ -15,6 +15,8 @@