On 26.01.2018 17:26, Samuel Holland wrote:

On 01/26/18 09:19, Philipp Rossak wrote:

quoted

This patch adds the thermal zones to the H3. We have only one sensor and
that is placed in the cpu.

Signed-off-by: Philipp Rossak <redacted>
---
  arch/arm/boot/dts/sun8i-h3.dtsi | 9 +++++++++
  1 file changed, 9 insertions(+)

diff --git a/arch/arm/boot/dts/sun8i-h3.dtsi b/arch/arm/boot/dts/sun8i-h3.dtsi
index fbb007e5798e..3f83f6a27c74 100644
--- a/arch/arm/boot/dts/sun8i-h3.dtsi
+++ b/arch/arm/boot/dts/sun8i-h3.dtsi

@@ -72,6 +72,15 @@
  		};
  	};
  
+	thermal-zones {
+		cpu-thermal {
+			/* milliseconds */
+			polling-delay-passive = <250>;
+			polling-delay = <1000>;
+			thermal-sensors = <&ths 0>;

If #thermal-sensor-cells = <0>, shouldn't this just be <&ths> ?

I think both works, but I my intention was to have an accurate 
description in DT.
I thought that that is important in DT.

quoted

+		};
+	};
+
  	timer {
  		compatible = "arm,armv7-timer";
  		interrupts = <GIC_PPI 13 (GIC_CPU_MASK_SIMPLE(4) | IRQ_TYPE_LEVEL_LOW)>,

Thanks,
Samuel

Regards,
Philipp