2023-04-18 16:57:14

by Nishanth Menon

[permalink] [raw]
Subject: [PATCH 0/5] arm64: dts: ti: Drop bootargs

Hi,

This has come up multiple times, so cleaning it all up.
See [1] for context. AM64 is covered in [2].

Nishanth Menon (5):
arm64: dts: ti: k3-am62x-sk-common: Drop bootargs
arm64: dts: ti: k3-am65*: Drop bootargs
arm64: dts: ti: k3-j721e-*: Drop bootargs
arm64: dts: ti: k3-j7200-common-proc-board: Drop bootargs
arm64: dts: ti: k3-j721s2-common-proc-board: Drop bootargs

arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi | 1 -
arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi | 1 -
arch/arm64/boot/dts/ti/k3-am654-base-board.dts | 1 -
arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts | 1 -
arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts | 1 -
arch/arm64/boot/dts/ti/k3-j721e-sk.dts | 1 -
arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts | 1 -
7 files changed, 7 deletions(-)

[1] https://lore.kernel.org/linux-arm-kernel/[email protected]/
[2] https://lore.kernel.org/all/[email protected]/
--
2.40.0


2023-04-18 16:57:14

by Nishanth Menon

[permalink] [raw]
Subject: [PATCH 3/5] arm64: dts: ti: k3-j721e-*: Drop bootargs

Drop bootargs from the dts

Link: https://lore.kernel.org/linux-arm-kernel/[email protected]/
Suggested-by: Krzysztof Kozlowski <[email protected]>
Signed-off-by: Nishanth Menon <[email protected]>
---
arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts | 1 -
arch/arm64/boot/dts/ti/k3-j721e-sk.dts | 1 -
2 files changed, 2 deletions(-)

diff --git a/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts b/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts
index 7db0603125aa..c11c092c1ce0 100644
--- a/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts
+++ b/arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts
@@ -17,7 +17,6 @@ / {

chosen {
stdout-path = "serial2:115200n8";
- bootargs = "console=ttyS2,115200n8 earlycon=ns16550a,mmio32,0x02800000";
};

gpio_keys: gpio-keys {
diff --git a/arch/arm64/boot/dts/ti/k3-j721e-sk.dts b/arch/arm64/boot/dts/ti/k3-j721e-sk.dts
index f650a7fd66b4..ad7b45aeed0a 100644
--- a/arch/arm64/boot/dts/ti/k3-j721e-sk.dts
+++ b/arch/arm64/boot/dts/ti/k3-j721e-sk.dts
@@ -18,7 +18,6 @@ / {

chosen {
stdout-path = "serial2:115200n8";
- bootargs = "console=ttyS2,115200n8 earlycon=ns16550a,mmio32,0x02800000";
};

memory@80000000 {
--
2.40.0

2023-04-18 16:57:16

by Nishanth Menon

[permalink] [raw]
Subject: [PATCH 2/5] arm64: dts: ti: k3-am65*: Drop bootargs

Drop bootargs from the dts

Cc: Jan Kiszka <[email protected]>
Link: https://lore.kernel.org/linux-arm-kernel/[email protected]/
Suggested-by: Krzysztof Kozlowski <[email protected]>
Signed-off-by: Nishanth Menon <[email protected]>
---
arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi | 1 -
arch/arm64/boot/dts/ti/k3-am654-base-board.dts | 1 -
2 files changed, 2 deletions(-)

diff --git a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
index 96ac2b476b11..7d256a1638ff 100644
--- a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
@@ -21,7 +21,6 @@ aliases {

chosen {
stdout-path = "serial3:115200n8";
- bootargs = "earlycon=ns16550a,mmio32,0x02810000";
};

reserved-memory {
diff --git a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
index 592ab2b54cb3..0d6fc89eba7a 100644
--- a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
+++ b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
@@ -15,7 +15,6 @@ / {

chosen {
stdout-path = "serial2:115200n8";
- bootargs = "earlycon=ns16550a,mmio32,0x02800000";
};

memory@80000000 {
--
2.40.0

2023-04-18 16:58:52

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH 0/5] arm64: dts: ti: Drop bootargs

On 18/04/2023 18:52, Nishanth Menon wrote:
> Hi,
>
> This has come up multiple times, so cleaning it all up.
> See [1] for context. AM64 is covered in [2].
>
> Nishanth Menon (5):
> arm64: dts: ti: k3-am62x-sk-common: Drop bootargs
> arm64: dts: ti: k3-am65*: Drop bootargs
> arm64: dts: ti: k3-j721e-*: Drop bootargs
> arm64: dts: ti: k3-j7200-common-proc-board: Drop bootargs
> arm64: dts: ti: k3-j721s2-common-proc-board: Drop bootargs
>
> arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi | 1 -
> arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi | 1 -
> arch/arm64/boot/dts/ti/k3-am654-base-board.dts | 1 -
> arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts | 1 -
> arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts | 1 -
> arch/arm64/boot/dts/ti/k3-j721e-sk.dts | 1 -

For the series:

Acked-by: Krzysztof Kozlowski <[email protected]>

Best regards,
Krzysztof

2023-04-19 12:03:46

by Roger Quadros

[permalink] [raw]
Subject: Re: [PATCH 0/5] arm64: dts: ti: Drop bootargs



On 18/04/2023 19:52, Nishanth Menon wrote:
> Hi,
>
> This has come up multiple times, so cleaning it all up.
> See [1] for context. AM64 is covered in [2].
>
> Nishanth Menon (5):
> arm64: dts: ti: k3-am62x-sk-common: Drop bootargs
> arm64: dts: ti: k3-am65*: Drop bootargs
> arm64: dts: ti: k3-j721e-*: Drop bootargs
> arm64: dts: ti: k3-j7200-common-proc-board: Drop bootargs
> arm64: dts: ti: k3-j721s2-common-proc-board: Drop bootargs
>
> arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi | 1 -
> arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi | 1 -
> arch/arm64/boot/dts/ti/k3-am654-base-board.dts | 1 -
> arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts | 1 -
> arch/arm64/boot/dts/ti/k3-j721e-common-proc-board.dts | 1 -
> arch/arm64/boot/dts/ti/k3-j721e-sk.dts | 1 -
> arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts | 1 -
> 7 files changed, 7 deletions(-)
>
> [1] https://lore.kernel.org/linux-arm-kernel/[email protected]/
> [2] https://lore.kernel.org/all/[email protected]/

Reviewed-by: Roger Quadros <[email protected]>

2023-04-19 12:32:00

by Jan Kiszka

[permalink] [raw]
Subject: Re: [PATCH 2/5] arm64: dts: ti: k3-am65*: Drop bootargs

On 18.04.23 18:52, Nishanth Menon wrote:
> Drop bootargs from the dts
>

No reasoning provided, apparently also in the other patches of this
series. Given that these DTS here are not the last ones using bootargs
in upstream, that seems reasonable to state here.

Jan

> Cc: Jan Kiszka <[email protected]>
> Link: https://lore.kernel.org/linux-arm-kernel/[email protected]/
> Suggested-by: Krzysztof Kozlowski <[email protected]>
> Signed-off-by: Nishanth Menon <[email protected]>
> ---
> arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi | 1 -
> arch/arm64/boot/dts/ti/k3-am654-base-board.dts | 1 -
> 2 files changed, 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
> index 96ac2b476b11..7d256a1638ff 100644
> --- a/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
> +++ b/arch/arm64/boot/dts/ti/k3-am65-iot2050-common.dtsi
> @@ -21,7 +21,6 @@ aliases {
>
> chosen {
> stdout-path = "serial3:115200n8";
> - bootargs = "earlycon=ns16550a,mmio32,0x02810000";
> };
>
> reserved-memory {
> diff --git a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
> index 592ab2b54cb3..0d6fc89eba7a 100644
> --- a/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am654-base-board.dts
> @@ -15,7 +15,6 @@ / {
>
> chosen {
> stdout-path = "serial2:115200n8";
> - bootargs = "earlycon=ns16550a,mmio32,0x02800000";
> };
>
> memory@80000000 {

--
Siemens AG, Technology
Competence Center Embedded Linux

2023-04-19 13:15:02

by Nishanth Menon

[permalink] [raw]
Subject: Re: [PATCH 2/5] arm64: dts: ti: k3-am65*: Drop bootargs

On 14:28-20230419, Jan Kiszka wrote:
> On 18.04.23 18:52, Nishanth Menon wrote:
> > Drop bootargs from the dts
> >
>
> No reasoning provided, apparently also in the other patches of this
> series. Given that these DTS here are not the last ones using bootargs
> in upstream, that seems reasonable to state here.
>
> Jan

Will update.


--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D