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

Message ID 6c91ebfecf86da98a7682c6d262ff1218a1d7343.1518166039.git.viresh.kumar@linaro.org
State New
Headers show
Series
  • Untitled series #8799
Related show

Commit Message

Viresh Kumar Feb. 9, 2018, 8:58 a.m.
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.

Remove the unused properties from the CPU nodes.

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

---
 arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts | 4 ----
 1 file changed, 4 deletions(-)

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

Comments

Viresh Kumar March 12, 2018, 4:32 a.m. | #1
On 09-02-18, 10:03, Neil Armstrong wrote:
> On 09/02/2018 09: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.

> > 

> > Remove the unused properties from the CPU nodes.

> > 

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

> > ---

> >  arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts | 4 ----

> >  1 file changed, 4 deletions(-)

> > 

> > diff --git a/arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts b/arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts

> > index 1448c3dba08e..658215c9822c 100644

> > --- a/arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts

> > +++ b/arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts

> > @@ -209,14 +209,10 @@

> >  };

> >  

> >  &cpu0 {

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

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

> >  	#cooling-cells = <2>;

> >  };

> >  

> >  &cpu4 {

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

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

> >  	#cooling-cells = <2>;

> >  };

> >  

> > 

> 

> Acked-by: Neil Armstrong <narmstrong@baylibre.com>


Is Kevin going to apply these for 4.17 ?

-- 
viresh
--
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
Arnd Bergmann March 27, 2018, 12:29 p.m. | #2
On Tue, Mar 20, 2018 at 2:37 AM, Viresh Kumar <viresh.kumar@linaro.org> wrote:
> On 09-02-18, 14:28, 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.

>>

>> Remove the unused properties from the CPU nodes.

>>

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

>> ---

>>  arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts | 4 ----

>>  1 file changed, 4 deletions(-)

>

> @Arnd: Can you please pick this and patch 8/10 from this series for 4.17 ? Both

> are already Acked by Neil. I have just re-bounced both the messages to

> arm@kernel.org. Thanks.


Picked up patches 6, 7 and 8 now. For some reason I didn't see patch 8
in my arm@kernel.org inbox, but I found it on the mailing list.

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

Patch

diff --git a/arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts b/arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts
index 1448c3dba08e..658215c9822c 100644
--- a/arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts
+++ b/arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts
@@ -209,14 +209,10 @@ 
 };
 
 &cpu0 {
-	cooling-min-level = <0>;
-	cooling-max-level = <6>;
 	#cooling-cells = <2>;
 };
 
 &cpu4 {
-	cooling-min-level = <0>;
-	cooling-max-level = <4>;
 	#cooling-cells = <2>;
 };