linux-kernel - Re: [PATCH 1/2] arm64: dts: mediatek: mt8189: Add thermal controller node

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20251110-vagueness-waggle-50b8efe728a8@spud>
Date: Mon, 10 Nov 2025 18:45:20 +0000
From: Conor Dooley <conor@...nel.org>
To: Hanchien Lin <hanchien.lin@...iatek.com>
Cc: "Rafael J . Wysocki" <rafael@...nel.org>,
	Daniel Lezcano <daniel.lezcano@...aro.org>,
	Zhang Rui <rui.zhang@...el.com>, Lukasz Luba <lukasz.luba@....com>,
	Rob Herring <robh@...nel.org>,
	Krzysztof Kozlowski <krzk+dt@...nel.org>,
	Conor Dooley <conor+dt@...nel.org>,
	Philipp Zabel <p.zabel@...gutronix.de>,
	Matthias Brugger <matthias.bgg@...il.com>,
	AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>,
	Balsam CHIHI <bchihi@...libre.com>, linux-pm@...r.kernel.org,
	devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
	linux-arm-kernel@...ts.infradead.org,
	linux-mediatek@...ts.infradead.org,
	Project_Global_Chrome_Upstream_Group@...iatek.com,
	raymond.sun@...iatek.com, Irving-CH.lin@...iatek.com
Subject: Re: [PATCH 1/2] arm64: dts: mediatek: mt8189: Add thermal controller
 node

On Mon, Nov 10, 2025 at 05:40:38PM +0800, Hanchien Lin wrote:
> Add device tree node for the thermal controller on MediaTek MT8189 SoC.

$subject is incorrect for a binding change.
pw-bot: changes-requested

> 
> Signed-off-by: Hanchien Lin <hanchien.lin@...iatek.com>
> ---
>  .../thermal/mediatek,lvts-thermal.yaml        | 27 +++++++++++++++++--
>  .../thermal/mediatek,lvts-thermal.h           | 20 ++++++++++++++
>  2 files changed, 45 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/thermal/mediatek,lvts-thermal.yaml b/Documentation/devicetree/bindings/thermal/mediatek,lvts-thermal.yaml
> index 0259cd3ce9c5..0f7fd69f5fdf 100644
> --- a/Documentation/devicetree/bindings/thermal/mediatek,lvts-thermal.yaml
> +++ b/Documentation/devicetree/bindings/thermal/mediatek,lvts-thermal.yaml
> @@ -22,6 +22,8 @@ properties:
>        - mediatek,mt8186-lvts
>        - mediatek,mt8188-lvts-ap
>        - mediatek,mt8188-lvts-mcu
> +      - mediatek,mt8189-lvts-ap
> +      - mediatek,mt8189-lvts-mcu
>        - mediatek,mt8192-lvts-ap
>        - mediatek,mt8192-lvts-mcu
>        - mediatek,mt8195-lvts-ap
> @@ -58,6 +60,21 @@ properties:
>  allOf:
>    - $ref: thermal-sensor.yaml#
>  
> +  - if:
> +      properties:
> +        compatible:
> +          contains:
> +            enum:
> +              - mediatek,mt8189-lvts-ap
> +              - mediatek,mt8189-lvts-mcu
> +    then:
> +      properties:
> +        nvmem-cells:
> +          minItems: 2
> +
> +        nvmem-cell-names:
> +          minItems: 2
> +
>    - if:
>        properties:
>          compatible:
> @@ -75,6 +92,10 @@ allOf:
>          nvmem-cell-names:
>            maxItems: 1
>  
> +      required:
> +        - clocks
> +        - resets
> +
>    - if:
>        properties:
>          compatible:
> @@ -91,12 +112,14 @@ allOf:
>          nvmem-cell-names:
>            minItems: 2
>  
> +      required:
> +        - clocks
> +        - resets
> +
>  required:
>    - compatible
>    - reg
>    - interrupts
> -  - clocks
> -  - resets
>    - nvmem-cells
>    - nvmem-cell-names
>  
> diff --git a/include/dt-bindings/thermal/mediatek,lvts-thermal.h b/include/dt-bindings/thermal/mediatek,lvts-thermal.h
> index ddc7302a510a..6c9103dfdc2d 100644
> --- a/include/dt-bindings/thermal/mediatek,lvts-thermal.h
> +++ b/include/dt-bindings/thermal/mediatek,lvts-thermal.h
> @@ -42,6 +42,26 @@
>  #define MT8188_AP_CAM1		6
>  #define MT8188_AP_CAM2		7
>  
> +#define MT8189_MCU_BIG_CPU1     0
> +#define MT8189_MCU_BIG_CPU2     1
> +#define MT8189_MCU_BIG_CPU3     2
> +#define MT8189_MCU_BIG_CPU4     3
> +#define MT8189_MCU_LITTLE_CPU1  4
> +#define MT8189_MCU_LITTLE_CPU2  5
> +#define MT8189_MCU_LITTLE_CPU3  6
> +#define MT8189_MCU_LITTLE_CPU4  7
> +#define MT8189_MCU_LITTLE_CPU5  8
> +#define MT8189_MCU_LITTLE_CPU6  9
> +#define MT8189_MCU_LITTLE_CPU7  10
> +#define MT8189_MCU_LITTLE_CPU8  11
> +
> +#define MT8189_AP_SOC1          12
> +#define MT8189_AP_SOC2          13
> +#define MT8189_AP_SOC3          14
> +#define MT8189_AP_APU           15
> +#define MT8189_AP_GPU1          16
> +#define MT8189_AP_GPU2          17
> +
>  #define MT8195_MCU_BIG_CPU0     0
>  #define MT8195_MCU_BIG_CPU1     1
>  #define MT8195_MCU_BIG_CPU2     2
> -- 
> 2.45.2
> 

Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)