Hi Chris, On Wed, Oct 4, 2017 at 4:43 PM, Chris Brandt <Chris.Brandt@xxxxxxxxxxx> wrote: > To be clear: > > On Wednesday, October 04, 2017 1, Geert Uytterhoeven wrote: >> >> You could use "renesas,r7s72100-ports" as a fallback property for >> RZ/A1M, >> >> though. >> > >> > Do you mean in the DT? Or in pinctrl-rza1.c? >> >> I mean in DT, i.e.: >> >> pinctrl: pin-controller@fcfe3000 { >> compatible = "renesas,r7s72101-ports", "renesas,r7s72100- >> ports"; >> ... >> >> That way you don't have to extend the pinctrl driver. > > I will submit a v2 with: > > [ renesas,rza1-pinctrl.txt ] > > Required properties: > - compatible > this shall be "renesas,r7s72100-ports" for RZ/A1H, > "renesas,r7s72101-ports" for RZ/M or ... with a fallback for "renesas,r7s72100-ports, as RZ/A1M is compatible with RZ/A1H (and I assume RZ/A1H is the predecessor?). > "renesas,r7s72102-ports" for RZ/A1L > > > [ r7s72100.dtsi ] > pinctrl: pin-controller@fcfe3000 { > compatible = "renesas,r7s72101-ports", "renesas,r7s72100-ports"; I meant in r7s72101.dtsi (which we don't have!). 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