Re: [PATCH 03/14] dt-bindings: soc: Document the Renesas RZ/T2H Evaluation board for the R9A09G077 SoC

[Thierry Bultel <thierry.bultel.yh@bp.renesas.com>]

On Mon, Feb 10, 2025 at 02:21:05PM +0100, Geert Uytterhoeven wrote:
> Hi Thierry,
> 
> Thanks for your patch!
> 
> On Wed, 29 Jan 2025 at 17:52, Thierry Bultel
> <thierry.bultel.yh@bp.renesas.com> wrote:
> > Add the RZ/T2H Evaluation board (r9a9g077m44-dev) in documentation.
> 
> This is the Evaluation Board Kit for RZ/T2H, "RZ/T2H-EVKIT" aka "rzt2hevb"?

Yes. AFAIK there is no other one as of today.

> 
> > Signed-off-by: Thierry Bultel <thierry.bultel.yh@bp.renesas.com>
> 
> > --- a/Documentation/devicetree/bindings/soc/renesas/renesas.yaml
> > +++ b/Documentation/devicetree/bindings/soc/renesas/renesas.yaml
> > @@ -537,6 +537,8 @@ properties:
> >
> >        - description: RZ/T2H (R9A09G077)
> >          items:
> > +          - enum:
> > +            - renesas,r9a9g077m44-dev # RZ/T2H Evaluation Board
> 
> Perhaps "renesas,rzt2hevb" or "renesas,rzt2h-evb"?

We would lose the 'm44' variant information by doing so,
is that acceptable ?
But anyway this should rather match the dts file name
that comes in a later patch (named: r9a09g077m44-rzt2h-evk.dts)

So what about 'r9a09g077m44-rzt2h-evk' here instead ?


> 
> >            - enum:
> >              - renesas,r9a09g077 # RZ/T2H with Quad Cortex-A55 + Dual Cortex-R52
> >              - renesas,r9a09g077m04 # RZ/T2H with Single Cortex-A55 + Dual Cortex-R52 - no security
> 
> 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