Hi Prabhakar, Thanks for your patch! On Mon, Feb 19, 2024 at 5:10 PM Prabhakar <prabhakar.csengg@xxxxxxxxx> wrote: > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> > > Document Renesas RZ/V2H{P} (R9A09G057) SoC variants. I think "RZ/V2H(P)" would be better, as curly braces are usually used to group multiple values (e.g. "RZ/G2{L,LC}"). > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> > Reviewed-by: Fabrizio Castro <fabrizio.castro.jz@xxxxxxxxxxx> > --- > .../devicetree/bindings/soc/renesas/renesas.yaml | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/Documentation/devicetree/bindings/soc/renesas/renesas.yaml b/Documentation/devicetree/bindings/soc/renesas/renesas.yaml > index c1ce4da2dc32..109fbc8d48db 100644 > --- a/Documentation/devicetree/bindings/soc/renesas/renesas.yaml > +++ b/Documentation/devicetree/bindings/soc/renesas/renesas.yaml > @@ -513,6 +513,14 @@ properties: > - renesas,rzv2mevk2 # RZ/V2M Eval Board v2.0 > - const: renesas,r9a09g011 > > + - description: RZ/V2H{P} (R9A09G057) RZ/V2H(P) > + items: > + - enum: > + - renesas,r9a09g057h41 # RZ/V2H > + - renesas,r9a09g057h42 # RZ/V2H with Mali-G31 support > + - renesas,r9a09g057h44 # RZ/V2HP with Mali-G31 + Mali-C55 support > + - const: renesas,r9a09g057 > + > additionalProperties: true The rest LGTM, so Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx 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