Re: [PATCH v4 1/2] dt-bindings: timer: renesas,ostm: Document RZ/V2N (R9A09G056) support

[Conor Dooley <conor@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 2286 bytes --]

On Thu, May 15, 2025 at 07:22:06PM +0100, Prabhakar wrote:
> From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> 
> Document support for the Renesas OS Timer (OSTM) found on the Renesas
> RZ/V2N (R9A09G056) SoC. The OSTM IP on RZ/V2N is identical to that on
> other RZ families, so no driver changes are required as `renesas,ostm`
> will be used as fallback compatible.
> 
> Also update the bindings to require the "resets" property for RZ/V2N
> by inverting the logic: all SoCs except RZ/A1 and RZ/A2 now require
> the "resets" property.
> 
> Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> ---
> v3->v4:
> - No changes in this version.
> 
> v2->v3:
> - Added Acked-by tag from Conor.

But v3 -> v4 doesn't mention why it was dropped?

> 
> v1->v2:
> - Inveted the logic in the dt-bindings to require the "resets" property
> - Updated commit message
> - Added a Reviewed-by tag from Geert.
> ---
>  .../devicetree/bindings/timer/renesas,ostm.yaml      | 12 ++++++------
>  1 file changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/timer/renesas,ostm.yaml b/Documentation/devicetree/bindings/timer/renesas,ostm.yaml
> index 9ba858f094ab..0983c1efec80 100644
> --- a/Documentation/devicetree/bindings/timer/renesas,ostm.yaml
> +++ b/Documentation/devicetree/bindings/timer/renesas,ostm.yaml
> @@ -26,6 +26,7 @@ properties:
>            - renesas,r9a07g043-ostm # RZ/G2UL and RZ/Five
>            - renesas,r9a07g044-ostm # RZ/G2{L,LC}
>            - renesas,r9a07g054-ostm # RZ/V2L
> +          - renesas,r9a09g056-ostm # RZ/V2N
>            - renesas,r9a09g057-ostm # RZ/V2H(P)
>        - const: renesas,ostm        # Generic
>  
> @@ -54,12 +55,11 @@ required:
>  if:
>    properties:
>      compatible:
> -      contains:
> -        enum:
> -          - renesas,r9a07g043-ostm
> -          - renesas,r9a07g044-ostm
> -          - renesas,r9a07g054-ostm
> -          - renesas,r9a09g057-ostm
> +      not:
> +        contains:
> +          enum:
> +            - renesas,r7s72100-ostm
> +            - renesas,r7s9210-ostm
>  then:
>    required:
>      - resets
> -- 
> 2.49.0
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]