* [PATCH 00/10] thermal: Remove "cooling-{min|max}-level" properties
@ 2018-02-09 8:58 Viresh Kumar
2018-02-09 8:58 ` [PATCH 06/10] ARM64: dts: meson: Remove "cooling-{min|max}-level" for CPU nodes Viresh Kumar
2018-02-09 8:58 ` [PATCH 08/10] ARM64: dts: meson: Remove "cooling-{min|max}-level" for gpio-fan node Viresh Kumar
0 siblings, 2 replies; 8+ messages in thread
From: Viresh Kumar @ 2018-02-09 8:58 UTC (permalink / raw)
To: linus-amlogic
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 cooling devices is found by cooling-device's driver itself.
Also note that a very small number of platforms that support cooling
devices currently contain these properties in their dts files. Most of
them don't specify them (as no one is going to use them anyway).
Only two type of cooling devices are using these properties currently:
cpu-cooling and gpio-fan. This series first updates all the dts files to
not specify the min/max cooling states and then removes the bindings.
The maintainers are requested to apply the relevant patches to their
trees directly as there is no dependency between the patches.
--
viresh
Viresh Kumar (10):
ARM: dts: exynos: Remove "cooling-{min|max}-level" for CPU nodes
ARM: dts: omap: Remove "cooling-{min|max}-level" for CPU nodes
ARM: dts: mt7623: Remove "cooling-{min|max}-level" for CPU nodes
ARM: dts: sun[4-7]i: Remove "cooling-{min|max}-level" for CPU nodes
ARM64: dts: hi6220: Remove "cooling-{min|max}-level" for CPU nodes
ARM64: dts: meson: Remove "cooling-{min|max}-level" for CPU nodes
ARM: dts: gemini: Remove "cooling-{min|max}-level" for gpio-fan node
ARM64: dts: meson: Remove "cooling-{min|max}-level" for gpio-fan node
dt-bindings: cpufreq-dt: Remove "cooling-{min|max}-level" properties
dt-bindings: thermal: Remove "cooling-{min|max}-level" properties
Documentation/devicetree/bindings/cpufreq/cpufreq-dt.txt | 4 ----
.../devicetree/bindings/cpufreq/cpufreq-mediatek.txt | 4 ----
Documentation/devicetree/bindings/thermal/thermal.txt | 16 +---------------
arch/arm/boot/dts/dra7.dtsi | 2 --
arch/arm/boot/dts/exynos4210.dtsi | 2 --
arch/arm/boot/dts/exynos4412.dtsi | 2 --
arch/arm/boot/dts/exynos5250.dtsi | 2 --
arch/arm/boot/dts/exynos5420-cpus.dtsi | 16 ----------------
arch/arm/boot/dts/exynos5422-cpus.dtsi | 16 ----------------
arch/arm/boot/dts/gemini-dlink-dns-313.dts | 2 --
arch/arm/boot/dts/mt7623.dtsi | 2 --
arch/arm/boot/dts/omap443x.dtsi | 2 --
arch/arm/boot/dts/omap4460.dtsi | 2 --
arch/arm/boot/dts/omap5.dtsi | 2 --
arch/arm/boot/dts/sun4i-a10-olinuxino-lime.dts | 1 -
arch/arm/boot/dts/sun4i-a10.dtsi | 2 --
arch/arm/boot/dts/sun5i-a13.dtsi | 2 --
arch/arm/boot/dts/sun6i-a31.dtsi | 2 --
arch/arm/boot/dts/sun7i-a20.dtsi | 2 --
arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts | 6 ------
arch/arm64/boot/dts/hisilicon/hi6220.dtsi | 2 --
21 files changed, 1 insertion(+), 90 deletions(-)
--
2.15.0.194.g9af6a3dea062
^ permalink raw reply [flat|nested] 8+ messages in thread* [PATCH 06/10] ARM64: dts: meson: Remove "cooling-{min|max}-level" for CPU nodes
2018-02-09 8:58 [PATCH 00/10] thermal: Remove "cooling-{min|max}-level" properties Viresh Kumar
@ 2018-02-09 8:58 ` Viresh Kumar
2018-02-09 9:03 ` Neil Armstrong
2018-03-20 1:37 ` Viresh Kumar
2018-02-09 8:58 ` [PATCH 08/10] ARM64: dts: meson: Remove "cooling-{min|max}-level" for gpio-fan node Viresh Kumar
1 sibling, 2 replies; 8+ messages in thread
From: Viresh Kumar @ 2018-02-09 8:58 UTC (permalink / raw)
To: linus-amlogic
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>;
};
--
2.15.0.194.g9af6a3dea062
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH 06/10] ARM64: dts: meson: Remove "cooling-{min|max}-level" for CPU nodes
2018-02-09 8:58 ` [PATCH 06/10] ARM64: dts: meson: Remove "cooling-{min|max}-level" for CPU nodes Viresh Kumar
@ 2018-02-09 9:03 ` Neil Armstrong
2018-03-12 4:32 ` Viresh Kumar
2018-03-20 1:37 ` Viresh Kumar
1 sibling, 1 reply; 8+ messages in thread
From: Neil Armstrong @ 2018-02-09 9:03 UTC (permalink / raw)
To: linus-amlogic
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>
^ permalink raw reply [flat|nested] 8+ messages in thread* [PATCH 06/10] ARM64: dts: meson: Remove "cooling-{min|max}-level" for CPU nodes
2018-02-09 9:03 ` Neil Armstrong
@ 2018-03-12 4:32 ` Viresh Kumar
0 siblings, 0 replies; 8+ messages in thread
From: Viresh Kumar @ 2018-03-12 4:32 UTC (permalink / raw)
To: linus-amlogic
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
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 06/10] ARM64: dts: meson: Remove "cooling-{min|max}-level" for CPU nodes
2018-02-09 8:58 ` [PATCH 06/10] ARM64: dts: meson: Remove "cooling-{min|max}-level" for CPU nodes Viresh Kumar
2018-02-09 9:03 ` Neil Armstrong
@ 2018-03-20 1:37 ` Viresh Kumar
2018-03-27 12:29 ` Arnd Bergmann
1 sibling, 1 reply; 8+ messages in thread
From: Viresh Kumar @ 2018-03-20 1:37 UTC (permalink / raw)
To: linus-amlogic
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 at kernel.org. Thanks.
--
viresh
^ permalink raw reply [flat|nested] 8+ messages in thread* [PATCH 06/10] ARM64: dts: meson: Remove "cooling-{min|max}-level" for CPU nodes
2018-03-20 1:37 ` Viresh Kumar
@ 2018-03-27 12:29 ` Arnd Bergmann
0 siblings, 0 replies; 8+ messages in thread
From: Arnd Bergmann @ 2018-03-27 12:29 UTC (permalink / raw)
To: linus-amlogic
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 at kernel.org. Thanks.
Picked up patches 6, 7 and 8 now. For some reason I didn't see patch 8
in my arm at kernel.org inbox, but I found it on the mailing list.
Arnd
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH 08/10] ARM64: dts: meson: Remove "cooling-{min|max}-level" for gpio-fan node
2018-02-09 8:58 [PATCH 00/10] thermal: Remove "cooling-{min|max}-level" properties Viresh Kumar
2018-02-09 8:58 ` [PATCH 06/10] ARM64: dts: meson: Remove "cooling-{min|max}-level" for CPU nodes Viresh Kumar
@ 2018-02-09 8:58 ` Viresh Kumar
2018-02-09 9:04 ` Neil Armstrong
1 sibling, 1 reply; 8+ messages in thread
From: Viresh Kumar @ 2018-02-09 8:58 UTC (permalink / raw)
To: linus-amlogic
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
gpio-fan cooling device is found by referring to the
"gpio-fan,speed-map" instead.
Remove the unused properties from the gpio-fan node.
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
---
arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts | 2 --
1 file changed, 2 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 658215c9822c..8ee4a693c438 100644
--- a/arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts
+++ b/arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts
@@ -59,8 +59,6 @@
1 1
2 2
3 3>;
- cooling-min-level = <0>;
- cooling-max-level = <3>;
#cooling-cells = <2>;
};
--
2.15.0.194.g9af6a3dea062
^ permalink raw reply related [flat|nested] 8+ messages in thread* [PATCH 08/10] ARM64: dts: meson: Remove "cooling-{min|max}-level" for gpio-fan node
2018-02-09 8:58 ` [PATCH 08/10] ARM64: dts: meson: Remove "cooling-{min|max}-level" for gpio-fan node Viresh Kumar
@ 2018-02-09 9:04 ` Neil Armstrong
0 siblings, 0 replies; 8+ messages in thread
From: Neil Armstrong @ 2018-02-09 9:04 UTC (permalink / raw)
To: linus-amlogic
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
> gpio-fan cooling device is found by referring to the
> "gpio-fan,speed-map" instead.
>
> Remove the unused properties from the gpio-fan node.
>
> Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
> ---
> arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts | 2 --
> 1 file changed, 2 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 658215c9822c..8ee4a693c438 100644
> --- a/arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts
> +++ b/arch/arm64/boot/dts/amlogic/meson-gxm-khadas-vim2.dts
> @@ -59,8 +59,6 @@
> 1 1
> 2 2
> 3 3>;
> - cooling-min-level = <0>;
> - cooling-max-level = <3>;
> #cooling-cells = <2>;
> };
>
>
Acked-by: Neil Armstrong <narmstrong@baylibre.com>
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2018-03-27 12:29 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-02-09 8:58 [PATCH 00/10] thermal: Remove "cooling-{min|max}-level" properties Viresh Kumar
2018-02-09 8:58 ` [PATCH 06/10] ARM64: dts: meson: Remove "cooling-{min|max}-level" for CPU nodes Viresh Kumar
2018-02-09 9:03 ` Neil Armstrong
2018-03-12 4:32 ` Viresh Kumar
2018-03-20 1:37 ` Viresh Kumar
2018-03-27 12:29 ` Arnd Bergmann
2018-02-09 8:58 ` [PATCH 08/10] ARM64: dts: meson: Remove "cooling-{min|max}-level" for gpio-fan node Viresh Kumar
2018-02-09 9:04 ` Neil Armstrong
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).