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

Biju Das <biju.das.jz@xxxxxxxxxxxxxx> · Mon, 24 Jul 2023 15:29:27 +0000

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@xxxxxxxxxxxxxx>
> 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@xxxxxxxxxxxxxx>
> > Acked-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx>
> 
> Thanks for your patch!
> 
> Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx>
> 
> > --- 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