lkml.org 
[lkml]   [2019]   [Aug]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    SubjectRe: [PATCH v4 4/6] arm64: dts: meson: sei510: Add minimal thermal zone
    Date
    Guillaume La Roque <glaroque@baylibre.com> writes:

    > Add minimal thermal zone for two temperature sensor
    > One is located close to the DDR and the other one is
    > located close to the PLLs (between the CPU and GPU)
    >
    > Signed-off-by: Guillaume La Roque <glaroque@baylibre.com>
    > Acked-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>
    > ---
    > .../boot/dts/amlogic/meson-g12a-sei510.dts | 70 +++++++++++++++++++
    > 1 file changed, 70 insertions(+)
    >
    > diff --git a/arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts b/arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts
    > index c9fa23a56562..35d2ebbd6d4e 100644
    > --- a/arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts
    > +++ b/arch/arm64/boot/dts/amlogic/meson-g12a-sei510.dts
    > @@ -10,6 +10,7 @@
    > #include <dt-bindings/input/input.h>
    > #include <dt-bindings/gpio/meson-g12a-gpio.h>
    > #include <dt-bindings/sound/meson-g12a-tohdmitx.h>
    > +#include <dt-bindings/thermal/thermal.h>
    >
    > / {
    > compatible = "seirobotics,sei510", "amlogic,g12a";
    > @@ -33,6 +34,67 @@
    > ethernet0 = &ethmac;
    > };
    >
    > + thermal-zones {
    > + cpu-thermal {
    > + polling-delay = <1000>;
    > + polling-delay-passive = <100>;
    > + thermal-sensors = <&cpu_temp>;
    > +
    > + trips {
    > + cpu_hot: cpu-hot {
    > + temperature = <85000>; /* millicelsius */
    > + hysteresis = <2000>; /* millicelsius */
    > + type = "hot";
    > + };
    > +
    > + cpu_critical: cpu-critical {
    > + temperature = <110000>; /* millicelsius */
    > + hysteresis = <2000>; /* millicelsius */
    > + type = "critical";
    > + };
    > + };
    > +
    > + cooling-maps {
    > + map0 {
    > + trip = <&cpu_hot>;
    > + cooling-device = <&cpu0 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
    > + <&cpu1 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
    > + <&cpu2 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
    > + <&cpu3 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
    > + };
    > +
    > + map1 {
    > + trip = <&cpu_critical>;
    > + cooling-device = <&cpu0 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
    > + <&cpu1 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
    > + <&cpu2 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>,
    > + <&cpu3 THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
    > + };
    > + };
    > + };
    > +
    > + ddr-thermal {
    > + polling-delay = <1000>;
    > + polling-delay-passive = <100>;
    > + thermal-sensors = <&ddr_temp>;
    > +
    > + trips {
    > + ddr_critical: ddr-critical {
    > + temperature = <110000>; /* millicelsius */
    > + hysteresis = <2000>; /* millicelsius */
    > + type = "critical";
    > + };
    > + };
    > +
    > + cooling-maps {
    > + map {
    > + trip = <&ddr_critical>;
    > + cooling-device = <&mali THERMAL_NO_LIMIT THERMAL_NO_LIMIT>;
    > + };
    > + };
    > + };
    > + };
    > +
    > mono_dac: audio-codec-0 {
    > compatible = "maxim,max98357a";
    > #sound-dai-cells = <0>;
    > @@ -321,6 +383,7 @@
    > operating-points-v2 = <&cpu_opp_table>;
    > clocks = <&clkc CLKID_CPU_CLK>;
    > clock-latency = <50000>;
    > + #cooling-cells = <2>;
    > };
    >
    > &cpu1 {
    > @@ -328,6 +391,7 @@
    > operating-points-v2 = <&cpu_opp_table>;
    > clocks = <&clkc CLKID_CPU_CLK>;
    > clock-latency = <50000>;
    > + #cooling-cells = <2>;
    > };
    >
    > &cpu2 {
    > @@ -335,6 +399,7 @@
    > operating-points-v2 = <&cpu_opp_table>;
    > clocks = <&clkc CLKID_CPU_CLK>;
    > clock-latency = <50000>;
    > + #cooling-cells = <2>;
    > };
    >
    > &cpu3 {
    > @@ -342,6 +407,7 @@
    > operating-points-v2 = <&cpu_opp_table>;
    > clocks = <&clkc CLKID_CPU_CLK>;
    > clock-latency = <50000>;
    > + #cooling-cells = <2>;
    > };
    >
    > &cvbs_vdac_port {
    > @@ -368,6 +434,10 @@
    > status = "okay";
    > };
    >
    > +&mali {
    > + #cooling-cells = <2>;
    > +};
    > +

    Is there a reason these #cooling-cells properties belong in the SoC
    .dtsi and not the board .dts. Seems like you'll have to repeat this in
    every board .dts which doesn't seem necessary.

    Same comment for patch 5/6

    Kevin

    \
     
     \ /
      Last update: 2019-08-22 01:30    [W:4.109 / U:0.112 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site