* [PATCH] arm: dts: mt7623: increase passive cooling trip
@ 2021-07-25 16:34 ` Frank Wunderlich
0 siblings, 0 replies; 10+ messages in thread
From: Frank Wunderlich @ 2021-07-25 16:34 UTC (permalink / raw)
To: linux-mediatek
Cc: Frank Wunderlich, Rob Herring, Matthias Brugger, devicetree,
linux-arm-kernel, linux-kernel
From: Frank Wunderlich <frank-w@public-files.de>
MT7623/BPI-R2 has idle temperature after bootup from 48 degrees celsius
increase the passive trip temp threshold to not trottle CPU frequency at
this temperature
Signed-off-by: Frank Wunderlich <frank-w@public-files.de>
---
arch/arm/boot/dts/mt7623.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/mt7623.dtsi b/arch/arm/boot/dts/mt7623.dtsi
index b7247ab082aa..11933494e03d 100644
--- a/arch/arm/boot/dts/mt7623.dtsi
+++ b/arch/arm/boot/dts/mt7623.dtsi
@@ -160,7 +160,7 @@ cpu_thermal: cpu-thermal {
trips {
cpu_passive: cpu-passive {
- temperature = <47000>;
+ temperature = <57000>;
hysteresis = <2000>;
type = "passive";
};
--
2.25.1
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH] arm: dts: mt7623: increase passive cooling trip
@ 2021-07-25 16:34 ` Frank Wunderlich
0 siblings, 0 replies; 10+ messages in thread
From: Frank Wunderlich @ 2021-07-25 16:34 UTC (permalink / raw)
To: linux-mediatek
Cc: Frank Wunderlich, Rob Herring, Matthias Brugger, devicetree,
linux-arm-kernel, linux-kernel
From: Frank Wunderlich <frank-w@public-files.de>
MT7623/BPI-R2 has idle temperature after bootup from 48 degrees celsius
increase the passive trip temp threshold to not trottle CPU frequency at
this temperature
Signed-off-by: Frank Wunderlich <frank-w@public-files.de>
---
arch/arm/boot/dts/mt7623.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/mt7623.dtsi b/arch/arm/boot/dts/mt7623.dtsi
index b7247ab082aa..11933494e03d 100644
--- a/arch/arm/boot/dts/mt7623.dtsi
+++ b/arch/arm/boot/dts/mt7623.dtsi
@@ -160,7 +160,7 @@ cpu_thermal: cpu-thermal {
trips {
cpu_passive: cpu-passive {
- temperature = <47000>;
+ temperature = <57000>;
hysteresis = <2000>;
type = "passive";
};
--
2.25.1
_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH] arm: dts: mt7623: increase passive cooling trip
@ 2021-07-25 16:34 ` Frank Wunderlich
0 siblings, 0 replies; 10+ messages in thread
From: Frank Wunderlich @ 2021-07-25 16:34 UTC (permalink / raw)
To: linux-mediatek
Cc: Frank Wunderlich, Rob Herring, Matthias Brugger, devicetree,
linux-arm-kernel, linux-kernel
From: Frank Wunderlich <frank-w@public-files.de>
MT7623/BPI-R2 has idle temperature after bootup from 48 degrees celsius
increase the passive trip temp threshold to not trottle CPU frequency at
this temperature
Signed-off-by: Frank Wunderlich <frank-w@public-files.de>
---
arch/arm/boot/dts/mt7623.dtsi | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/boot/dts/mt7623.dtsi b/arch/arm/boot/dts/mt7623.dtsi
index b7247ab082aa..11933494e03d 100644
--- a/arch/arm/boot/dts/mt7623.dtsi
+++ b/arch/arm/boot/dts/mt7623.dtsi
@@ -160,7 +160,7 @@ cpu_thermal: cpu-thermal {
trips {
cpu_passive: cpu-passive {
- temperature = <47000>;
+ temperature = <57000>;
hysteresis = <2000>;
type = "passive";
};
--
2.25.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] arm: dts: mt7623: increase passive cooling trip
2021-07-25 16:34 ` Frank Wunderlich
(?)
@ 2021-08-04 17:15 ` Matthias Brugger
-1 siblings, 0 replies; 10+ messages in thread
From: Matthias Brugger @ 2021-08-04 17:15 UTC (permalink / raw)
To: Frank Wunderlich, linux-mediatek
Cc: Frank Wunderlich, Rob Herring, devicetree, linux-arm-kernel,
linux-kernel
On 25/07/2021 18:34, Frank Wunderlich wrote:
> From: Frank Wunderlich <frank-w@public-files.de>
>
> MT7623/BPI-R2 has idle temperature after bootup from 48 degrees celsius
> increase the passive trip temp threshold to not trottle CPU frequency at
> this temperature
>
> Signed-off-by: Frank Wunderlich <frank-w@public-files.de>
pushed to v5.14-next/dts32
Thanks!
> ---
> arch/arm/boot/dts/mt7623.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/mt7623.dtsi b/arch/arm/boot/dts/mt7623.dtsi
> index b7247ab082aa..11933494e03d 100644
> --- a/arch/arm/boot/dts/mt7623.dtsi
> +++ b/arch/arm/boot/dts/mt7623.dtsi
> @@ -160,7 +160,7 @@ cpu_thermal: cpu-thermal {
>
> trips {
> cpu_passive: cpu-passive {
> - temperature = <47000>;
> + temperature = <57000>;
> hysteresis = <2000>;
> type = "passive";
> };
>
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] arm: dts: mt7623: increase passive cooling trip
@ 2021-08-04 17:15 ` Matthias Brugger
0 siblings, 0 replies; 10+ messages in thread
From: Matthias Brugger @ 2021-08-04 17:15 UTC (permalink / raw)
To: Frank Wunderlich, linux-mediatek
Cc: Frank Wunderlich, Rob Herring, devicetree, linux-arm-kernel,
linux-kernel
On 25/07/2021 18:34, Frank Wunderlich wrote:
> From: Frank Wunderlich <frank-w@public-files.de>
>
> MT7623/BPI-R2 has idle temperature after bootup from 48 degrees celsius
> increase the passive trip temp threshold to not trottle CPU frequency at
> this temperature
>
> Signed-off-by: Frank Wunderlich <frank-w@public-files.de>
pushed to v5.14-next/dts32
Thanks!
> ---
> arch/arm/boot/dts/mt7623.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/mt7623.dtsi b/arch/arm/boot/dts/mt7623.dtsi
> index b7247ab082aa..11933494e03d 100644
> --- a/arch/arm/boot/dts/mt7623.dtsi
> +++ b/arch/arm/boot/dts/mt7623.dtsi
> @@ -160,7 +160,7 @@ cpu_thermal: cpu-thermal {
>
> trips {
> cpu_passive: cpu-passive {
> - temperature = <47000>;
> + temperature = <57000>;
> hysteresis = <2000>;
> type = "passive";
> };
>
_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] arm: dts: mt7623: increase passive cooling trip
@ 2021-08-04 17:15 ` Matthias Brugger
0 siblings, 0 replies; 10+ messages in thread
From: Matthias Brugger @ 2021-08-04 17:15 UTC (permalink / raw)
To: Frank Wunderlich, linux-mediatek
Cc: Frank Wunderlich, Rob Herring, devicetree, linux-arm-kernel,
linux-kernel
On 25/07/2021 18:34, Frank Wunderlich wrote:
> From: Frank Wunderlich <frank-w@public-files.de>
>
> MT7623/BPI-R2 has idle temperature after bootup from 48 degrees celsius
> increase the passive trip temp threshold to not trottle CPU frequency at
> this temperature
>
> Signed-off-by: Frank Wunderlich <frank-w@public-files.de>
pushed to v5.14-next/dts32
Thanks!
> ---
> arch/arm/boot/dts/mt7623.dtsi | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/mt7623.dtsi b/arch/arm/boot/dts/mt7623.dtsi
> index b7247ab082aa..11933494e03d 100644
> --- a/arch/arm/boot/dts/mt7623.dtsi
> +++ b/arch/arm/boot/dts/mt7623.dtsi
> @@ -160,7 +160,7 @@ cpu_thermal: cpu-thermal {
>
> trips {
> cpu_passive: cpu-passive {
> - temperature = <47000>;
> + temperature = <57000>;
> hysteresis = <2000>;
> type = "passive";
> };
>
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] arm: dts: mt7623: increase passive cooling trip
2021-07-25 16:34 ` Frank Wunderlich
@ 2022-03-05 15:54 ` Kurt Fitzner
-1 siblings, 0 replies; 10+ messages in thread
From: Kurt Fitzner @ 2022-03-05 15:54 UTC (permalink / raw)
To: Frank Wunderlich, linux-mediatek
Cc: Matthias Brugger, Rob Herring, linux-arm-kernel
This issue and patch just came to my attention, my apologies for late
comments.
The solution provided in this patch is, I believe, not optimal.
On 2021-07-25 1:34 p.m., Frank Wunderlich wrote:
> From: Frank Wunderlich <frank-w@public-files.de>
>
> MT7623/BPI-R2 has idle temperature after bootup from 48 degrees celsius
> increase the passive trip temp threshold to not trottle CPU frequency at
> this temperature
The dts file for the mt7623 has four trip points:
- Trip point 0, Type=Passive @ 47°C (originally, now 57°C)
- Trip point 1, Type=Active @ 67°C
- Trip point 2, Type=Hot @ 87°C
- Trip point 3, Type=Critical @ 107°C
It makes little sense to have passive CPU throttling employed *before*
engaging a fan. Generally users want a fan (if present) to be engaged
before intentional performance degradation is employed.
> cpu_passive: cpu-passive {
> - temperature = <47000>;
> + temperature = <57000>;
The old temperature of 47°C appears, in practice, to be a good
temperature for an active cooling trip point. It is just above the
temperature that a generally idle mt7623 sits at in enclosures where
this SoC is employed, but also a temperature that a CPU under load will
quickly reach.
Therefore I propose the types of trip points 0 and 1 be reversed and the
temperatures left as they were.
--- a/arch/arm/boot/dts/mt7623.dtsi 2022-02-27 18:36:33 -0400
+++ b/arch/arm/boot/dts/mt7623.dtsi 2022-03-05 11:37:52 -0400
@@ -158,19 +158,19 @@
thermal-sensors = <&thermal 0>;
trips {
cpu_passive: cpu-passive {
- temperature = <57000>;
+ temperature = <47000>;
hysteresis = <2000>;
- type = "passive";
+ type = "active";
};
cpu_active: cpu-active {
temperature = <67000>;
hysteresis = <2000>;
- type = "active";
+ type = "passive";
};
cpu_hot: cpu-hot {
temperature = <87000>;
hysteresis = <2000>;
_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] arm: dts: mt7623: increase passive cooling trip
@ 2022-03-05 15:54 ` Kurt Fitzner
0 siblings, 0 replies; 10+ messages in thread
From: Kurt Fitzner @ 2022-03-05 15:54 UTC (permalink / raw)
To: Frank Wunderlich, linux-mediatek
Cc: Frank Wunderlich, Rob Herring, Matthias Brugger, linux-arm-kernel
This issue and patch just came to my attention, my apologies for late
comments.
The solution provided in this patch is, I believe, not optimal.
On 2021-07-25 1:34 p.m., Frank Wunderlich wrote:
> From: Frank Wunderlich <frank-w@public-files.de>
>
> MT7623/BPI-R2 has idle temperature after bootup from 48 degrees celsius
> increase the passive trip temp threshold to not trottle CPU frequency at
> this temperature
The dts file for the mt7623 has four trip points:
- Trip point 0, Type=Passive @ 47°C (originally, now 57°C)
- Trip point 1, Type=Active @ 67°C
- Trip point 2, Type=Hot @ 87°C
- Trip point 3, Type=Critical @ 107°C
It makes little sense to have passive CPU throttling employed *before*
engaging a fan. Generally users want a fan (if present) to be engaged
before intentional performance degradation is employed.
> cpu_passive: cpu-passive {
> - temperature = <47000>;
> + temperature = <57000>;
The old temperature of 47°C appears, in practice, to be a good
temperature for an active cooling trip point. It is just above the
temperature that a generally idle mt7623 sits at in enclosures where
this SoC is employed, but also a temperature that a CPU under load will
quickly reach.
Therefore I propose the types of trip points 0 and 1 be reversed and the
temperatures left as they were.
--- a/arch/arm/boot/dts/mt7623.dtsi 2022-02-27 18:36:33 -0400
+++ b/arch/arm/boot/dts/mt7623.dtsi 2022-03-05 11:37:52 -0400
@@ -158,19 +158,19 @@
thermal-sensors = <&thermal 0>;
trips {
cpu_passive: cpu-passive {
- temperature = <57000>;
+ temperature = <47000>;
hysteresis = <2000>;
- type = "passive";
+ type = "active";
};
cpu_active: cpu-active {
temperature = <67000>;
hysteresis = <2000>;
- type = "active";
+ type = "passive";
};
cpu_hot: cpu-hot {
temperature = <87000>;
hysteresis = <2000>;
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Aw: Re: [PATCH] arm: dts: mt7623: increase passive cooling trip
2022-03-05 15:54 ` Kurt Fitzner
@ 2022-03-05 16:37 ` Frank Wunderlich
-1 siblings, 0 replies; 10+ messages in thread
From: Frank Wunderlich @ 2022-03-05 16:37 UTC (permalink / raw)
To: Kurt Fitzner
Cc: Frank Wunderlich, linux-mediatek, Rob Herring, Matthias Brugger,
linux-arm-kernel
Hi Kurt,
> Gesendet: Samstag, 05. März 2022 um 16:54 Uhr
> Von: "Kurt Fitzner" <kurt@va1der.ca>
> This issue and patch just came to my attention, my apologies for late
> comments.
>
> The solution provided in this patch is, I believe, not optimal.
does your Patch fixes the issue? if yes you can send a Patch based on current mainline dts.
it looks a bit weired to me having active before passive...but i'm no expert in the cooling trips.
i just increased the temps :)
please see notes below
> On 2021-07-25 1:34 p.m., Frank Wunderlich wrote:
> > From: Frank Wunderlich <frank-w@public-files.de>
> >
> > MT7623/BPI-R2 has idle temperature after bootup from 48 degrees celsius
> > increase the passive trip temp threshold to not trottle CPU frequency at
> > this temperature
>
> The dts file for the mt7623 has four trip points:
> - Trip point 0, Type=Passive @ 47°C (originally, now 57°C)
> - Trip point 1, Type=Active @ 67°C
> - Trip point 2, Type=Hot @ 87°C
> - Trip point 3, Type=Critical @ 107°C
>
> It makes little sense to have passive CPU throttling employed *before*
> engaging a fan. Generally users want a fan (if present) to be engaged
> before intentional performance degradation is employed.
>
> > cpu_passive: cpu-passive {
> > - temperature = <47000>;
> > + temperature = <57000>;
>
> The old temperature of 47°C appears, in practice, to be a good
> temperature for an active cooling trip point. It is just above the
> temperature that a generally idle mt7623 sits at in enclosures where
> this SoC is employed, but also a temperature that a CPU under load will
> quickly reach.
>
> Therefore I propose the types of trip points 0 and 1 be reversed and the
> temperatures left as they were.
>
>
> --- a/arch/arm/boot/dts/mt7623.dtsi 2022-02-27 18:36:33 -0400
> +++ b/arch/arm/boot/dts/mt7623.dtsi 2022-03-05 11:37:52 -0400
> @@ -158,19 +158,19 @@
>
> thermal-sensors = <&thermal 0>;
>
> trips {
> cpu_passive: cpu-passive {
> - temperature = <57000>;
> + temperature = <47000>;
> hysteresis = <2000>;
> - type = "passive";
> + type = "active";
> };
if type is different, imho the name+label should be changed too
> cpu_active: cpu-active {
> temperature = <67000>;
> hysteresis = <2000>;
> - type = "active";
> + type = "passive";
> };
dito
> cpu_hot: cpu-hot {
> temperature = <87000>;
> hysteresis = <2000>;
regards Frank
_______________________________________________
Linux-mediatek mailing list
Linux-mediatek@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-mediatek
^ permalink raw reply [flat|nested] 10+ messages in thread
* Aw: Re: [PATCH] arm: dts: mt7623: increase passive cooling trip
@ 2022-03-05 16:37 ` Frank Wunderlich
0 siblings, 0 replies; 10+ messages in thread
From: Frank Wunderlich @ 2022-03-05 16:37 UTC (permalink / raw)
To: Kurt Fitzner
Cc: Frank Wunderlich, linux-mediatek, Rob Herring, Matthias Brugger,
linux-arm-kernel
Hi Kurt,
> Gesendet: Samstag, 05. März 2022 um 16:54 Uhr
> Von: "Kurt Fitzner" <kurt@va1der.ca>
> This issue and patch just came to my attention, my apologies for late
> comments.
>
> The solution provided in this patch is, I believe, not optimal.
does your Patch fixes the issue? if yes you can send a Patch based on current mainline dts.
it looks a bit weired to me having active before passive...but i'm no expert in the cooling trips.
i just increased the temps :)
please see notes below
> On 2021-07-25 1:34 p.m., Frank Wunderlich wrote:
> > From: Frank Wunderlich <frank-w@public-files.de>
> >
> > MT7623/BPI-R2 has idle temperature after bootup from 48 degrees celsius
> > increase the passive trip temp threshold to not trottle CPU frequency at
> > this temperature
>
> The dts file for the mt7623 has four trip points:
> - Trip point 0, Type=Passive @ 47°C (originally, now 57°C)
> - Trip point 1, Type=Active @ 67°C
> - Trip point 2, Type=Hot @ 87°C
> - Trip point 3, Type=Critical @ 107°C
>
> It makes little sense to have passive CPU throttling employed *before*
> engaging a fan. Generally users want a fan (if present) to be engaged
> before intentional performance degradation is employed.
>
> > cpu_passive: cpu-passive {
> > - temperature = <47000>;
> > + temperature = <57000>;
>
> The old temperature of 47°C appears, in practice, to be a good
> temperature for an active cooling trip point. It is just above the
> temperature that a generally idle mt7623 sits at in enclosures where
> this SoC is employed, but also a temperature that a CPU under load will
> quickly reach.
>
> Therefore I propose the types of trip points 0 and 1 be reversed and the
> temperatures left as they were.
>
>
> --- a/arch/arm/boot/dts/mt7623.dtsi 2022-02-27 18:36:33 -0400
> +++ b/arch/arm/boot/dts/mt7623.dtsi 2022-03-05 11:37:52 -0400
> @@ -158,19 +158,19 @@
>
> thermal-sensors = <&thermal 0>;
>
> trips {
> cpu_passive: cpu-passive {
> - temperature = <57000>;
> + temperature = <47000>;
> hysteresis = <2000>;
> - type = "passive";
> + type = "active";
> };
if type is different, imho the name+label should be changed too
> cpu_active: cpu-active {
> temperature = <67000>;
> hysteresis = <2000>;
> - type = "active";
> + type = "passive";
> };
dito
> cpu_hot: cpu-hot {
> temperature = <87000>;
> hysteresis = <2000>;
regards Frank
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2022-03-05 16:39 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-25 16:34 [PATCH] arm: dts: mt7623: increase passive cooling trip Frank Wunderlich
2021-07-25 16:34 ` Frank Wunderlich
2021-07-25 16:34 ` Frank Wunderlich
2021-08-04 17:15 ` Matthias Brugger
2021-08-04 17:15 ` Matthias Brugger
2021-08-04 17:15 ` Matthias Brugger
2022-03-05 15:54 ` Kurt Fitzner
2022-03-05 15:54 ` Kurt Fitzner
2022-03-05 16:37 ` Aw: " Frank Wunderlich
2022-03-05 16:37 ` Frank Wunderlich
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.