Hi Ulrich, On Tue, Aug 16, 2016 at 4:47 PM, Ulrich Hecht <ulrich.hecht+renesas@xxxxxxxxx> wrote: > Adds pin control for SCIF2. > > Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@xxxxxxxxx> > --- > arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts > index e72be38..d4a774c 100644 > --- a/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts > +++ b/arch/arm64/boot/dts/renesas/r8a7796-salvator-x.dts > @@ -31,11 +31,20 @@ > }; > }; > > +&pfc { > + scif2_pins: scif2 { > + groups = "scif2_data_a", "scif2_clk"; > + function = "scif2"; > + }; > +}; I think you misunderstood my comment: I didn't mean to add "scif2_clk" (which is not used on Salvator-X), but "scif_clk", which is the external clock for all (H)SCIF instances, provided by 14.7456 MHz crystal X1 on Salvator-X. This is used to program the BRG in SCIF2 to exactly 115200 bps. 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