Hi Laurent, On Mon, Jan 30, 2017 at 7:17 PM, Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> wrote: > On Thursday 26 Jan 2017 12:51:03 Sergei Shtylyov wrote: >> On 1/25/2017 9:09 PM, Jacopo Mondi wrote: >> > +&pinctrl { >> > + pinctrl-names = "default"; >> > + pinctrl-0 = <&scif2_pins>; >> > + >> > + scif2_pins: serial2 { >> > + /* P3_0 as TxD2; P3_2 as RxD2 */ >> > + renesas-rz,pins = <RZ_PIN(3, 0) ALTERNATE_FUNC_6>, >> >> The comma should be after the vendor prefix ("renesas"). > > And the "-rz" suffix isn't needed, "renesas,pins" will do. Then it can just become "pins"? Or not? 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 linux-gpio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html