diff mbox

ARM: shmobile: lager dts: Drop console= bootargs parameter

Message ID 1417541988-28799-1-git-send-email-geert+renesas@glider.be (mailing list archive)
State Accepted
Commit 569dd56c9971c699472d3b270e988baf7f6de8c9
Delegated to: Simon Horman
Headers show

Commit Message

Geert Uytterhoeven Dec. 2, 2014, 5:39 p.m. UTC
Since commit FIXME ("ARM: shmobile: lager: Remove legacy board support"),
lager is restricted to booting from DT, so chosen/stdout-path is always
used, and we can drop the "console=" parameter from chosen/bootargs.

Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
This depends on "[PATCH 2/5] ARM: shmobile: lager: Remove legacy board
support" by Laurent Pinchart, and allows to simplify his "[PATCH 4/5] ARM:
shmobile: lager: Rename SCIFA[01] serial ports to ttySC[01]".
---
 arch/arm/boot/dts/r8a7790-lager.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Laurent Pinchart Dec. 2, 2014, 8:52 p.m. UTC | #1
Hi Geert,

Thank you for the patch.

On Tuesday 02 December 2014 18:39:48 Geert Uytterhoeven wrote:
> Since commit FIXME ("ARM: shmobile: lager: Remove legacy board support"),
> lager is restricted to booting from DT, so chosen/stdout-path is always
> used, and we can drop the "console=" parameter from chosen/bootargs.
> 
> Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>

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

> ---
> This depends on "[PATCH 2/5] ARM: shmobile: lager: Remove legacy board
> support" by Laurent Pinchart, and allows to simplify his "[PATCH 4/5] ARM:
> shmobile: lager: Rename SCIFA[01] serial ports to ttySC[01]".

Does it imply your ack on the series ? :-)

> ---
>  arch/arm/boot/dts/r8a7790-lager.dts | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/arm/boot/dts/r8a7790-lager.dts
> b/arch/arm/boot/dts/r8a7790-lager.dts index
> 636d53bb87a27062..7301d029b39d5d9b 100644
> --- a/arch/arm/boot/dts/r8a7790-lager.dts
> +++ b/arch/arm/boot/dts/r8a7790-lager.dts
> @@ -52,7 +52,7 @@
>  	};
> 
>  	chosen {
> -		bootargs = "console=ttySC6,115200 ignore_loglevel rw root=/dev/nfs
> ip=dhcp"; +		bootargs = "ignore_loglevel rw root=/dev/nfs ip=dhcp";
>  		stdout-path = &scifa0;
>  	};
Simon Horman Dec. 4, 2014, 12:20 p.m. UTC | #2
On Tue, Dec 02, 2014 at 10:52:48PM +0200, Laurent Pinchart wrote:
> Hi Geert,
> 
> Thank you for the patch.
> 
> On Tuesday 02 December 2014 18:39:48 Geert Uytterhoeven wrote:
> > Since commit FIXME ("ARM: shmobile: lager: Remove legacy board support"),
> > lager is restricted to booting from DT, so chosen/stdout-path is always
> > used, and we can drop the "console=" parameter from chosen/bootargs.
> > 
> > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be>
> 
> Acked-by: Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>
> 
> > ---
> > This depends on "[PATCH 2/5] ARM: shmobile: lager: Remove legacy board
> > support" by Laurent Pinchart, and allows to simplify his "[PATCH 4/5] ARM:
> > shmobile: lager: Rename SCIFA[01] serial ports to ttySC[01]".
> 
> Does it imply your ack on the series ? :-)

I have queued up the version of this patch which Laurent rebased.
--
To unsubscribe from this list: send the line "unsubscribe linux-sh" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/arm/boot/dts/r8a7790-lager.dts b/arch/arm/boot/dts/r8a7790-lager.dts
index 636d53bb87a27062..7301d029b39d5d9b 100644
--- a/arch/arm/boot/dts/r8a7790-lager.dts
+++ b/arch/arm/boot/dts/r8a7790-lager.dts
@@ -52,7 +52,7 @@ 
 	};
 
 	chosen {
-		bootargs = "console=ttySC6,115200 ignore_loglevel rw root=/dev/nfs ip=dhcp";
+		bootargs = "ignore_loglevel rw root=/dev/nfs ip=dhcp";
 		stdout-path = &scifa0;
 	};