Re: [PATCH 6/9] arm64: dts: mediatek: mt8186: add default thermal zones

Nicolas Pitre
2024-01-15T17:48:12Z

On Fri, 12 Jan 2024, Krzysztof Kozlowski wrote:

> On 11/01/2024 23:30, Nicolas Pitre wrote:
>
> > diff --git a/arch/arm64/boot/dts/mediatek/mt8186.dtsi b/arch/arm64/boot/dts/mediatek/mt8186.dtsi
> > index 8fc563dce6..91b902a9f0 100644
> > --- a/arch/arm64/boot/dts/mediatek/mt8186.dtsi
> > +++ b/arch/arm64/boot/dts/mediatek/mt8186.dtsi
> > @@ -2115,4 +2117,238 @@ larb19: smi@1c10f000 {
> > power-domains = <&spm MT8186_POWER_DOMAIN_IPE>;
> > };
> > };
> > +
> > + thermal_zones: thermal-zones {
> > + cpu_zone0-thermal {
>
> No underscores in node names. Could one CPU have multiple names? If not,
> then it is just "cpu0-thermal".

Well... I'm not completely clear about this given the available info,
but this thermal zone would not be matching a single CPU but a few of
them, hence several "zones of CPUs".

Nicolas