From: Jon Hunter <jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
To: Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>,
Thierry Reding
<thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
Alexandre Courbot
<gnurou-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: Stefan Agner <stefan-XLVq0VzYD2Y@public.gmane.org>,
Marc Dietrich <marvin24-Mmb7MZpHnFY@public.gmane.org>,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
Subject: Re: [PATCH 1/2] ARM: tegra: Add stdout-path for various boards
Date: Mon, 7 Mar 2016 19:22:27 +0000 [thread overview]
Message-ID: <56DDD4F3.2010109@nvidia.com> (raw)
In-Reply-To: <1455025920-21875-1-git-send-email-jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
Hi Thierry,
I forgot about these patches. Not urgent, but may be we can queue them
for v4.7.
Cheers
Jon
On 09/02/16 13:51, Jon Hunter wrote:
> For Tegra boards, the device-tree alias serial0 is used for the console
> and so add the stdout-path information so that the console no longer
> needs to be passed via the kernel boot parameters.
>
> This has been tested on boards, tegra20-trimslice, tegra30-beaver,
> tegra114-dalmore and tegra124-jetson-tk1.
>
> Signed-off-by: Jon Hunter <jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> ---
>
> The only boards that I have not modified are tegra114-roth and
> tegra114-tn7 because these boards have two entries for console
> in the *.dts file. AFAICT we cannot suppport multiple consoles
> via the stdout-path and if a console is specified by the kernel
> boot parameters then this sets the kernel variable
> "console_set_on_cmdline" and prevents the stdout-path console
> being registered. If there is a better way to handle these boards
> via device-tree let me know.
>
> arch/arm/boot/dts/tegra114-dalmore.dts | 4 ++++
> arch/arm/boot/dts/tegra124-jetson-tk1.dts | 4 ++++
> arch/arm/boot/dts/tegra124-nyan.dtsi | 4 ++++
> arch/arm/boot/dts/tegra124-venice2.dts | 4 ++++
> arch/arm/boot/dts/tegra20-harmony.dts | 4 ++++
> arch/arm/boot/dts/tegra20-iris-512.dts | 4 ++++
> arch/arm/boot/dts/tegra20-medcom-wide.dts | 4 ++++
> arch/arm/boot/dts/tegra20-paz00.dts | 4 ++++
> arch/arm/boot/dts/tegra20-seaboard.dts | 4 ++++
> arch/arm/boot/dts/tegra20-tamonten.dtsi | 4 ++++
> arch/arm/boot/dts/tegra20-trimslice.dts | 4 ++++
> arch/arm/boot/dts/tegra20-ventana.dts | 4 ++++
> arch/arm/boot/dts/tegra20-whistler.dts | 4 ++++
> arch/arm/boot/dts/tegra30-apalis-eval.dts | 4 ++++
> arch/arm/boot/dts/tegra30-beaver.dts | 4 ++++
> arch/arm/boot/dts/tegra30-cardhu.dtsi | 4 ++++
> arch/arm/boot/dts/tegra30-colibri-eval-v3.dts | 4 ++++
> 17 files changed, 68 insertions(+)
>
> diff --git a/arch/arm/boot/dts/tegra114-dalmore.dts b/arch/arm/boot/dts/tegra114-dalmore.dts
> index 8b7aa0dcdc6e..b5748eeed473 100644
> --- a/arch/arm/boot/dts/tegra114-dalmore.dts
> +++ b/arch/arm/boot/dts/tegra114-dalmore.dts
> @@ -18,6 +18,10 @@
> serial0 = &uartd;
> };
>
> + chosen {
> + stdout-path = "serial0:115200n8";
> + };
> +
> memory {
> reg = <0x80000000 0x40000000>;
> };
> diff --git a/arch/arm/boot/dts/tegra124-jetson-tk1.dts b/arch/arm/boot/dts/tegra124-jetson-tk1.dts
> index 4ee2e63e11d0..a7264a15080a 100644
> --- a/arch/arm/boot/dts/tegra124-jetson-tk1.dts
> +++ b/arch/arm/boot/dts/tegra124-jetson-tk1.dts
> @@ -19,6 +19,10 @@
> serial2 = &uartb;
> };
>
> + chosen {
> + stdout-path = "serial0:115200n8";
> + };
> +
> memory {
> reg = <0x0 0x80000000 0x0 0x80000000>;
> };
> diff --git a/arch/arm/boot/dts/tegra124-nyan.dtsi b/arch/arm/boot/dts/tegra124-nyan.dtsi
> index ec1aa64ded68..e2cd39e3c95e 100644
> --- a/arch/arm/boot/dts/tegra124-nyan.dtsi
> +++ b/arch/arm/boot/dts/tegra124-nyan.dtsi
> @@ -8,6 +8,10 @@
> serial0 = &uarta;
> };
>
> + chosen {
> + stdout-path = "serial0:115200n8";
> + };
> +
> memory {
> reg = <0x0 0x80000000 0x0 0x80000000>;
> };
> diff --git a/arch/arm/boot/dts/tegra124-venice2.dts b/arch/arm/boot/dts/tegra124-venice2.dts
> index cfbdf429b45d..604f4b7c2685 100644
> --- a/arch/arm/boot/dts/tegra124-venice2.dts
> +++ b/arch/arm/boot/dts/tegra124-venice2.dts
> @@ -13,6 +13,10 @@
> serial0 = &uarta;
> };
>
> + chosen {
> + stdout-path = "serial0:115200n8";
> + };
> +
> memory {
> reg = <0x0 0x80000000 0x0 0x80000000>;
> };
> diff --git a/arch/arm/boot/dts/tegra20-harmony.dts b/arch/arm/boot/dts/tegra20-harmony.dts
> index b926a07b9443..4b73c76f981a 100644
> --- a/arch/arm/boot/dts/tegra20-harmony.dts
> +++ b/arch/arm/boot/dts/tegra20-harmony.dts
> @@ -13,6 +13,10 @@
> serial0 = &uartd;
> };
>
> + chosen {
> + stdout-path = "serial0:115200n8";
> + };
> +
> memory {
> reg = <0x00000000 0x40000000>;
> };
> diff --git a/arch/arm/boot/dts/tegra20-iris-512.dts b/arch/arm/boot/dts/tegra20-iris-512.dts
> index 1dd7d7bfdfcc..bb56dfe9e10c 100644
> --- a/arch/arm/boot/dts/tegra20-iris-512.dts
> +++ b/arch/arm/boot/dts/tegra20-iris-512.dts
> @@ -11,6 +11,10 @@
> serial1 = &uartd;
> };
>
> + chosen {
> + stdout-path = "serial0:115200n8";
> + };
> +
> host1x@50000000 {
> hdmi@54280000 {
> status = "okay";
> diff --git a/arch/arm/boot/dts/tegra20-medcom-wide.dts b/arch/arm/boot/dts/tegra20-medcom-wide.dts
> index 9b87526ab0b7..34c6588e92ef 100644
> --- a/arch/arm/boot/dts/tegra20-medcom-wide.dts
> +++ b/arch/arm/boot/dts/tegra20-medcom-wide.dts
> @@ -10,6 +10,10 @@
> serial0 = &uartd;
> };
>
> + chosen {
> + stdout-path = "serial0:115200n8";
> + };
> +
> pwm@7000a000 {
> status = "okay";
> };
> diff --git a/arch/arm/boot/dts/tegra20-paz00.dts b/arch/arm/boot/dts/tegra20-paz00.dts
> index ed7e1009326c..81a10a9df814 100644
> --- a/arch/arm/boot/dts/tegra20-paz00.dts
> +++ b/arch/arm/boot/dts/tegra20-paz00.dts
> @@ -14,6 +14,10 @@
> serial1 = &uartc;
> };
>
> + chosen {
> + stdout-path = "serial0:115200n8";
> + };
> +
> memory {
> reg = <0x00000000 0x20000000>;
> };
> diff --git a/arch/arm/boot/dts/tegra20-seaboard.dts b/arch/arm/boot/dts/tegra20-seaboard.dts
> index aea8994b35f2..0aed748567aa 100644
> --- a/arch/arm/boot/dts/tegra20-seaboard.dts
> +++ b/arch/arm/boot/dts/tegra20-seaboard.dts
> @@ -13,6 +13,10 @@
> serial0 = &uartd;
> };
>
> + chosen {
> + stdout-path = "serial0:115200n8";
> + };
> +
> memory {
> reg = <0x00000000 0x40000000>;
> };
> diff --git a/arch/arm/boot/dts/tegra20-tamonten.dtsi b/arch/arm/boot/dts/tegra20-tamonten.dtsi
> index 13d4e6185275..025e9e8037da 100644
> --- a/arch/arm/boot/dts/tegra20-tamonten.dtsi
> +++ b/arch/arm/boot/dts/tegra20-tamonten.dtsi
> @@ -10,6 +10,10 @@
> serial0 = &uartd;
> };
>
> + chosen {
> + stdout-path = "serial0:115200n8";
> + };
> +
> memory {
> reg = <0x00000000 0x20000000>;
> };
> diff --git a/arch/arm/boot/dts/tegra20-trimslice.dts b/arch/arm/boot/dts/tegra20-trimslice.dts
> index d99af4ef9c64..69d25ca47e04 100644
> --- a/arch/arm/boot/dts/tegra20-trimslice.dts
> +++ b/arch/arm/boot/dts/tegra20-trimslice.dts
> @@ -13,6 +13,10 @@
> serial0 = &uarta;
> };
>
> + chosen {
> + stdout-path = "serial0:115200n8";
> + };
> +
> memory {
> reg = <0x00000000 0x40000000>;
> };
> diff --git a/arch/arm/boot/dts/tegra20-ventana.dts b/arch/arm/boot/dts/tegra20-ventana.dts
> index 04c58e9ca490..c61533a9bf0b 100644
> --- a/arch/arm/boot/dts/tegra20-ventana.dts
> +++ b/arch/arm/boot/dts/tegra20-ventana.dts
> @@ -13,6 +13,10 @@
> serial0 = &uartd;
> };
>
> + chosen {
> + stdout-path = "serial0:115200n8";
> + };
> +
> memory {
> reg = <0x00000000 0x40000000>;
> };
> diff --git a/arch/arm/boot/dts/tegra20-whistler.dts b/arch/arm/boot/dts/tegra20-whistler.dts
> index 340d81108df1..bd7658572239 100644
> --- a/arch/arm/boot/dts/tegra20-whistler.dts
> +++ b/arch/arm/boot/dts/tegra20-whistler.dts
> @@ -13,6 +13,10 @@
> serial0 = &uarta;
> };
>
> + chosen {
> + stdout-path = "serial0:115200n8";
> + };
> +
> memory {
> reg = <0x00000000 0x20000000>;
> };
> diff --git a/arch/arm/boot/dts/tegra30-apalis-eval.dts b/arch/arm/boot/dts/tegra30-apalis-eval.dts
> index f2879cfcca62..b914bcb381d7 100644
> --- a/arch/arm/boot/dts/tegra30-apalis-eval.dts
> +++ b/arch/arm/boot/dts/tegra30-apalis-eval.dts
> @@ -17,6 +17,10 @@
> serial3 = &uartd;
> };
>
> + chosen {
> + stdout-path = "serial0:115200n8";
> + };
> +
> pcie-controller@00003000 {
> status = "okay";
>
> diff --git a/arch/arm/boot/dts/tegra30-beaver.dts b/arch/arm/boot/dts/tegra30-beaver.dts
> index 3dede3934446..1eca3b28ac64 100644
> --- a/arch/arm/boot/dts/tegra30-beaver.dts
> +++ b/arch/arm/boot/dts/tegra30-beaver.dts
> @@ -12,6 +12,10 @@
> serial0 = &uarta;
> };
>
> + chosen {
> + stdout-path = "serial0:115200n8";
> + };
> +
> memory {
> reg = <0x80000000 0x7ff00000>;
> };
> diff --git a/arch/arm/boot/dts/tegra30-cardhu.dtsi b/arch/arm/boot/dts/tegra30-cardhu.dtsi
> index bb1ca158273c..de9d6ccace14 100644
> --- a/arch/arm/boot/dts/tegra30-cardhu.dtsi
> +++ b/arch/arm/boot/dts/tegra30-cardhu.dtsi
> @@ -35,6 +35,10 @@
> serial1 = &uartc;
> };
>
> + chosen {
> + stdout-path = "serial0:115200n8";
> + };
> +
> memory {
> reg = <0x80000000 0x40000000>;
> };
> diff --git a/arch/arm/boot/dts/tegra30-colibri-eval-v3.dts b/arch/arm/boot/dts/tegra30-colibri-eval-v3.dts
> index 3ff019f47d00..93e1ffd95bfa 100644
> --- a/arch/arm/boot/dts/tegra30-colibri-eval-v3.dts
> +++ b/arch/arm/boot/dts/tegra30-colibri-eval-v3.dts
> @@ -15,6 +15,10 @@
> serial2 = &uartd;
> };
>
> + chosen {
> + stdout-path = "serial0:115200n8";
> + };
> +
> host1x@50000000 {
> dc@54200000 {
> rgb {
>
next prev parent reply other threads:[~2016-03-07 19:22 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-09 13:51 [PATCH 1/2] ARM: tegra: Add stdout-path for various boards Jon Hunter
[not found] ` <1455025920-21875-1-git-send-email-jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2016-02-09 13:52 ` [PATCH 2/2] ARM64: " Jon Hunter
[not found] ` <1455025920-21875-2-git-send-email-jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2016-04-05 14:05 ` Thierry Reding
2016-02-09 17:00 ` [PATCH 1/2] ARM: " Stefan Agner
2016-02-10 0:46 ` Alexandre Courbot
2016-03-07 19:22 ` Jon Hunter [this message]
2016-04-05 14:04 ` Thierry Reding
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=56DDD4F3.2010109@nvidia.com \
--to=jonathanh-ddmlm1+adcrqt0dzr+alfa@public.gmane.org \
--cc=devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=gnurou-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
--cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=marvin24-Mmb7MZpHnFY@public.gmane.org \
--cc=stefan-XLVq0VzYD2Y@public.gmane.org \
--cc=swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org \
--cc=thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).