Re: [PATCH v2 13/16] arm: dts: sun8i: h3: add thermal zone to H3

[Philipp Rossak]

On 29.01.2018 10:50, Maxime Ripard wrote:
> On Mon, Jan 29, 2018 at 12:29:16AM +0100, Philipp Rossak wrote:
> > 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 <embed3d@xxxxxxxxx>
> > ---
> >   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 the thermal-sensor-cells value is indeed 0, the phandle parsing
> will be broken here.
>
> Maxime

Ok, then I will remove the 0.

Thanks,

Philipp