diff mbox series

[v2,3/6] dt-bindings: timer: renesas,rz-mtu3: Document RZ/G2UL SoC

Message ID 20230724091927.123847-4-biju.das.jz@bp.renesas.com (mailing list archive)
State Superseded
Delegated to: Geert Uytterhoeven
Headers show
Series Add RZ/G2UL MTU3a support | expand

Commit Message

Biju Das July 24, 2023, 9:19 a.m. UTC 
Add MTU3a binding documentation for Renesas RZ/G2UL SoC.

MTU3a block is identical to one found on RZ/G2L, so no driver changes are
required. The fallback compatible string "renesas,rz-mtu3" will be used
on RZ/G2UL.

Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
Acked-by: Conor Dooley <conor.dooley@microchip.com>
---
v1->v2:
 * Added ack from Conor Dooley.
---
 Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml | 1 +
 1 file changed, 1 insertion(+)

Comments

Geert Uytterhoeven July 24, 2023, 3:26 p.m. UTC | #1 
Hi Biju,

On Mon, Jul 24, 2023 at 11:19 AM Biju Das <biju.das.jz@bp.renesas.com> wrote:
> Add MTU3a binding documentation for Renesas RZ/G2UL SoC.
>
> MTU3a block is identical to one found on RZ/G2L, so no driver changes are
> required. The fallback compatible string "renesas,rz-mtu3" will be used
> on RZ/G2UL.
>
> Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
> Acked-by: Conor Dooley <conor.dooley@microchip.com>

Thanks for your patch!

Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>

> --- a/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml
> +++ b/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml
> @@ -109,6 +109,7 @@ properties:
>    compatible:
>      items:
>        - enum:
> +          - renesas,r9a07g043-mtu3  # RZ/G2UL
>            - renesas,r9a07g044-mtu3  # RZ/G2{L,LC}
>            - renesas,r9a07g054-mtu3  # RZ/V2L
>        - const: renesas,rz-mtu3

As patch 5/6 applies to both RZ/G2UL and RZ/Five, I think you should
add "and RZ/Five", too.

Gr{oetje,eeting}s,

                        Geert


--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
                                -- Linus Torvalds
Biju Das July 24, 2023, 3:29 p.m. UTC | #2 
Hi Geert Uytterhoeven,

Thanks for the feedback.

> Subject: Re: [PATCH v2 3/6] dt-bindings: timer: renesas,rz-mtu3:
> Document RZ/G2UL SoC
> 
> Hi Biju,
> 
> On Mon, Jul 24, 2023 at 11:19 AM Biju Das <biju.das.jz@bp.renesas.com>
> wrote:
> > Add MTU3a binding documentation for Renesas RZ/G2UL SoC.
> >
> > MTU3a block is identical to one found on RZ/G2L, so no driver changes
> > are required. The fallback compatible string "renesas,rz-mtu3" will be
> > used on RZ/G2UL.
> >
> > Signed-off-by: Biju Das <biju.das.jz@bp.renesas.com>
> > Acked-by: Conor Dooley <conor.dooley@microchip.com>
> 
> Thanks for your patch!
> 
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> 
> > --- a/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml
> > +++ b/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml
> > @@ -109,6 +109,7 @@ properties:
> >    compatible:
> >      items:
> >        - enum:
> > +          - renesas,r9a07g043-mtu3  # RZ/G2UL
> >            - renesas,r9a07g044-mtu3  # RZ/G2{L,LC}
> >            - renesas,r9a07g054-mtu3  # RZ/V2L
> >        - const: renesas,rz-mtu3
> 
> As patch 5/6 applies to both RZ/G2UL and RZ/Five, I think you should add
> "and RZ/Five", too.

OK, will add RZ/Five as well.

Cheers,
Biju
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml b/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml
index 670a2ebaacdb..2269e0bf8818 100644
--- a/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml
+++ b/Documentation/devicetree/bindings/timer/renesas,rz-mtu3.yaml
@@ -109,6 +109,7 @@  properties:
   compatible:
     items:
       - enum:
+          - renesas,r9a07g043-mtu3  # RZ/G2UL
           - renesas,r9a07g044-mtu3  # RZ/G2{L,LC}
           - renesas,r9a07g054-mtu3  # RZ/V2L
       - const: renesas,rz-mtu3