Hi Geert, > -----Original Message----- > From: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> > Sent: 06 March 2025 11:17 > Subject: Re: [PATCH 1/8] dt-bindings: memory: Document RZ/G3E support > > Hi Biju, > > On Thu, 6 Mar 2025 at 11:59, Biju Das <biju.das.jz@xxxxxxxxxxxxxx> wrote: > > > -----Original Message----- > > > From: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> > > > Sent: 06 March 2025 10:44 > > > Subject: Re: [PATCH 1/8] dt-bindings: memory: Document RZ/G3E > > > support > > > > > > On Fri, 28 Feb 2025 at 17:08, Biju Das <biju.das.jz@xxxxxxxxxxxxxx> wrote: > > > > Document support for the Expanded Serial Peripheral Interface > > > > (xSPI) Controller in the Renesas RZ/G3E (R9A09G047) SoC. > > > > > > > > Signed-off-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx> > > > > > > Thanks for your patch! > > > > > > > --- /dev/null > > > > +++ b/Documentation/devicetree/bindings/memory-controllers/renesas > > > > +++ ,rz- > > > > +++ xspi.yaml > > > > > > > +properties: > > > > + compatible: > > > > + items: > > > > + - const: renesas,r9a09g047-xspi # RZ/G3E > > > > + - const: renesas,rz-xspi # a generic RZ xSPI device > > > > > > Is this the same variant of RPC-IF as used on older "RZ" SoCs like > > > RZ/A1 and RZ/A2? > > > If the answer is yes, then I do not object against introducing rz-xspi. > > > > No, by looking at the registers, > > I think one on RZ/A1 and RZ/A2 is same as RZ/G2L. So > > renesas,rzg2l-rpc-if will fit there. > > In that case I think "renesas,rz-xspi" is a too generic name. if rz-xspi is too generic, what about using rzg3e-xspi? Note: RZ/G3S and RZ/V2H have similar IP's Cheers, Biju
