Hi Prabhakar, On Wed, Nov 17, 2021 at 2:05 AM Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> wrote: > Add RSPI binding documentation for Renesas RZ/G2L SoC. > > RSPI block is identical to one found on RZ/A, so no driver changes are > required the fallback compatible string "renesas,rspi-rz" will be used ... required. The ... > on RZ/G2L > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> > Reviewed-by: Biju Das <biju.das.jz@xxxxxxxxxxxxxx> Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > --- a/Documentation/devicetree/bindings/spi/renesas,rspi.yaml > +++ b/Documentation/devicetree/bindings/spi/renesas,rspi.yaml > @@ -21,7 +21,8 @@ properties: > - enum: > - renesas,rspi-r7s72100 # RZ/A1H > - renesas,rspi-r7s9210 # RZ/A2 > - - const: renesas,rspi-rz # RZ/A > + - renesas,r9a07g044-rspi # RZ/G2{L,LC} > + - const: renesas,rspi-rz # RZ/A and RZ/G2{L,LC} > > - items: > - enum: > @@ -116,6 +117,16 @@ allOf: > required: > - interrupt-names > > + - if: > + properties: > + compatible: > + contains: > + enum: > + - renesas,r9a07g044-rspi > + then: > + required: > + - resets > + You may want to merge this with the existing section that makes resets required for renesas,qspi. > - if: > properties: > compatible: 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