* [PATCH] arm64: tegra: Update console for Jetson Xavier and Orin
@ 2022-10-28 12:35 Jon Hunter
2022-10-28 14:17 ` Thierry Reding
0 siblings, 1 reply; 3+ messages in thread
From: Jon Hunter @ 2022-10-28 12:35 UTC (permalink / raw)
To: Rob Herring, Krzysztof Kozlowski, Thierry Reding
Cc: devicetree, linux-tegra, Jon Hunter
The Tegra Combined UART (TCU) is the default serial interface for Jetson
Xavier and Orin platforms and so update the bootargs for these platforms
to use the TCU.
Signed-off-by: Jon Hunter <jonathanh@nvidia.com>
---
arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi | 2 +-
arch/arm64/boot/dts/nvidia/tegra194-p3668.dtsi | 2 +-
arch/arm64/boot/dts/nvidia/tegra234-p3737-0000+p3701-0000.dts | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi b/arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi
index 3837ebc67c8e..ccdb32c67861 100644
--- a/arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi
+++ b/arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi
@@ -23,7 +23,7 @@ aliases {
};
chosen {
- bootargs = "console=ttyS0,115200n8";
+ bootargs = "console=ttyTCU0,115200n8";
stdout-path = "serial0:115200n8";
};
diff --git a/arch/arm64/boot/dts/nvidia/tegra194-p3668.dtsi b/arch/arm64/boot/dts/nvidia/tegra194-p3668.dtsi
index 916ea3419ee5..f147324d72f3 100644
--- a/arch/arm64/boot/dts/nvidia/tegra194-p3668.dtsi
+++ b/arch/arm64/boot/dts/nvidia/tegra194-p3668.dtsi
@@ -20,7 +20,7 @@ aliases {
};
chosen {
- bootargs = "console=ttyS0,115200n8";
+ bootargs = "console=ttyTCU0,115200n8";
stdout-path = "serial0:115200n8";
};
diff --git a/arch/arm64/boot/dts/nvidia/tegra234-p3737-0000+p3701-0000.dts b/arch/arm64/boot/dts/nvidia/tegra234-p3737-0000+p3701-0000.dts
index df703fb0cfff..96aa2267b06d 100644
--- a/arch/arm64/boot/dts/nvidia/tegra234-p3737-0000+p3701-0000.dts
+++ b/arch/arm64/boot/dts/nvidia/tegra234-p3737-0000+p3701-0000.dts
@@ -2025,7 +2025,7 @@ hda@3510000 {
};
chosen {
- bootargs = "console=ttyS0,115200n8";
+ bootargs = "console=ttyTCU0,115200n8";
stdout-path = "serial0:115200n8";
};
--
2.25.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] arm64: tegra: Update console for Jetson Xavier and Orin
2022-10-28 12:35 [PATCH] arm64: tegra: Update console for Jetson Xavier and Orin Jon Hunter
@ 2022-10-28 14:17 ` Thierry Reding
2022-10-28 14:45 ` Jon Hunter
0 siblings, 1 reply; 3+ messages in thread
From: Thierry Reding @ 2022-10-28 14:17 UTC (permalink / raw)
To: Jon Hunter; +Cc: Rob Herring, Krzysztof Kozlowski, devicetree, linux-tegra
[-- Attachment #1: Type: text/plain, Size: 2654 bytes --]
On Fri, Oct 28, 2022 at 01:35:56PM +0100, Jon Hunter wrote:
> The Tegra Combined UART (TCU) is the default serial interface for Jetson
> Xavier and Orin platforms and so update the bootargs for these platforms
> to use the TCU.
>
> Signed-off-by: Jon Hunter <jonathanh@nvidia.com>
> ---
> arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi | 2 +-
> arch/arm64/boot/dts/nvidia/tegra194-p3668.dtsi | 2 +-
> arch/arm64/boot/dts/nvidia/tegra234-p3737-0000+p3701-0000.dts | 2 +-
> 3 files changed, 3 insertions(+), 3 deletions(-)
I'm a little confused now. You're certainly right that the TCU is the
serial interface, but then why haven't we seen any fallout from this?
That is, why does console=ttyS0,115200n8 still work just fine?
Looking a little more into stdout-path (which we already point to the
TCU), that ends up adding TCU as the preferred console already. So I
wonder if we really need to set the console here at all.
Do you see any difference in behavior after this patch? Does it fix
anything? Could you perhaps try to remove this option altogether and see
if that makes a difference?
Thierry
>
> diff --git a/arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi b/arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi
> index 3837ebc67c8e..ccdb32c67861 100644
> --- a/arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi
> +++ b/arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi
> @@ -23,7 +23,7 @@ aliases {
> };
>
> chosen {
> - bootargs = "console=ttyS0,115200n8";
> + bootargs = "console=ttyTCU0,115200n8";
> stdout-path = "serial0:115200n8";
> };
>
> diff --git a/arch/arm64/boot/dts/nvidia/tegra194-p3668.dtsi b/arch/arm64/boot/dts/nvidia/tegra194-p3668.dtsi
> index 916ea3419ee5..f147324d72f3 100644
> --- a/arch/arm64/boot/dts/nvidia/tegra194-p3668.dtsi
> +++ b/arch/arm64/boot/dts/nvidia/tegra194-p3668.dtsi
> @@ -20,7 +20,7 @@ aliases {
> };
>
> chosen {
> - bootargs = "console=ttyS0,115200n8";
> + bootargs = "console=ttyTCU0,115200n8";
> stdout-path = "serial0:115200n8";
> };
>
> diff --git a/arch/arm64/boot/dts/nvidia/tegra234-p3737-0000+p3701-0000.dts b/arch/arm64/boot/dts/nvidia/tegra234-p3737-0000+p3701-0000.dts
> index df703fb0cfff..96aa2267b06d 100644
> --- a/arch/arm64/boot/dts/nvidia/tegra234-p3737-0000+p3701-0000.dts
> +++ b/arch/arm64/boot/dts/nvidia/tegra234-p3737-0000+p3701-0000.dts
> @@ -2025,7 +2025,7 @@ hda@3510000 {
> };
>
> chosen {
> - bootargs = "console=ttyS0,115200n8";
> + bootargs = "console=ttyTCU0,115200n8";
> stdout-path = "serial0:115200n8";
> };
>
> --
> 2.25.1
>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] arm64: tegra: Update console for Jetson Xavier and Orin
2022-10-28 14:17 ` Thierry Reding
@ 2022-10-28 14:45 ` Jon Hunter
0 siblings, 0 replies; 3+ messages in thread
From: Jon Hunter @ 2022-10-28 14:45 UTC (permalink / raw)
To: Thierry Reding; +Cc: Rob Herring, Krzysztof Kozlowski, devicetree, linux-tegra
On 28/10/2022 15:17, Thierry Reding wrote:
> On Fri, Oct 28, 2022 at 01:35:56PM +0100, Jon Hunter wrote:
>> The Tegra Combined UART (TCU) is the default serial interface for Jetson
>> Xavier and Orin platforms and so update the bootargs for these platforms
>> to use the TCU.
>>
>> Signed-off-by: Jon Hunter <jonathanh@nvidia.com>
>> ---
>> arch/arm64/boot/dts/nvidia/tegra194-p2888.dtsi | 2 +-
>> arch/arm64/boot/dts/nvidia/tegra194-p3668.dtsi | 2 +-
>> arch/arm64/boot/dts/nvidia/tegra234-p3737-0000+p3701-0000.dts | 2 +-
>> 3 files changed, 3 insertions(+), 3 deletions(-)
>
> I'm a little confused now. You're certainly right that the TCU is the
> serial interface, but then why haven't we seen any fallout from this?
> That is, why does console=ttyS0,115200n8 still work just fine?
For Jetson Xavier platforms, ttyS0 is the UART on the 40-pin header and
yes could be used, but probably not obvious. I thought about keeping
both and we could if we wanted. I certainly have not used ttyS0 on Orin.
> Looking a little more into stdout-path (which we already point to the
> TCU), that ends up adding TCU as the preferred console already. So I
> wonder if we really need to set the console here at all.
>
> Do you see any difference in behavior after this patch? Does it fix
> anything? Could you perhaps try to remove this option altogether and see
> if that makes a difference?
I wondered about that, but I was not seeing any console until the rootfs
was mounted. However, looking at of_console_check() it appears that the
cmdline takes precedence over stdout-path. OK, let me try removing the
bootargs all together.
Jon
--
nvpublic
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2022-10-28 14:45 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-10-28 12:35 [PATCH] arm64: tegra: Update console for Jetson Xavier and Orin Jon Hunter
2022-10-28 14:17 ` Thierry Reding
2022-10-28 14:45 ` Jon Hunter
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).