Re: [PATCH 1/3] dt-bindings: pinctrl: add compatible for Renesas RZ/T2H

[Geert Uytterhoeven]

Hi Prabhakar,

On Mon, 2 Jun 2025 at 09:12, Lad, Prabhakar <prabhakar.csengg@xxxxxxxxx> wrote:
> On Mon, May 26, 2025 at 6:03 PM Geert Uytterhoeven <geert@xxxxxxxxxxxxxx> wrote:
> > On Mon, 19 May 2025 at 23:57, Thierry Bultel
> > <thierry.bultel.yh@xxxxxxxxxxxxxx> wrote:
> > > Document RZ/T2H (a.k.a r9a09g077) pinctrl
> > >
> > > Signed-off-by: Thierry Bultel <thierry.bultel.yh@xxxxxxxxxxxxxx>
> >
> > > --- a/Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml
> > > +++ b/Documentation/devicetree/bindings/pinctrl/renesas,pfc.yaml
> > > @@ -194,3 +209,13 @@ examples:
> > >                      power-source = <3300>;
> > >              };
> > >      };
> > > +
> > > +  - |
> > > +    pinctrl: pinctrl@812c0000 {
> >
> > The unit address does not match the first reg property.
> >
> > > +            compatible = "renesas,pfc-r9a09g077";
> > > +            reg = <0x802c0000 0x2000>,
> > > +                  <0x812c0000 0x2000>;
> > > +            gpio-controller;
> > > +            #gpio-cells = <2>;
> > > +            gpio-ranges = <&pinctrl 0 0 287>;
> >
> > GPIOs without interrupts?
> >
> I think the intention here was to go without interrupt support for the
> initial series and when later support for ICU is added we add this
> property. Hope thats OK?

Yes, that is fine.

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