diff mbox

[1/4] ARM: dts: iwg22d: Use /dev/ttySC3 as debug console

Message ID 1507899796-7941-2-git-send-email-fabrizio.castro@bp.renesas.com (mailing list archive)
State New, archived
Headers show

Commit Message

Fabrizio Castro Oct. 13, 2017, 1:03 p.m. UTC
The BSP release from iWave uses /dev/ttySC3 as debug console, this patch
renames the alias accordingly for compatibility.

Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
Signed-off-by: Chris Paterson <chris.paterson2@renesas.com>
---
 arch/arm/boot/dts/r8a7745-iwg22d-sodimm.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Biju Das Oct. 16, 2017, 8:34 a.m. UTC | #1
Hi,

Looks like the chosen node "stdout-path = "serial0:115200n8";" is not updated with this patch.

So either drop the patch or fix the chosen node.

Regards,
Biju

> -----Original Message-----
> From: Fabrizio Castro [mailto:fabrizio.castro@bp.renesas.com]
> Sent: 13 October 2017 14:03
> To: Rob Herring <robh+dt@kernel.org>; Mark Rutland
> <mark.rutland@arm.com>; Russell King <linux@armlinux.org.uk>
> Cc: Simon Horman <horms@verge.net.au>; Magnus Damm
> <magnus.damm@gmail.com>; Geert Uytterhoeven <geert@linux-m68k.org>;
> devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org; linux-
> renesas-soc@vger.kernel.org; Chris Paterson <Chris.Paterson2@renesas.com>;
> Biju Das <biju.das@bp.renesas.com>; Fabrizio Castro
> <fabrizio.castro@bp.renesas.com>; Chris Paterson
> <Chris.Paterson2@renesas.com>
> Subject: [PATCH 1/4] ARM: dts: iwg22d: Use /dev/ttySC3 as debug console
>
> The BSP release from iWave uses /dev/ttySC3 as debug console, this patch
> renames the alias accordingly for compatibility.
>
> Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com>
> Signed-off-by: Chris Paterson <chris.paterson2@renesas.com>
> ---
>  arch/arm/boot/dts/r8a7745-iwg22d-sodimm.dts | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/boot/dts/r8a7745-iwg22d-sodimm.dts
> b/arch/arm/boot/dts/r8a7745-iwg22d-sodimm.dts
> index 52153ec..edda4f1 100644
> --- a/arch/arm/boot/dts/r8a7745-iwg22d-sodimm.dts
> +++ b/arch/arm/boot/dts/r8a7745-iwg22d-sodimm.dts
> @@ -16,7 +16,7 @@
>  compatible = "iwave,g22d", "iwave,g22m", "renesas,r8a7745";
>
>  aliases {
> -serial0 = &scif4;
> +serial3 = &scif4;
>  ethernet0 = &avb;
>  };
>
> --
> 2.7.4




Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.
Simon Horman Oct. 17, 2017, 7:59 a.m. UTC | #2
On Mon, Oct 16, 2017 at 08:34:35AM +0000, Biju Das wrote:
> Hi,
> 
> Looks like the chosen node "stdout-path = "serial0:115200n8";" is not updated with this patch.
> 
> So either drop the patch or fix the chosen node.

Please repost this patch with the above resolved.
Fabrizio Castro Oct. 23, 2017, 11:01 a.m. UTC | #3
Hello Simon,

> > Hi,
> >
> > Looks like the chosen node "stdout-path = "serial0:115200n8";" is not updated with this patch.
> >
> > So either drop the patch or fix the chosen node.
>
> Please repost this patch with the above resolved.

I'll repost.

Best regards,
Fab



Renesas Electronics Europe Ltd, Dukes Meadow, Millboard Road, Bourne End, Buckinghamshire, SL8 5FH, UK. Registered in England & Wales under Registered No. 04586709.
diff mbox

Patch

diff --git a/arch/arm/boot/dts/r8a7745-iwg22d-sodimm.dts b/arch/arm/boot/dts/r8a7745-iwg22d-sodimm.dts
index 52153ec..edda4f1 100644
--- a/arch/arm/boot/dts/r8a7745-iwg22d-sodimm.dts
+++ b/arch/arm/boot/dts/r8a7745-iwg22d-sodimm.dts
@@ -16,7 +16,7 @@ 
 	compatible = "iwave,g22d", "iwave,g22m", "renesas,r8a7745";
 
 	aliases {
-		serial0 = &scif4;
+		serial3 = &scif4;
 		ethernet0 = &avb;
 	};