Hi Fabrizio, Chris, On Mon, Oct 9, 2017 at 11:06 AM, Fabrizio Castro <fabrizio.castro@xxxxxxxxxxxxxx> wrote: > Signed-off-by: Fabrizio Castro <fabrizio.castro@xxxxxxxxxxxxxx> > Signed-off-by: Chris Paterson <chris.paterson2@xxxxxxxxxxx> Thanks for your patch! Reviewed-by: Geert Uytterhoeven <geert+renesas@xxxxxxxxx> > --- a/arch/arm/boot/dts/iwg20d-q7-common.dtsi > +++ b/arch/arm/boot/dts/iwg20d-q7-common.dtsi > @@ -11,6 +11,7 @@ > / { > aliases { > serial0 = &scif0; > + serial3 = &scifb1; Given this port is not labeled "serial3", but called "data uart", you may want to add a comment to avoid confusion: serial3 = &scifb1; /* data uart */ > ethernet0 = &avb; > }; 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