[PATCH 05/10] dt-bindings: thermal: r9a09g047-tsu: document RZ/T2H and RZ/N2H

Cosmin Tanislav posted 10 patches 3 months, 2 weeks ago
There is a newer version of this series
[PATCH 05/10] dt-bindings: thermal: r9a09g047-tsu: document RZ/T2H and RZ/N2H
Posted by Cosmin Tanislav 3 months, 2 weeks ago
The Renesas RZ/T2H (R9A09G077) and RZ/N2H (R9A09G087) SoCs include a
Temperature Sensor Unit (TSU). The device provides real-time temperature
measurements for thermal management, utilizing a single dedicated
channel for temperature sensing.

Signed-off-by: Cosmin Tanislav <cosmin-gabriel.tanislav.xa@renesas.com>
---
 .../thermal/renesas,r9a09g047-tsu.yaml        | 22 ++++++++++++++++---
 1 file changed, 19 insertions(+), 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/thermal/renesas,r9a09g047-tsu.yaml b/Documentation/devicetree/bindings/thermal/renesas,r9a09g047-tsu.yaml
index 8d3f3c24f0f2..3cb66b51831d 100644
--- a/Documentation/devicetree/bindings/thermal/renesas,r9a09g047-tsu.yaml
+++ b/Documentation/devicetree/bindings/thermal/renesas,r9a09g047-tsu.yaml
@@ -16,7 +16,14 @@ description:
 
 properties:
   compatible:
-    const: renesas,r9a09g047-tsu
+    oneOf:
+      - items:
+          - const: renesas,r9a09g047-tsu # RZ/G3E
+      - items:
+          - const: renesas,r9a09g077-tsu # RZ/T2H
+      - items:
+          - const: renesas,r9a09g087-tsu # RZ/N2H
+          - const: renesas,r9a09g077-tsu # RZ/T2H
 
   reg:
     maxItems: 1
@@ -59,12 +66,21 @@ required:
   - compatible
   - reg
   - clocks
-  - resets
   - power-domains
   - interrupts
   - interrupt-names
   - "#thermal-sensor-cells"
-  - renesas,tsu-trim
+
+allOf:
+  - if:
+      properties:
+        compatible:
+          contains:
+            const: renesas,r9a09g047-tsu
+    then:
+      required:
+        - resets
+        - renesas,tsu-trim
 
 additionalProperties: false
 
-- 
2.51.1.dirty
Re: [PATCH 05/10] dt-bindings: thermal: r9a09g047-tsu: document RZ/T2H and RZ/N2H
Posted by Conor Dooley 3 months, 2 weeks ago
On Thu, Oct 23, 2025 at 11:19:19AM +0300, Cosmin Tanislav wrote:
> The Renesas RZ/T2H (R9A09G077) and RZ/N2H (R9A09G087) SoCs include a
> Temperature Sensor Unit (TSU). The device provides real-time temperature
> measurements for thermal management, utilizing a single dedicated
> channel for temperature sensing.
> 
> Signed-off-by: Cosmin Tanislav <cosmin-gabriel.tanislav.xa@renesas.com>
> ---
>  .../thermal/renesas,r9a09g047-tsu.yaml        | 22 ++++++++++++++++---
>  1 file changed, 19 insertions(+), 3 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/thermal/renesas,r9a09g047-tsu.yaml b/Documentation/devicetree/bindings/thermal/renesas,r9a09g047-tsu.yaml
> index 8d3f3c24f0f2..3cb66b51831d 100644
> --- a/Documentation/devicetree/bindings/thermal/renesas,r9a09g047-tsu.yaml
> +++ b/Documentation/devicetree/bindings/thermal/renesas,r9a09g047-tsu.yaml
> @@ -16,7 +16,14 @@ description:
>  
>  properties:
>    compatible:
> -    const: renesas,r9a09g047-tsu
> +    oneOf:

> +      - items:
> +          - const: renesas,r9a09g047-tsu # RZ/G3E
> +      - items:
> +          - const: renesas,r9a09g077-tsu # RZ/T2H

These two should just go together in an enum, no need to overcomplicate
things with items like this. Items is only needed when you have more
than one.
pw-bot: changes-requested

Cheers,
Conor.

> +      - items:
> +          - const: renesas,r9a09g087-tsu # RZ/N2H
> +          - const: renesas,r9a09g077-tsu # RZ/T2H
>  
>    reg:
>      maxItems: 1
> @@ -59,12 +66,21 @@ required:
>    - compatible
>    - reg
>    - clocks
> -  - resets
>    - power-domains
>    - interrupts
>    - interrupt-names
>    - "#thermal-sensor-cells"
> -  - renesas,tsu-trim
> +
> +allOf:
> +  - if:
> +      properties:
> +        compatible:
> +          contains:
> +            const: renesas,r9a09g047-tsu
> +    then:
> +      required:
> +        - resets
> +        - renesas,tsu-trim
>  
>  additionalProperties: false
>  
> -- 
> 2.51.1.dirty
>