[PATCH] ARM: shmobile: marzen: Board specific serial port order

[PATCH] ARM: shmobile: marzen: Board specific serial port order

[Magnus Damm]

From: Magnus Damm <damm+renesas@opensource.se>

Convert Marzen to use board-specific serial0 and serial1 instead
of the older shared SoC-specific SCIF index.

Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
---

 Cooked up against renesas-devel-20150807-v4.2-rc5

 arch/arm/boot/dts/r8a7779-marzen.dts |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

--- 0001/arch/arm/boot/dts/r8a7779-marzen.dts
+++ work/arch/arm/boot/dts/r8a7779-marzen.dts	2015-08-07 11:04:37.432366518 +0900
@@ -19,12 +19,12 @@
 	compatible = "renesas,marzen", "renesas,r8a7779";
 
 	aliases {
-		serial2 = &scif2;
-		serial4 = &scif4;
+		serial0 = &scif2;
+		serial1 = &scif4;
 	};
 
 	chosen {
-		bootargs = "console=ttySC2,115200 ignore_loglevel root=/dev/nfs ip=on";
+		bootargs = "ignore_loglevel root=/dev/nfs ip=on";
 		stdout-path = &scif2;
 	};
 

Re: [PATCH] ARM: shmobile: marzen: Board specific serial port order

[Geert Uytterhoeven]

Hi Magnus,

On Fri, Aug 7, 2015 at 4:45 AM, Magnus Damm <magnus.damm@gmail.com> wrote:
> From: Magnus Damm <damm+renesas@opensource.se>
>
> Convert Marzen to use board-specific serial0 and serial1 instead
> of the older shared SoC-specific SCIF index.

Thank you for your patch!

Please note that this is a user-visible change.
Hardcoded "ttySC2" in e.g. kernel boot command line and /etc/inittab must be
changed to ttySC0.

> Signed-off-by: Magnus Damm <damm+renesas@opensource.se>

Nevertheless:
Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>

> --- 0001/arch/arm/boot/dts/r8a7779-marzen.dts
> +++ work/arch/arm/boot/dts/r8a7779-marzen.dts   2015-08-07 11:04:37.432366518 +0900
> @@ -19,12 +19,12 @@
>         compatible = "renesas,marzen", "renesas,r8a7779";
>
>         aliases {
> -               serial2 = &scif2;
> -               serial4 = &scif4;
> +               serial0 = &scif2;
> +               serial1 = &scif4;
>         };
>
>         chosen {
> -               bootargs = "console=ttySC2,115200 ignore_loglevel root=/dev/nfs ip=on";

The "console=..." part can indeed be removed, now legacy marzen is gone.

> +               bootargs = "ignore_loglevel root=/dev/nfs ip=on";
>                 stdout-path = &scif2;
>         };




-- 
Gr{oetje,eeting}s,

                        Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@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

Re: [PATCH] ARM: shmobile: marzen: Board specific serial port order

[Laurent Pinchart]

On Friday 07 August 2015 09:13:08 Geert Uytterhoeven wrote:
> Hi Magnus,
> 
> On Fri, Aug 7, 2015 at 4:45 AM, Magnus Damm <magnus.damm@gmail.com> wrote:
> > From: Magnus Damm <damm+renesas@opensource.se>
> > 
> > Convert Marzen to use board-specific serial0 and serial1 instead
> > of the older shared SoC-specific SCIF index.
> 
> Thank you for your patch!
> 
> Please note that this is a user-visible change.
> Hardcoded "ttySC2" in e.g. kernel boot command line and /etc/inittab must be
> changed to ttySC0.
> 
> > Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
> 
> Nevertheless:
> Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>

And

Acked-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>

given how long I've been pushing for this :-)

> > --- 0001/arch/arm/boot/dts/r8a7779-marzen.dts
> > +++ work/arch/arm/boot/dts/r8a7779-marzen.dts   2015-08-07
> > 11:04:37.432366518 +0900 @@ -19,12 +19,12 @@
> >         compatible = "renesas,marzen", "renesas,r8a7779";
> >         
> >         aliases {
> > -               serial2 = &scif2;
> > -               serial4 = &scif4;
> > +               serial0 = &scif2;
> > +               serial1 = &scif4;
> >         };
> >         
> >         chosen {
> > -               bootargs = "console=ttySC2,115200 ignore_loglevel
> > root=/dev/nfs ip=on";
>
> The "console=..." part can indeed be removed, now legacy marzen is gone.
>
> > +               bootargs = "ignore_loglevel root=/dev/nfs ip=on";
> >                 stdout-path = &scif2;
> >         };

-- 
Regards,

Laurent Pinchart

Re: [PATCH] ARM: shmobile: marzen: Board specific serial port order

[Simon Horman]

On Fri, Aug 07, 2015 at 01:19:59PM +0300, Laurent Pinchart wrote:
> On Friday 07 August 2015 09:13:08 Geert Uytterhoeven wrote:
> > Hi Magnus,
> > 
> > On Fri, Aug 7, 2015 at 4:45 AM, Magnus Damm <magnus.damm@gmail.com> wrote:
> > > From: Magnus Damm <damm+renesas@opensource.se>
> > > 
> > > Convert Marzen to use board-specific serial0 and serial1 instead
> > > of the older shared SoC-specific SCIF index.
> > 
> > Thank you for your patch!
> > 
> > Please note that this is a user-visible change.
> > Hardcoded "ttySC2" in e.g. kernel boot command line and /etc/inittab must be
> > changed to ttySC0.
> > 
> > > Signed-off-by: Magnus Damm <damm+renesas@opensource.se>
> > 
> > Nevertheless:
> > Acked-by: Geert Uytterhoeven <geert+renesas@glider.be>
> 
> And
> 
> Acked-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> 
> given how long I've been pushing for this :-)

Thanks, I have queued this up for v4.4.