Hi Chris, On Wed, Oct 4, 2017 at 6:31 PM, Chris Brandt <chris.brandt@xxxxxxxxxxx> wrote: > Describe how to specify RZ/A1M and RZ/A1L devices. > > Signed-off-by: Chris Brandt <chris.brandt@xxxxxxxxxxx> > --- > v2: > * Added description for RZ/A1M > --- > Documentation/devicetree/bindings/pinctrl/renesas,rza1-pinctrl.txt | 4 +++- > 1 file changed, 3 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..f2fc86f8dbab 100644 > --- a/Documentation/devicetree/bindings/pinctrl/renesas,rza1-pinctrl.txt > +++ b/Documentation/devicetree/bindings/pinctrl/renesas,rza1-pinctrl.txt > @@ -13,7 +13,9 @@ Pin controller node > > Required properties: > - compatible > - this shall be "renesas,r7s72100-ports". > + this shall be "renesas,r7s72100-ports" for RZ/A1H, "renesas,r7s72101-ports" > + with a fallback of "renesas,r7s72100-ports" for RZ/A1M (as A1M is compatible > + with A1H), or "renesas,r7s72102-ports" for RZ/A1L Sorry to bother you again, but having this in a single long sentence is hard to read. What about splitting per SoC, like is done in other bindings? For the actual content: Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> 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 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html