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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Dec 8, 2021 at 3:27 PM Biju Das <biju.das.jz@xxxxxxxxxxxxxx> 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@xxxxxxxxxxx>
> and others for r8a77990 SoC.
>
> Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>
> Reviewed-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx>

LGTM (I'm no thermal expert, so an additional pair of eyes wouldn't hurt), so
Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
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@xxxxxxxxxxxxxx

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



[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux