Re: [PATCH] arm64: dts: mediatek: mt8183: remove thermal zones without trips.

[Daniel Lezcano]

On 14/09/2022 16:15, Matthias Brugger wrote:
> On 14/09/2022 15:13, Amjad Ouled-Ameur wrote:
> > Thermal zones without trip point are not registered by thermal core.
> >
> > tzts1 ~ tzts6 zones of mt8183 were intially introduced for test-purpose
> > only but are not supposed to remain on DT.
> >
> > Remove the zones above and keep only cpu_thermal.
>
> My understanding is that this thermal zones exist but are not used. DTS 
> should describe the HW as it is, so I propose to add
> status = "disabled";
> to the zones.

At least the trip points must be declared as stated in the bindings

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/devicetree/bindings/thermal/thermal-zones.yaml#n213

--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog