RE: [PATCH] dt-bindings: can: renesas,rcar-canfd: Restrict resets in top-level

[Biju Das]

Hi Krzysztof Kozlowski,

> -----Original Message-----
> From: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxxxxxxxx>
> Sent: 01 October 2026 07:08
> Subject: [PATCH] dt-bindings: can: renesas,rcar-canfd: Restrict resets in top-level
> 
> Properties varying per variant of the device should still have the broadest constraints set in the top
> level, so add such for the "resets".
> Since top-level part says "reset-names" cannot accept one item, disallow it for the two variants which do
> not use it in practice.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxxxxxxxx>

Reviewed-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx>

Cheers,
Biju

> ---
>  .../devicetree/bindings/net/can/renesas,rcar-canfd.yaml   | 8 +++++---
>  1 file changed, 5 insertions(+), 3 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/net/can/renesas,rcar-canfd.yaml
> b/Documentation/devicetree/bindings/net/can/renesas,rcar-canfd.yaml
> index b9d9dd7a7967..ca92e1a9a051 100644
> --- a/Documentation/devicetree/bindings/net/can/renesas,rcar-canfd.yaml
> +++ b/Documentation/devicetree/bindings/net/can/renesas,rcar-canfd.yaml
> @@ -132,7 +132,9 @@ properties:
>    power-domains:
>      maxItems: 1
> 
> -  resets: true
> +  resets:
> +    minItems: 1
> +    maxItems: 2
> 
>    reset-names:
>      items:
> @@ -209,7 +211,6 @@ allOf:
> 
>          resets:
>            minItems: 2
> -          maxItems: 2
> 
>    - if:
>        properties:
> @@ -231,6 +232,8 @@ allOf:
>          resets:
>            maxItems: 1
> 
> +        reset-names: false
> +
>    - if:
>        properties:
>          compatible:
> @@ -246,7 +249,6 @@ allOf:
> 
>          resets:
>            minItems: 2
> -          maxItems: 2
> 
>    - if:
>        properties:
> --
> 2.53.0
>