| Message ID | 51ac4a5f0466aeed3f223663d9c34d6345b2a1f2.1527244201.git.viresh.kumar@linaro.org |
|---|---|
| State | Accepted |
| Commit | 38dc27c85e642cd6229f3a4b2c85128501852158 |
| Headers | show |
| Series | [01/15] arm: dts: armada: Fix "#cooling-cells" property's name | expand |
2018-05-25 19:31 GMT+09:00 Viresh Kumar <viresh.kumar@linaro.org>: > The cooling device properties, like "#cooling-cells" and > "dynamic-power-coefficient", should either be present for all the CPUs > of a cluster or none. If these are present only for a subset of CPUs of > a cluster then things will start falling apart as soon as the CPUs are > brought online in a different order. For example, this will happen > because the operating system looks for such properties in the CPU node > it is trying to bring up, so that it can register a cooling device. > > Add such missing properties. > > Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> Applied to linux-uniphier. I had already sent a PR for v4.18-rc1 before I received this patch. Please wait for v4.19-rc1. Thanks. > --- > arch/arm/boot/dts/uniphier-pxs2.dtsi | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/arch/arm/boot/dts/uniphier-pxs2.dtsi b/arch/arm/boot/dts/uniphier-pxs2.dtsi > index debcbd15c24b..40ed15465095 100644 > --- a/arch/arm/boot/dts/uniphier-pxs2.dtsi > +++ b/arch/arm/boot/dts/uniphier-pxs2.dtsi > @@ -36,6 +36,7 @@ > enable-method = "psci"; > next-level-cache = <&l2>; > operating-points-v2 = <&cpu_opp>; > + #cooling-cells = <2>; > }; > > cpu2: cpu@2 { > @@ -46,6 +47,7 @@ > enable-method = "psci"; > next-level-cache = <&l2>; > operating-points-v2 = <&cpu_opp>; > + #cooling-cells = <2>; > }; > > cpu3: cpu@3 { > @@ -56,6 +58,7 @@ > enable-method = "psci"; > next-level-cache = <&l2>; > operating-points-v2 = <&cpu_opp>; > + #cooling-cells = <2>; > }; > }; > > -- > 2.15.0.194.g9af6a3dea062 > > -- > To unsubscribe from this list: send the line "unsubscribe devicetree" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- Best Regards Masahiro Yamada -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/arch/arm/boot/dts/uniphier-pxs2.dtsi b/arch/arm/boot/dts/uniphier-pxs2.dtsi index debcbd15c24b..40ed15465095 100644 --- a/arch/arm/boot/dts/uniphier-pxs2.dtsi +++ b/arch/arm/boot/dts/uniphier-pxs2.dtsi @@ -36,6 +36,7 @@ enable-method = "psci"; next-level-cache = <&l2>; operating-points-v2 = <&cpu_opp>; + #cooling-cells = <2>; }; cpu2: cpu@2 { @@ -46,6 +47,7 @@ enable-method = "psci"; next-level-cache = <&l2>; operating-points-v2 = <&cpu_opp>; + #cooling-cells = <2>; }; cpu3: cpu@3 { @@ -56,6 +58,7 @@ enable-method = "psci"; next-level-cache = <&l2>; operating-points-v2 = <&cpu_opp>; + #cooling-cells = <2>; }; };
The cooling device properties, like "#cooling-cells" and "dynamic-power-coefficient", should either be present for all the CPUs of a cluster or none. If these are present only for a subset of CPUs of a cluster then things will start falling apart as soon as the CPUs are brought online in a different order. For example, this will happen because the operating system looks for such properties in the CPU node it is trying to bring up, so that it can register a cooling device. Add such missing properties. Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> --- arch/arm/boot/dts/uniphier-pxs2.dtsi | 3 +++ 1 file changed, 3 insertions(+) -- 2.15.0.194.g9af6a3dea062 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html