Hi Chris, On Tue, Oct 3, 2017 at 9:33 AM, Chris Brandt <chris.brandt@xxxxxxxxxxx> wrote: > Compatible "r7s72102-ports" was added for RZ/A1L devices > > Signed-off-by: Chris Brandt <chris.brandt@xxxxxxxxxxx> > --- > Documentation/devicetree/bindings/pinctrl/renesas,rza1-pinctrl.txt | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/pinctrl/renesas,rza1-pinctrl.txt b/Documentation/devicetree/bindings/pinctrl/renesas,rza1-pinctrl.txt > index 43e21474528a..2a3366cee848 100644 > --- a/Documentation/devicetree/bindings/pinctrl/renesas,rza1-pinctrl.txt > +++ b/Documentation/devicetree/bindings/pinctrl/renesas,rza1-pinctrl.txt > @@ -13,7 +13,8 @@ Pin controller node > > Required properties: > - compatible > - this shall be "renesas,r7s72100-ports". > + this shall be "renesas,r7s72100-ports" for RZ/A1H and RZ/A1M or > + "renesas,r7s72102-ports" for RZ/A1L Shouldn't you also document "renesas,r7s72101-ports" for RZ/A1M? You could use "renesas,r7s72100-ports" as a fallback property for RZ/A1M, though. BTW, is there a way to distinguish the various RZ/A1 SoCs at runtime? 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