Hi Thierry,
On Mon, 10 Feb 2025 at 15:03, Thierry Bultel
<thierry.bultel.yh@xxxxxxxxxxxxxx> wrote:
> On Mon, Feb 10, 2025 at 02:21:05PM +0100, Geert Uytterhoeven wrote:
> > On Wed, 29 Jan 2025 at 17:52, Thierry Bultel
> > <thierry.bultel.yh@xxxxxxxxxxxxxx> 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.
>
> > > --- 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 ?
You would still have the "m44" info in the fallback compatible value:
compatible = "renesas,rzt2h-evb", "renesas,r9a9g077m44", "renesas,r9a9g077";
A variant with the single-core variant would have:
compatible = "renesas,rzt2h-evb", "renesas,r9a9g077m04", "renesas,r9a9g077";
Cfr. e.g. the Salvator-X(S) development boards, which are available
with different processor options (R-Car H3/M3-W/M3-N + "Gen3e" variants).
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
