diff mbox series

[2/2] arm64: dts: renesas: r9a07g044: Create thermal zone to support IPA

Message ID 20211208142729.2456-3-biju.das.jz@bp.renesas.com (mailing list archive)
State Mainlined
Commit 88404c56fde05eb741552a33fdfe6d7d20c1c986
Delegated to: Geert Uytterhoeven
Headers show
Series Enable Thermal support for RZ/G2L | expand

Commit Message

Biju Das Dec. 8, 2021, 2:27 p.m. UTC
Setup a thermal zone driven by SoC temperature sensor.
Create passive trip points and bind them to CPUFreq cooling
device that supports power extension.

Based on the work done by Dien Pham <dien.pham.ry@renesas.com>
and others for r8a77990 SoC.

Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
---
 arch/arm64/boot/dts/renesas/r9a07g044.dtsi | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

Comments

Geert Uytterhoeven Dec. 14, 2021, 11:13 a.m. UTC | #1
On Wed, Dec 8, 2021 at 3:27 PM Biju Das <biju.das.jz@bp.renesas.com> wrote:
> Setup a thermal zone driven by SoC temperature sensor.
> Create passive trip points and bind them to CPUFreq cooling
> device that supports power extension.
>
> Based on the work done by Dien Pham <dien.pham.ry@renesas.com>
> and others for r8a77990 SoC.
>
> Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
> Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>

LGTM (I'm no thermal expert, so an additional pair of eyes wouldn't hurt), so
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
i.e. will queue in renesas-devel for v5.17.

> --- a/arch/arm64/boot/dts/renesas/r9a07g044.dtsi
> +++ b/arch/arm64/boot/dts/renesas/r9a07g044.dtsi
> @@ -88,6 +88,7 @@
>                         compatible = "arm,cortex-a55";
>                         reg = <0>;
>                         device_type = "cpu";
> +                       #cooling-cells = <2>;
>                         next-level-cache = <&L3_CA55>;
>                         enable-method = "psci";
>                         clocks = <&cpg CPG_CORE R9A07G044_CLK_I>;
> @@ -917,6 +918,15 @@
>                         polling-delay-passive = <250>;
>                         polling-delay = <1000>;
>                         thermal-sensors = <&tsu 0>;
> +                       sustainable-power = <717>;
> +
> +                       cooling-maps {
> +                               map0 {
> +                                       trip = <&target>;
> +                                       cooling-device = <&cpu0 0 2>;
> +                                       contribution = <1024>;
> +                               };
> +                       };
>
>                         trips {
>                                 sensor_crit: sensor-crit {
> @@ -924,6 +934,12 @@
>                                         hysteresis = <1000>;
>                                         type = "critical";
>                                 };
> +
> +                               target: trip-point {
> +                                       temperature = <100000>;
> +                                       hysteresis = <1000>;
> +                                       type = "passive";
> +                               };
>                         };
>                 };
>         };

Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
diff mbox series

Patch

diff --git a/arch/arm64/boot/dts/renesas/r9a07g044.dtsi b/arch/arm64/boot/dts/renesas/r9a07g044.dtsi
index ea528580f306..2639fd383010 100644
--- a/arch/arm64/boot/dts/renesas/r9a07g044.dtsi
+++ b/arch/arm64/boot/dts/renesas/r9a07g044.dtsi
@@ -88,6 +88,7 @@ 
 			compatible = "arm,cortex-a55";
 			reg = <0>;
 			device_type = "cpu";
+			#cooling-cells = <2>;
 			next-level-cache = <&L3_CA55>;
 			enable-method = "psci";
 			clocks = <&cpg CPG_CORE R9A07G044_CLK_I>;
@@ -917,6 +918,15 @@ 
 			polling-delay-passive = <250>;
 			polling-delay = <1000>;
 			thermal-sensors = <&tsu 0>;
+			sustainable-power = <717>;
+
+			cooling-maps {
+				map0 {
+					trip = <&target>;
+					cooling-device = <&cpu0 0 2>;
+					contribution = <1024>;
+				};
+			};
 
 			trips {
 				sensor_crit: sensor-crit {
@@ -924,6 +934,12 @@ 
 					hysteresis = <1000>;
 					type = "critical";
 				};
+
+				target: trip-point {
+					temperature = <100000>;
+					hysteresis = <1000>;
+					type = "passive";
+				};
 			};
 		};
 	};