diff mbox series

[05/10] ARM64: dts: hi6220: Remove "cooling-{min|max}-level" for CPU nodes

Message ID 68740229a7397336ae92bad65f69a58c91819c37.1518166039.git.viresh.kumar@linaro.org
State Accepted
Commit 183879d8c6a6b4d8904330b21de4f27fb121179a
Headers show
Series None | expand

Commit Message

Viresh Kumar Feb. 9, 2018, 8:58 a.m. UTC
The "cooling-min-level" and "cooling-max-level" properties are not
parsed by any part of the kernel currently and the max cooling state of
a CPU cooling device is found by referring to the cpufreq table instead.

Moreover, the entries are incorrect here as min level is 4 and the max
level is 0.

Remove the unused properties from the CPU nodes.

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>

---
 arch/arm64/boot/dts/hisilicon/hi6220.dtsi | 2 --
 1 file changed, 2 deletions(-)

-- 
2.15.0.194.g9af6a3dea062

Comments

Wei Xu March 2, 2018, 4:03 p.m. UTC | #1
Hi Viresh,

On 2018/2/9 8:58, Viresh Kumar wrote:
> The "cooling-min-level" and "cooling-max-level" properties are not

> parsed by any part of the kernel currently and the max cooling state of

> a CPU cooling device is found by referring to the cpufreq table instead.

> 

> Moreover, the entries are incorrect here as min level is 4 and the max

> level is 0.

> 

> Remove the unused properties from the CPU nodes.

> 

> Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>

> ---


Applied into hisilicon dt tree.
Thanks!

BR,
Wei

>  arch/arm64/boot/dts/hisilicon/hi6220.dtsi | 2 --

>  1 file changed, 2 deletions(-)

> 

> diff --git a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi

> index 6a180d1926e8..fca8e4ee98e7 100644

> --- a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi

> +++ b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi

> @@ -88,8 +88,6 @@

>  			next-level-cache = <&CLUSTER0_L2>;

>  			clocks = <&stub_clock 0>;

>  			operating-points-v2 = <&cpu_opp_table>;

> -			cooling-min-level = <4>;

> -			cooling-max-level = <0>;

>  			#cooling-cells = <2>; /* min followed by max */

>  			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP>;

>  			dynamic-power-coefficient = <311>;

>
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi
index 6a180d1926e8..fca8e4ee98e7 100644
--- a/arch/arm64/boot/dts/hisilicon/hi6220.dtsi
+++ b/arch/arm64/boot/dts/hisilicon/hi6220.dtsi
@@ -88,8 +88,6 @@ 
 			next-level-cache = <&CLUSTER0_L2>;
 			clocks = <&stub_clock 0>;
 			operating-points-v2 = <&cpu_opp_table>;
-			cooling-min-level = <4>;
-			cooling-max-level = <0>;
 			#cooling-cells = <2>; /* min followed by max */
 			cpu-idle-states = <&CPU_SLEEP &CLUSTER_SLEEP>;
 			dynamic-power-coefficient = <311>;