* [PATCH 1/4] arm64: defconfig: Enable devfreq cooling device
[not found] <20220818203928.131059-1-cpe@outsight.tech>
@ 2022-08-18 20:39 ` Clément Péron
2022-08-18 20:39 ` [PATCH 2/4] arm64: dts: allwinner: h6: Add cooling map for GPU Clément Péron
` (2 subsequent siblings)
3 siblings, 0 replies; 6+ messages in thread
From: Clément Péron @ 2022-08-18 20:39 UTC (permalink / raw)
To: Catalin Marinas, Will Deacon
Cc: Clément Péron, linux-arm-kernel, linux-kernel
From: Clément Péron <peron.clem@gmail.com>
Devfreq cooling device framework is used in Panfrost
to throttle GPU in order to regulate its temperature.
Enable this driver for ARM64 SoC.
Signed-off-by: Clément Péron <peron.clem@gmail.com>
---
arch/arm64/configs/defconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index d5b2d2dd4904..109004e44d21 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -584,6 +584,7 @@ CONFIG_SENSORS_INA2XX=m
CONFIG_SENSORS_INA3221=m
CONFIG_THERMAL_GOV_POWER_ALLOCATOR=y
CONFIG_CPU_THERMAL=y
+CONFIG_DEVFREQ_THERMAL=y
CONFIG_THERMAL_EMULATION=y
CONFIG_IMX_SC_THERMAL=m
CONFIG_IMX8MM_THERMAL=m
--
2.34.1
^ permalink raw reply [flat|nested] 6+ messages in thread* [PATCH 2/4] arm64: dts: allwinner: h6: Add cooling map for GPU
[not found] <20220818203928.131059-1-cpe@outsight.tech>
2022-08-18 20:39 ` [PATCH 1/4] arm64: defconfig: Enable devfreq cooling device Clément Péron
@ 2022-08-18 20:39 ` Clément Péron
2022-08-19 4:13 ` Jernej Škrabec
2022-08-18 20:39 ` [PATCH 3/4] arm64: dts: allwinner: h6: Add GPU OPP table Clément Péron
2022-08-18 20:39 ` [PATCH 4/4] arm64: dts: allwinner: beelink-gs1: Enable GPU OPP Clément Péron
3 siblings, 1 reply; 6+ messages in thread
From: Clément Péron @ 2022-08-18 20:39 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland
Cc: Clément Péron, devicetree, linux-arm-kernel,
linux-sunxi, linux-kernel
From: Clément Péron <peron.clem@gmail.com>
Add a simple cooling map for the GPU.
Signed-off-by: Clément Péron <peron.clem@gmail.com>
---
arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi | 22 ++++++++++++++++++++
1 file changed, 22 insertions(+)
diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi b/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi
index 5a28303d3d4c..943ae5374dd6 100644
--- a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi
+++ b/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi
@@ -186,6 +186,7 @@ gpu: gpu@1800000 {
clocks = <&ccu CLK_GPU>, <&ccu CLK_BUS_GPU>;
clock-names = "core", "bus";
resets = <&ccu RST_BUS_GPU>;
+ #cooling-cells = <2>;
status = "disabled";
};
@@ -1075,6 +1076,27 @@ gpu-thermal {
polling-delay-passive = <0>;
polling-delay = <0>;
thermal-sensors = <&ths 1>;
+
+ trips {
+ gpu_alert: gpu-alert {
+ temperature = <85000>;
+ hysteresis = <2000>;
+ type = "passive";
+ };
+
+ gpu-crit {
+ temperature = <100000>;
+ hysteresis = <0>;
+ type = "critical";
+ };
+ };
+
+ cooling-maps {
+ map0 {
+ trip = <&gpu_alert>;
+ cooling-device = <&gpu THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
+ };
+ };
};
};
};
--
2.34.1
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH 2/4] arm64: dts: allwinner: h6: Add cooling map for GPU
2022-08-18 20:39 ` [PATCH 2/4] arm64: dts: allwinner: h6: Add cooling map for GPU Clément Péron
@ 2022-08-19 4:13 ` Jernej Škrabec
2022-08-19 9:08 ` Clément Péron
0 siblings, 1 reply; 6+ messages in thread
From: Jernej Škrabec @ 2022-08-19 4:13 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Chen-Yu Tsai, Samuel Holland,
Clément Péron
Cc: Clément Péron, devicetree, linux-arm-kernel,
linux-sunxi, linux-kernel
Dne četrtek, 18. avgust 2022 ob 22:39:26 CEST je Clément Péron napisal(a):
> From: Clément Péron <peron.clem@gmail.com>
>
> Add a simple cooling map for the GPU.
>
> Signed-off-by: Clément Péron <peron.clem@gmail.com>
Is this really patch 2/4? I only got patches 2-4, but not patch 1.
Best regards,
Jernej
> ---
> arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi | 22 ++++++++++++++++++++
> 1 file changed, 22 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi
> b/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi index
> 5a28303d3d4c..943ae5374dd6 100644
> --- a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi
> +++ b/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi
> @@ -186,6 +186,7 @@ gpu: gpu@1800000 {
> clocks = <&ccu CLK_GPU>, <&ccu CLK_BUS_GPU>;
> clock-names = "core", "bus";
> resets = <&ccu RST_BUS_GPU>;
> + #cooling-cells = <2>;
> status = "disabled";
> };
>
> @@ -1075,6 +1076,27 @@ gpu-thermal {
> polling-delay-passive = <0>;
> polling-delay = <0>;
> thermal-sensors = <&ths 1>;
> +
> + trips {
> + gpu_alert: gpu-alert {
> + temperature = <85000>;
> + hysteresis = <2000>;
> + type = "passive";
> + };
> +
> + gpu-crit {
> + temperature =
<100000>;
> + hysteresis = <0>;
> + type = "critical";
> + };
> + };
> +
> + cooling-maps {
> + map0 {
> + trip = <&gpu_alert>;
> + cooling-device = <&gpu
THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
> + };
> + };
> };
> };
> };
^ permalink raw reply [flat|nested] 6+ messages in thread* Re: [PATCH 2/4] arm64: dts: allwinner: h6: Add cooling map for GPU
2022-08-19 4:13 ` Jernej Škrabec
@ 2022-08-19 9:08 ` Clément Péron
0 siblings, 0 replies; 6+ messages in thread
From: Clément Péron @ 2022-08-19 9:08 UTC (permalink / raw)
To: Jernej Škrabec
Cc: Rob Herring, Krzysztof Kozlowski, Chen-Yu Tsai, Samuel Holland,
devicetree, linux-arm-kernel, linux-sunxi, linux-kernel
Hi Jernej,
On Fri, 19 Aug 2022 at 06:13, Jernej Škrabec <jernej.skrabec@gmail.com> wrote:
>
> Dne četrtek, 18. avgust 2022 ob 22:39:26 CEST je Clément Péron napisal(a):
> > From: Clément Péron <peron.clem@gmail.com>
> >
> > Add a simple cooling map for the GPU.
> >
> > Signed-off-by: Clément Péron <peron.clem@gmail.com>
>
> Is this really patch 2/4? I only got patches 2-4, but not patch 1.
Sorry I have updated the way I send my patches, so each patch is sent
regarding what is outputted by 'scripts/get_maintainer.pl'
Patch 1 is https://lore.kernel.org/linux-arm-kernel/20220818203928.131059-2-cpe@outsight.tech/T/#md5d89721ac1861f31ca5d101ab9561371c35c904
Which has been sent to :
Catalin Marinas / Will Deacon <will@kernel.org>
And CC to :
linux-arm-kernel@lists.infradead.org / linux-kernel@vger.kernel.org
I will resend and force this patch to be sent to linux ML / Maintainer too.
Best regards,
Clement
>
> Best regards,
> Jernej
>
> > ---
> > arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi | 22 ++++++++++++++++++++
> > 1 file changed, 22 insertions(+)
> >
> > diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi
> > b/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi index
> > 5a28303d3d4c..943ae5374dd6 100644
> > --- a/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi
> > +++ b/arch/arm64/boot/dts/allwinner/sun50i-h6.dtsi
> > @@ -186,6 +186,7 @@ gpu: gpu@1800000 {
> > clocks = <&ccu CLK_GPU>, <&ccu CLK_BUS_GPU>;
> > clock-names = "core", "bus";
> > resets = <&ccu RST_BUS_GPU>;
> > + #cooling-cells = <2>;
> > status = "disabled";
> > };
> >
> > @@ -1075,6 +1076,27 @@ gpu-thermal {
> > polling-delay-passive = <0>;
> > polling-delay = <0>;
> > thermal-sensors = <&ths 1>;
> > +
> > + trips {
> > + gpu_alert: gpu-alert {
> > + temperature = <85000>;
> > + hysteresis = <2000>;
> > + type = "passive";
> > + };
> > +
> > + gpu-crit {
> > + temperature =
> <100000>;
> > + hysteresis = <0>;
> > + type = "critical";
> > + };
> > + };
> > +
> > + cooling-maps {
> > + map0 {
> > + trip = <&gpu_alert>;
> > + cooling-device = <&gpu
> THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
> > + };
> > + };
> > };
> > };
> > };
>
>
>
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 3/4] arm64: dts: allwinner: h6: Add GPU OPP table
[not found] <20220818203928.131059-1-cpe@outsight.tech>
2022-08-18 20:39 ` [PATCH 1/4] arm64: defconfig: Enable devfreq cooling device Clément Péron
2022-08-18 20:39 ` [PATCH 2/4] arm64: dts: allwinner: h6: Add cooling map for GPU Clément Péron
@ 2022-08-18 20:39 ` Clément Péron
2022-08-18 20:39 ` [PATCH 4/4] arm64: dts: allwinner: beelink-gs1: Enable GPU OPP Clément Péron
3 siblings, 0 replies; 6+ messages in thread
From: Clément Péron @ 2022-08-18 20:39 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland
Cc: Clément Péron, devicetree, linux-arm-kernel,
linux-sunxi, linux-kernel
From: Clément Péron <peron.clem@gmail.com>
Add an Operating Performance Points table for the GPU to
enable Dynamic Voltage & Frequency Scaling on the H6.
The voltage range is set with minival voltage set to the target
and the maximal voltage set to 1.2V. This allow DVFS framework to
work properly on board with fixed regulator.
Signed-off-by: Clément Péron <peron.clem@gmail.com>
---
.../boot/dts/allwinner/sun50i-h6-gpu-opp.dtsi | 88 +++++++++++++++++++
1 file changed, 88 insertions(+)
create mode 100644 arch/arm64/boot/dts/allwinner/sun50i-h6-gpu-opp.dtsi
diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h6-gpu-opp.dtsi b/arch/arm64/boot/dts/allwinner/sun50i-h6-gpu-opp.dtsi
new file mode 100644
index 000000000000..a66204243515
--- /dev/null
+++ b/arch/arm64/boot/dts/allwinner/sun50i-h6-gpu-opp.dtsi
@@ -0,0 +1,88 @@
+// SPDX-License-Identifier: (GPL-2.0+ OR MIT)
+// Copyright (C) 2022 Clément Péron <peron.clem@gmail.com>
+
+/ {
+ gpu_opp_table: gpu-opp-table {
+ compatible = "operating-points-v2";
+
+ opp@216000000 {
+ opp-hz = /bits/ 64 <216000000>;
+ opp-microvolt = <810000 810000 1200000>;
+ };
+
+ opp@264000000 {
+ opp-hz = /bits/ 64 <264000000>;
+ opp-microvolt = <810000 810000 1200000>;
+ };
+
+ opp@312000000 {
+ opp-hz = /bits/ 64 <312000000>;
+ opp-microvolt = <810000 810000 1200000>;
+ };
+
+ opp@336000000 {
+ opp-hz = /bits/ 64 <336000000>;
+ opp-microvolt = <810000 810000 1200000>;
+ };
+
+ opp@360000000 {
+ opp-hz = /bits/ 64 <360000000>;
+ opp-microvolt = <820000 820000 1200000>;
+ };
+
+ opp@384000000 {
+ opp-hz = /bits/ 64 <384000000>;
+ opp-microvolt = <830000 830000 1200000>;
+ };
+
+ opp@408000000 {
+ opp-hz = /bits/ 64 <408000000>;
+ opp-microvolt = <840000 840000 1200000>;
+ };
+
+ opp@420000000 {
+ opp-hz = /bits/ 64 <420000000>;
+ opp-microvolt = <850000 850000 1200000>;
+ };
+
+ opp@432000000 {
+ opp-hz = /bits/ 64 <432000000>;
+ opp-microvolt = <860000 860000 1200000>;
+ };
+
+ opp@456000000 {
+ opp-hz = /bits/ 64 <456000000>;
+ opp-microvolt = <870000 870000 1200000>;
+ };
+
+ opp@504000000 {
+ opp-hz = /bits/ 64 <504000000>;
+ opp-microvolt = <890000 890000 1200000>;
+ };
+
+ opp@540000000 {
+ opp-hz = /bits/ 64 <540000000>;
+ opp-microvolt = <910000 910000 1200000>;
+ };
+
+ opp@576000000 {
+ opp-hz = /bits/ 64 <576000000>;
+ opp-microvolt = <930000 930000 1200000>;
+ };
+
+ opp@624000000 {
+ opp-hz = /bits/ 64 <624000000>;
+ opp-microvolt = <950000 950000 1200000>;
+ };
+
+ opp@756000000 {
+ opp-hz = /bits/ 64 <756000000>;
+ opp-microvolt = <1040000 1040000 1200000>;
+ };
+ };
+
+};
+
+&gpu {
+ operating-points-v2 = <&gpu_opp_table>;
+};
--
2.34.1
^ permalink raw reply [flat|nested] 6+ messages in thread* [PATCH 4/4] arm64: dts: allwinner: beelink-gs1: Enable GPU OPP
[not found] <20220818203928.131059-1-cpe@outsight.tech>
` (2 preceding siblings ...)
2022-08-18 20:39 ` [PATCH 3/4] arm64: dts: allwinner: h6: Add GPU OPP table Clément Péron
@ 2022-08-18 20:39 ` Clément Péron
3 siblings, 0 replies; 6+ messages in thread
From: Clément Péron @ 2022-08-18 20:39 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Chen-Yu Tsai, Jernej Skrabec,
Samuel Holland
Cc: Clément Péron, devicetree, linux-arm-kernel,
linux-sunxi, linux-kernel
Enable GPU OPP table for Beelink GS1
Signed-off-by: Clément Péron <cpe@outsight.tech>
---
arch/arm64/boot/dts/allwinner/sun50i-h6-beelink-gs1.dts | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm64/boot/dts/allwinner/sun50i-h6-beelink-gs1.dts b/arch/arm64/boot/dts/allwinner/sun50i-h6-beelink-gs1.dts
index 6249e9e02928..20fc0584d1c6 100644
--- a/arch/arm64/boot/dts/allwinner/sun50i-h6-beelink-gs1.dts
+++ b/arch/arm64/boot/dts/allwinner/sun50i-h6-beelink-gs1.dts
@@ -5,6 +5,7 @@
#include "sun50i-h6.dtsi"
#include "sun50i-h6-cpu-opp.dtsi"
+#include "sun50i-h6-gpu-opp.dtsi"
#include <dt-bindings/gpio/gpio.h>
@@ -261,6 +262,7 @@ reg_dcdca: dcdca {
};
reg_dcdcc: dcdcc {
+ regulator-always-on;
regulator-enable-ramp-delay = <32000>;
regulator-min-microvolt = <810000>;
regulator-max-microvolt = <1080000>;
--
2.34.1
^ permalink raw reply [flat|nested] 6+ messages in thread