* [RFC PATCH] arm: dts: vexpress: Set chosen 'stdout-path'
@ 2021-09-24 21:48 Rob Herring
2021-09-29 15:31 ` Liviu Dudau
0 siblings, 1 reply; 4+ messages in thread
From: Rob Herring @ 2021-09-24 21:48 UTC (permalink / raw)
To: Sudeep Holla; +Cc: devicetree, Liviu Dudau, Lorenzo Pieralisi, linux-arm-kernel
Using chosen 'stdout-path' is the preferred way to set the console, but
several Arm, Ltd. platforms don't set it. Set it to the 1st serial port.
Cc: Liviu Dudau <liviu.dudau@arm.com>
Cc: Sudeep Holla <sudeep.holla@arm.com>
Cc: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
Cc: linux-arm-kernel@lists.infradead.org
Signed-off-by: Rob Herring <robh@kernel.org>
---
RFC because I don't have any idea what the baud rate is for real boards.
If anyone knows, please comment and I'll add it.
arch/arm/boot/dts/vexpress-v2p-ca15-tc1.dts | 4 +++-
arch/arm/boot/dts/vexpress-v2p-ca15_a7.dts | 4 +++-
arch/arm/boot/dts/vexpress-v2p-ca5s.dts | 4 +++-
arch/arm/boot/dts/vexpress-v2p-ca9.dts | 4 +++-
arch/arm64/boot/dts/arm/foundation-v8.dtsi | 4 +++-
arch/arm64/boot/dts/arm/fvp-base-revc.dts | 4 +++-
arch/arm64/boot/dts/arm/rtsm_ve-aemv8a.dts | 4 +++-
7 files changed, 21 insertions(+), 7 deletions(-)
diff --git a/arch/arm/boot/dts/vexpress-v2p-ca15-tc1.dts b/arch/arm/boot/dts/vexpress-v2p-ca15-tc1.dts
index 679537e17ff5..c296a3677447 100644
--- a/arch/arm/boot/dts/vexpress-v2p-ca15-tc1.dts
+++ b/arch/arm/boot/dts/vexpress-v2p-ca15-tc1.dts
@@ -20,7 +20,9 @@ / {
#address-cells = <2>;
#size-cells = <2>;
- chosen { };
+ chosen {
+ stdout-path = "serial0";
+ };
aliases {
serial0 = &v2m_serial0;
diff --git a/arch/arm/boot/dts/vexpress-v2p-ca15_a7.dts b/arch/arm/boot/dts/vexpress-v2p-ca15_a7.dts
index 511e87cc2bc5..7b7c1ec2f542 100644
--- a/arch/arm/boot/dts/vexpress-v2p-ca15_a7.dts
+++ b/arch/arm/boot/dts/vexpress-v2p-ca15_a7.dts
@@ -20,7 +20,9 @@ / {
#address-cells = <2>;
#size-cells = <2>;
- chosen { };
+ chosen {
+ stdout-path = "serial0";
+ };
aliases {
serial0 = &v2m_serial0;
diff --git a/arch/arm/boot/dts/vexpress-v2p-ca5s.dts b/arch/arm/boot/dts/vexpress-v2p-ca5s.dts
index 3b88209bacea..9b70ae1d3e35 100644
--- a/arch/arm/boot/dts/vexpress-v2p-ca5s.dts
+++ b/arch/arm/boot/dts/vexpress-v2p-ca5s.dts
@@ -20,7 +20,9 @@ / {
#address-cells = <1>;
#size-cells = <1>;
- chosen { };
+ chosen {
+ stdout-path = "serial0";
+ };
aliases {
serial0 = &v2m_serial0;
diff --git a/arch/arm/boot/dts/vexpress-v2p-ca9.dts b/arch/arm/boot/dts/vexpress-v2p-ca9.dts
index 5916e4877eac..4eef3db16b3d 100644
--- a/arch/arm/boot/dts/vexpress-v2p-ca9.dts
+++ b/arch/arm/boot/dts/vexpress-v2p-ca9.dts
@@ -20,7 +20,9 @@ / {
#address-cells = <1>;
#size-cells = <1>;
- chosen { };
+ chosen {
+ stdout-path = "serial0";
+ };
aliases {
serial0 = &v2m_serial0;
diff --git a/arch/arm64/boot/dts/arm/foundation-v8.dtsi b/arch/arm64/boot/dts/arm/foundation-v8.dtsi
index fbf13f7c2baf..6eb14717cb4d 100644
--- a/arch/arm64/boot/dts/arm/foundation-v8.dtsi
+++ b/arch/arm64/boot/dts/arm/foundation-v8.dtsi
@@ -18,7 +18,9 @@ / {
#address-cells = <2>;
#size-cells = <2>;
- chosen { };
+ chosen {
+ stdout-path = "serial0";
+ };
aliases {
serial0 = &v2m_serial0;
diff --git a/arch/arm64/boot/dts/arm/fvp-base-revc.dts b/arch/arm64/boot/dts/arm/fvp-base-revc.dts
index 5f519affe728..a6165a472a7b 100644
--- a/arch/arm64/boot/dts/arm/fvp-base-revc.dts
+++ b/arch/arm64/boot/dts/arm/fvp-base-revc.dts
@@ -24,7 +24,9 @@ / {
#address-cells = <2>;
#size-cells = <2>;
- chosen { };
+ chosen {
+ stdout-path = "serial0";
+ };
aliases {
serial0 = &v2m_serial0;
diff --git a/arch/arm64/boot/dts/arm/rtsm_ve-aemv8a.dts b/arch/arm64/boot/dts/arm/rtsm_ve-aemv8a.dts
index 258991ad7cc0..f8dee952cefc 100644
--- a/arch/arm64/boot/dts/arm/rtsm_ve-aemv8a.dts
+++ b/arch/arm64/boot/dts/arm/rtsm_ve-aemv8a.dts
@@ -23,7 +23,9 @@ / {
#address-cells = <2>;
#size-cells = <2>;
- chosen { };
+ chosen {
+ stdout-path = "serial0";
+ };
aliases {
serial0 = &v2m_serial0;
--
2.30.2
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [RFC PATCH] arm: dts: vexpress: Set chosen 'stdout-path'
2021-09-24 21:48 [RFC PATCH] arm: dts: vexpress: Set chosen 'stdout-path' Rob Herring
@ 2021-09-29 15:31 ` Liviu Dudau
2021-10-01 9:25 ` Sudeep Holla
0 siblings, 1 reply; 4+ messages in thread
From: Liviu Dudau @ 2021-09-29 15:31 UTC (permalink / raw)
To: Rob Herring; +Cc: Sudeep Holla, devicetree, Lorenzo Pieralisi, linux-arm-kernel
Hi Rob,
On Fri, Sep 24, 2021 at 04:48:44PM -0500, Rob Herring wrote:
> Using chosen 'stdout-path' is the preferred way to set the console, but
> several Arm, Ltd. platforms don't set it. Set it to the 1st serial port.
>
> Cc: Liviu Dudau <liviu.dudau@arm.com>
> Cc: Sudeep Holla <sudeep.holla@arm.com>
> Cc: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
> Cc: linux-arm-kernel@lists.infradead.org
> Signed-off-by: Rob Herring <robh@kernel.org>
> ---
> RFC because I don't have any idea what the baud rate is for real boards.
> If anyone knows, please comment and I'll add it.
I believe all of them default to 115200,8n1 but I'm only sure about the first two
boards.
Best regards,
Liviu
>
> arch/arm/boot/dts/vexpress-v2p-ca15-tc1.dts | 4 +++-
> arch/arm/boot/dts/vexpress-v2p-ca15_a7.dts | 4 +++-
> arch/arm/boot/dts/vexpress-v2p-ca5s.dts | 4 +++-
> arch/arm/boot/dts/vexpress-v2p-ca9.dts | 4 +++-
> arch/arm64/boot/dts/arm/foundation-v8.dtsi | 4 +++-
> arch/arm64/boot/dts/arm/fvp-base-revc.dts | 4 +++-
> arch/arm64/boot/dts/arm/rtsm_ve-aemv8a.dts | 4 +++-
> 7 files changed, 21 insertions(+), 7 deletions(-)
>
> diff --git a/arch/arm/boot/dts/vexpress-v2p-ca15-tc1.dts b/arch/arm/boot/dts/vexpress-v2p-ca15-tc1.dts
> index 679537e17ff5..c296a3677447 100644
> --- a/arch/arm/boot/dts/vexpress-v2p-ca15-tc1.dts
> +++ b/arch/arm/boot/dts/vexpress-v2p-ca15-tc1.dts
> @@ -20,7 +20,9 @@ / {
> #address-cells = <2>;
> #size-cells = <2>;
>
> - chosen { };
> + chosen {
> + stdout-path = "serial0";
> + };
>
> aliases {
> serial0 = &v2m_serial0;
> diff --git a/arch/arm/boot/dts/vexpress-v2p-ca15_a7.dts b/arch/arm/boot/dts/vexpress-v2p-ca15_a7.dts
> index 511e87cc2bc5..7b7c1ec2f542 100644
> --- a/arch/arm/boot/dts/vexpress-v2p-ca15_a7.dts
> +++ b/arch/arm/boot/dts/vexpress-v2p-ca15_a7.dts
> @@ -20,7 +20,9 @@ / {
> #address-cells = <2>;
> #size-cells = <2>;
>
> - chosen { };
> + chosen {
> + stdout-path = "serial0";
> + };
>
> aliases {
> serial0 = &v2m_serial0;
> diff --git a/arch/arm/boot/dts/vexpress-v2p-ca5s.dts b/arch/arm/boot/dts/vexpress-v2p-ca5s.dts
> index 3b88209bacea..9b70ae1d3e35 100644
> --- a/arch/arm/boot/dts/vexpress-v2p-ca5s.dts
> +++ b/arch/arm/boot/dts/vexpress-v2p-ca5s.dts
> @@ -20,7 +20,9 @@ / {
> #address-cells = <1>;
> #size-cells = <1>;
>
> - chosen { };
> + chosen {
> + stdout-path = "serial0";
> + };
>
> aliases {
> serial0 = &v2m_serial0;
> diff --git a/arch/arm/boot/dts/vexpress-v2p-ca9.dts b/arch/arm/boot/dts/vexpress-v2p-ca9.dts
> index 5916e4877eac..4eef3db16b3d 100644
> --- a/arch/arm/boot/dts/vexpress-v2p-ca9.dts
> +++ b/arch/arm/boot/dts/vexpress-v2p-ca9.dts
> @@ -20,7 +20,9 @@ / {
> #address-cells = <1>;
> #size-cells = <1>;
>
> - chosen { };
> + chosen {
> + stdout-path = "serial0";
> + };
>
> aliases {
> serial0 = &v2m_serial0;
> diff --git a/arch/arm64/boot/dts/arm/foundation-v8.dtsi b/arch/arm64/boot/dts/arm/foundation-v8.dtsi
> index fbf13f7c2baf..6eb14717cb4d 100644
> --- a/arch/arm64/boot/dts/arm/foundation-v8.dtsi
> +++ b/arch/arm64/boot/dts/arm/foundation-v8.dtsi
> @@ -18,7 +18,9 @@ / {
> #address-cells = <2>;
> #size-cells = <2>;
>
> - chosen { };
> + chosen {
> + stdout-path = "serial0";
> + };
>
> aliases {
> serial0 = &v2m_serial0;
> diff --git a/arch/arm64/boot/dts/arm/fvp-base-revc.dts b/arch/arm64/boot/dts/arm/fvp-base-revc.dts
> index 5f519affe728..a6165a472a7b 100644
> --- a/arch/arm64/boot/dts/arm/fvp-base-revc.dts
> +++ b/arch/arm64/boot/dts/arm/fvp-base-revc.dts
> @@ -24,7 +24,9 @@ / {
> #address-cells = <2>;
> #size-cells = <2>;
>
> - chosen { };
> + chosen {
> + stdout-path = "serial0";
> + };
>
> aliases {
> serial0 = &v2m_serial0;
> diff --git a/arch/arm64/boot/dts/arm/rtsm_ve-aemv8a.dts b/arch/arm64/boot/dts/arm/rtsm_ve-aemv8a.dts
> index 258991ad7cc0..f8dee952cefc 100644
> --- a/arch/arm64/boot/dts/arm/rtsm_ve-aemv8a.dts
> +++ b/arch/arm64/boot/dts/arm/rtsm_ve-aemv8a.dts
> @@ -23,7 +23,9 @@ / {
> #address-cells = <2>;
> #size-cells = <2>;
>
> - chosen { };
> + chosen {
> + stdout-path = "serial0";
> + };
>
> aliases {
> serial0 = &v2m_serial0;
> --
> 2.30.2
>
--
====================
| I would like to |
| fix the world, |
| but they're not |
| giving me the |
\ source code! /
---------------
¯\_(ツ)_/¯
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [RFC PATCH] arm: dts: vexpress: Set chosen 'stdout-path'
2021-09-29 15:31 ` Liviu Dudau
@ 2021-10-01 9:25 ` Sudeep Holla
2021-10-01 16:31 ` Liviu Dudau
0 siblings, 1 reply; 4+ messages in thread
From: Sudeep Holla @ 2021-10-01 9:25 UTC (permalink / raw)
To: Liviu Dudau
Cc: Rob Herring, devicetree, Lorenzo Pieralisi, Sudeep Holla,
linux-arm-kernel
On Wed, Sep 29, 2021 at 04:31:59PM +0100, Liviu Dudau wrote:
> Hi Rob,
>
> On Fri, Sep 24, 2021 at 04:48:44PM -0500, Rob Herring wrote:
> > Using chosen 'stdout-path' is the preferred way to set the console, but
> > several Arm, Ltd. platforms don't set it. Set it to the 1st serial port.
> >
> > Cc: Liviu Dudau <liviu.dudau@arm.com>
> > Cc: Sudeep Holla <sudeep.holla@arm.com>
> > Cc: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
> > Cc: linux-arm-kernel@lists.infradead.org
> > Signed-off-by: Rob Herring <robh@kernel.org>
> > ---
> > RFC because I don't have any idea what the baud rate is for real boards.
> > If anyone knows, please comment and I'll add it.
>
> I believe all of them default to 115200,8n1 but I'm only sure about the first two
> boards.
>
Sorry to contradict, but I ran TC2 this Monday with 38400 baud. So the first
four must be 38400, as the uart is in the MB IOFPGA and we just change the
coretile DB to get the first 4 platforms. Juno and N1SDP have SoC uart and
they are configured by default to 115200 IIUC.
--
Regards,
Sudeep
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [RFC PATCH] arm: dts: vexpress: Set chosen 'stdout-path'
2021-10-01 9:25 ` Sudeep Holla
@ 2021-10-01 16:31 ` Liviu Dudau
0 siblings, 0 replies; 4+ messages in thread
From: Liviu Dudau @ 2021-10-01 16:31 UTC (permalink / raw)
To: Sudeep Holla; +Cc: Rob Herring, devicetree, Lorenzo Pieralisi, linux-arm-kernel
On Fri, Oct 01, 2021 at 10:25:05AM +0100, Sudeep Holla wrote:
> On Wed, Sep 29, 2021 at 04:31:59PM +0100, Liviu Dudau wrote:
> > Hi Rob,
> >
> > On Fri, Sep 24, 2021 at 04:48:44PM -0500, Rob Herring wrote:
> > > Using chosen 'stdout-path' is the preferred way to set the console, but
> > > several Arm, Ltd. platforms don't set it. Set it to the 1st serial port.
> > >
> > > Cc: Liviu Dudau <liviu.dudau@arm.com>
> > > Cc: Sudeep Holla <sudeep.holla@arm.com>
> > > Cc: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
> > > Cc: linux-arm-kernel@lists.infradead.org
> > > Signed-off-by: Rob Herring <robh@kernel.org>
> > > ---
> > > RFC because I don't have any idea what the baud rate is for real boards.
> > > If anyone knows, please comment and I'll add it.
> >
> > I believe all of them default to 115200,8n1 but I'm only sure about the first two
> > boards.
> >
>
> Sorry to contradict, but I ran TC2 this Monday with 38400 baud. So the first
> four must be 38400, as the uart is in the MB IOFPGA and we just change the
> coretile DB to get the first 4 platforms. Juno and N1SDP have SoC uart and
> they are configured by default to 115200 IIUC.
Huh, you're right! I guess I was trying to cover some painful memories :)
Sorry for the noise Rob, please listen to Sudeep.
Best regards,
Liviu
>
> --
> Regards,
> Sudeep
--
====================
| I would like to |
| fix the world, |
| but they're not |
| giving me the |
\ source code! /
---------------
¯\_(ツ)_/¯
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-10-01 16:31 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-09-24 21:48 [RFC PATCH] arm: dts: vexpress: Set chosen 'stdout-path' Rob Herring
2021-09-29 15:31 ` Liviu Dudau
2021-10-01 9:25 ` Sudeep Holla
2021-10-01 16:31 ` Liviu Dudau
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).