Hi Fabrizio, Chris,

On Mon, Oct 9, 2017 at 11:06 AM, Fabrizio Castro
<fabrizio.cas...@bp.renesas.com> wrote:
> Signed-off-by: Fabrizio Castro <fabrizio.cas...@bp.renesas.com>
> Signed-off-by: Chris Paterson <chris.paters...@renesas.com>

Thanks for your patch!

Reviewed-by: Geert Uytterhoeven <geert+rene...@glider.be>

> --- 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 -- ge...@linux-m68k.org

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

Reply via email to