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

2012-10-30 Thread hongbo.zhang
From: hongbo.zhang hongbo.zh...@linaro.com V2-V3 Changes: 1. Moved the previous [PATCH V2 2/6] Thermal: make sure cpufreq cooling register after cpufreq driver from generic cpu cooling layer to ST-Ericsson driver, thus only 5 patches in total in V3 patch set. 2. Update ST-Ericsson thermal

[PATCH V3 1/5] Thermal: add indent for code alignment.

2012-10-30 Thread hongbo.zhang
From: hongbo.zhang hongbo.zh...@linaro.com The curly bracket should be aligned with corresponding if else statements. Signed-off-by: hongbo.zhang hongbo.zh...@linaro.com Reviewed-by: Viresh Kumar viresh.ku...@linaro.org --- drivers/thermal/cpu_cooling.c | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH V3 2/5] Thermal: fix bug of counting cpu frequencies.

2012-10-30 Thread hongbo.zhang
From: hongbo.zhang hongbo.zh...@linaro.com In the while loop for counting cpu frequencies, if table[i].frequency equals CPUFREQ_ENTRY_INVALID, index i won't be increased, so this leads to an endless loop, what's more the index i cannot be referred as cpu frequencies number

[PATCH V3 4/5] Thermal: Add ST-Ericsson DB8500 thermal driver.

2012-10-30 Thread hongbo.zhang
From: hongbo.zhang hongbo.zh...@linaro.com This diver is based on the thermal management framework in thermal_sys.c. A thermal zone device is created with the trip points to which cooling devices can be bound, the current cooling device is cpufreq, e.g. CPU frequency is clipped down to cool

[PATCH V3 5/5] Thermal: Add ST-Ericsson DB8500 thermal properties and platform data.

2012-10-30 Thread hongbo.zhang
From: hongbo.zhang hongbo.zh...@linaro.com This patch adds device tree properties for ST-Ericsson DB8500 thermal driver, also adds the platform data to support the old fashion. Signed-off-by: hongbo.zhang hongbo.zh...@linaro.com --- arch/arm/boot/dts/dbx5x0.dtsi | 14 + arch/arm

[PATCH V3 3/5] Thermal: Remove the cooling_cpufreq_list.

2012-10-30 Thread hongbo.zhang
From: hongbo.zhang hongbo.zh...@linaro.com Problem of using this list is that the cpufreq_get_max_state callback will be called when register cooling device by thermal_cooling_device_register, but this list isn't ready at this moment. What's more, there is no need to maintain such a list, we can

[PATCH V2 4/6] Thermal: Remove the cooling_cpufreq_list.

2012-10-26 Thread hongbo.zhang
From: "hongbo.zhang" Problem of using this list is that the cpufreq_get_max_state callback will be called when register cooling device by thermal_cooling_device_register, but this list isn't ready at this moment. What's more, there is no need to maintain such a list,

[PATCH V2 4/6] Thermal: Remove the cooling_cpufreq_list.

2012-10-26 Thread hongbo.zhang
From: hongbo.zhang hongbo.zh...@linaro.com Problem of using this list is that the cpufreq_get_max_state callback will be called when register cooling device by thermal_cooling_device_register, but this list isn't ready at this moment. What's more, there is no need to maintain such a list, we can

[PATCH V2 6/6] Thermal: Add ST-Ericsson DB8500 thermal properties and platform data.

2012-10-25 Thread hongbo.zhang
From: "hongbo.zhang" This patch adds device tree properties for ST-Ericsson DB8500 thermal driver, also adds the platform data to support the old fashion. Signed-off-by: hongbo.zhang --- arch/arm/boot/dts/dbx5x0.dtsi | 14 + arch/arm/boot/dts/snowball.dt

[PATCH V2 6/6] Thermal: Add ST-Ericsson DB8500 thermal properties and platform data.

2012-10-25 Thread hongbo.zhang
From: "hongbo.zhang" This patch adds device tree properties for ST-Ericsson DB8500 thermal driver, also adds the platform data to support the old fashion. Signed-off-by: hongbo.zhang --- arch/arm/boot/dts/dbx5x0.dtsi | 14 + arch/arm/boot/dts/snowball.dt

[PATCH V2 5/6] Thermal: Add ST-Ericsson DB8500 thermal driver.

2012-10-25 Thread hongbo.zhang
From: "hongbo.zhang" This diver is based on the thermal management framework in thermal_sys.c. A thermal zone device is created with the trip points to which cooling devices can be bound, the current cooling device is cpufreq, e.g. CPU frequency is clipped down to cool the CPU, and oth

[PATCH V2 5/6] Thermal: Add ST-Ericsson DB8500 thermal driver.

2012-10-25 Thread hongbo.zhang
From: hongbo.zhang hongbo.zh...@linaro.com This diver is based on the thermal management framework in thermal_sys.c. A thermal zone device is created with the trip points to which cooling devices can be bound, the current cooling device is cpufreq, e.g. CPU frequency is clipped down to cool

[PATCH V2 6/6] Thermal: Add ST-Ericsson DB8500 thermal properties and platform data.

2012-10-25 Thread hongbo.zhang
From: hongbo.zhang hongbo.zh...@linaro.com This patch adds device tree properties for ST-Ericsson DB8500 thermal driver, also adds the platform data to support the old fashion. Signed-off-by: hongbo.zhang hongbo.zh...@linaro.com --- arch/arm/boot/dts/dbx5x0.dtsi | 14 + arch/arm

[PATCH V2 6/6] Thermal: Add ST-Ericsson DB8500 thermal properties and platform data.

2012-10-25 Thread hongbo.zhang
From: hongbo.zhang hongbo.zh...@linaro.com This patch adds device tree properties for ST-Ericsson DB8500 thermal driver, also adds the platform data to support the old fashion. Signed-off-by: hongbo.zhang hongbo.zh...@linaro.com --- arch/arm/boot/dts/dbx5x0.dtsi | 14 + arch/arm

[PATCH V2 3/6] Thermal: fix bug of counting cpu frequencies.

2012-10-24 Thread hongbo.zhang
From: "hongbo.zhang" In the while loop for counting cpu frequencies, if table[i].frequency equals CPUFREQ_ENTRY_INVALID, index i won't be increased, so this leads to an endless loop, what's more the index i cannot be referred as cpu frequencies number if there is CPUFREQ_ENTRY_IN

[PATCH V2 6/6] Thermal: Add ST-Ericsson DB8500 thermal properties and platform data.

2012-10-24 Thread hongbo.zhang
From: "hongbo.zhang" This patch adds device tree properties for ST-Ericsson DB8500 thermal driver, also adds the platform data to support the old fashion. Signed-off-by: hongbo.zhang --- .../devicetree/bindings/thermal/db8500-thermal.txt | 40 ++ arch/arm/boot/dts/d

[PATCH V2 5/6] Thermal: Add ST-Ericsson DB8500 thermal dirver.

2012-10-24 Thread hongbo.zhang
From: "hongbo.zhang" This diver is based on the thermal management framework in thermal_sys.c. A thermal zone device is created with the trip points to which cooling devices can be bound, the current cooling device is cpufreq, e.g. CPU frequency is clipped down to cool the CPU, and oth

[PATCH V2 4/6] Thermal: Remove the cooling_cpufreq_list.

2012-10-24 Thread hongbo.zhang
From: "hongbo.zhang" Problem of using this list is that the cpufreq_get_max_state callback will be called when register cooling device by thermal_cooling_device_register, but this list isn't ready at this moment. What's more, there is no need to maintain such a list,

[PATCH V2 2/6] Thermal: make sure cpufreq cooling register after cpufreq driver

2012-10-24 Thread hongbo.zhang
From: "hongbo.zhang" The cpufreq works as a cooling device, so the cooling layer should check if the cpufreq driver is initialized or not. Signed-off-by: hongbo.zhang --- drivers/thermal/cpu_cooling.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/thermal/cpu_c

[PATCH V2 1/6] Thermal: add indent for code alignment.

2012-10-24 Thread hongbo.zhang
From: "hongbo.zhang" The curly bracket should be aligned with corresponding if else statements. Signed-off-by: hongbo.zhang Reviewed-by: Viresh Kumar --- drivers/thermal/cpu_cooling.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/thermal/cpu_cooling.c

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

2012-10-24 Thread hongbo.zhang
From: "hongbo.zhang" V1->V2 Changes: DB8500 thermal dirver: Accept comments from Francesco Lavra and Viresh Kumar, and split platform and driver parts into separate patches. Thermal layer: Cancel the patch for deferring bind due to new patch for generic cpu cooling layer to f

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

2012-10-24 Thread hongbo.zhang
From: hongbo.zhang hongbo.zh...@linaro.com V1-V2 Changes: DB8500 thermal dirver: Accept comments from Francesco Lavra and Viresh Kumar, and split platform and driver parts into separate patches. Thermal layer: Cancel the patch for deferring bind due to new patch for generic cpu cooling layer

[PATCH V2 1/6] Thermal: add indent for code alignment.

2012-10-24 Thread hongbo.zhang
From: hongbo.zhang hongbo.zh...@linaro.com The curly bracket should be aligned with corresponding if else statements. Signed-off-by: hongbo.zhang hongbo.zh...@linaro.com Reviewed-by: Viresh Kumar viresh.ku...@linaro.org --- drivers/thermal/cpu_cooling.c | 2 +- 1 file changed, 1 insertion(+), 1

[PATCH V2 2/6] Thermal: make sure cpufreq cooling register after cpufreq driver

2012-10-24 Thread hongbo.zhang
From: hongbo.zhang hongbo.zh...@linaro.com The cpufreq works as a cooling device, so the cooling layer should check if the cpufreq driver is initialized or not. Signed-off-by: hongbo.zhang hongbo.zh...@linaro.com --- drivers/thermal/cpu_cooling.c | 4 1 file changed, 4 insertions(+) diff

[PATCH V2 4/6] Thermal: Remove the cooling_cpufreq_list.

2012-10-24 Thread hongbo.zhang
From: hongbo.zhang hongbo.zh...@linaro.com Problem of using this list is that the cpufreq_get_max_state callback will be called when register cooling device by thermal_cooling_device_register, but this list isn't ready at this moment. What's more, there is no need to maintain such a list, we can

[PATCH V2 5/6] Thermal: Add ST-Ericsson DB8500 thermal dirver.

2012-10-24 Thread hongbo.zhang
From: hongbo.zhang hongbo.zh...@linaro.com This diver is based on the thermal management framework in thermal_sys.c. A thermal zone device is created with the trip points to which cooling devices can be bound, the current cooling device is cpufreq, e.g. CPU frequency is clipped down to cool

[PATCH V2 6/6] Thermal: Add ST-Ericsson DB8500 thermal properties and platform data.

2012-10-24 Thread hongbo.zhang
From: hongbo.zhang hongbo.zh...@linaro.com This patch adds device tree properties for ST-Ericsson DB8500 thermal driver, also adds the platform data to support the old fashion. Signed-off-by: hongbo.zhang hongbo.zh...@linaro.com --- .../devicetree/bindings/thermal/db8500-thermal.txt | 40

[PATCH V2 3/6] Thermal: fix bug of counting cpu frequencies.

2012-10-24 Thread hongbo.zhang
From: hongbo.zhang hongbo.zh...@linaro.com In the while loop for counting cpu frequencies, if table[i].frequency equals CPUFREQ_ENTRY_INVALID, index i won't be increased, so this leads to an endless loop, what's more the index i cannot be referred as cpu frequencies number

[PATCH 5/5] Thermal: Add ST-Ericsson db8500 thermal dirver.

2012-10-16 Thread hongbo.zhang
From: "hongbo.zhang" This diver is based on the thermal management framework in thermal_sys.c. A thermal zone device is created with the trip points to which cooling devices can be bound, the current cooling device is cpufreq, e.g. CPU frequency is clipped down to cool the CPU, and oth

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

2012-10-16 Thread hongbo.zhang
From: "hongbo.zhang" 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 --- drivers/thermal/cpu_cooling.c | 12 +++- 1 file changed, 11 insertions(+), 1 deletion(-)

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

2012-10-16 Thread hongbo.zhang
From: "hongbo.zhang" 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 --- drivers/thermal/cpu_cooling.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drive

[PATCH 2/5] Thermal: add indent for code alignment.

2012-10-16 Thread hongbo.zhang
From: "hongbo.zhang" Signed-off-by: hongbo.zhang --- drivers/thermal/cpu_cooling.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/thermal/cpu_cooling.c b/drivers/thermal/cpu_cooling.c index cc1c930..b6b4c2a 100644 --- a/drivers/thermal/cpu_cooling.c +++

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

2012-10-16 Thread hongbo.zhang
From: "hongbo.zhang" In the previous bind function, cdev->get_max_state(cdev, _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 get_max_s

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

2012-10-16 Thread hongbo.zhang
From: "hongbo.zhang" 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 returns. In th

[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

[PATCH 2/5] Thermal: add indent for code alignment.

2012-10-16 Thread hongbo.zhang
From: hongbo.zhang hongbo.zh...@linaro.com Signed-off-by: hongbo.zhang hongbo.zh...@linaro.com --- drivers/thermal/cpu_cooling.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/thermal/cpu_cooling.c b/drivers/thermal/cpu_cooling.c index cc1c930..b6b4c2a 100644

[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 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

[PATCH 5/5] Thermal: Add ST-Ericsson db8500 thermal dirver.

2012-10-16 Thread hongbo.zhang
From: hongbo.zhang hongbo.zh...@linaro.com This diver is based on the thermal management framework in thermal_sys.c. A thermal zone device is created with the trip points to which cooling devices can be bound, the current cooling device is cpufreq, e.g. CPU frequency is clipped down to cool

<    1   2   3