* [PATCH V2 0/5] arm64: dts: ti: Drop bootargs
@ 2023-04-19 14:12 ` Nishanth Menon
0 siblings, 0 replies; 16+ messages in thread
From: Nishanth Menon @ 2023-04-19 14:12 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Vignesh Raghavendra
Cc: linux-kernel, devicetree, linux-arm-kernel, Tero Kristo, Nishanth Menon
The bootargs atm contains (generally) two information: console and
earlycon - console is already covered by stdout-path and earlycon is
really meant to be used when debug is needed and not something of a
default to spin out to all users.
This has come up multiple times, so cleaning it all up.
See [1] for context. AM64 is covered in [2]. There are quite a few other
instances of the same in mailing list[3].
Changes since V1:
* No functional changes
* Commit message update to be verbose as to the rationale.
* Picked up Roger's and Krzysztof's acks.
V1: https://lore.kernel.org/all/20230418165212.1456415-1-nm@ti.com/
[1] https://lore.kernel.org/linux-arm-kernel/81134eb9-2b7d-05bc-3035-a47f020861a8@linaro.org/
[2] https://lore.kernel.org/all/20230414073328.381336-1-nm@ti.com/
[3] https://lore.kernel.org/linux-devicetree/?q=Krzysztof+bootargs
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(-)
--
2.40.0
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH V2 0/5] arm64: dts: ti: Drop bootargs
@ 2023-04-19 14:12 ` Nishanth Menon
0 siblings, 0 replies; 16+ messages in thread
From: Nishanth Menon @ 2023-04-19 14:12 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Vignesh Raghavendra
Cc: linux-kernel, devicetree, linux-arm-kernel, Tero Kristo, Nishanth Menon
The bootargs atm contains (generally) two information: console and
earlycon - console is already covered by stdout-path and earlycon is
really meant to be used when debug is needed and not something of a
default to spin out to all users.
This has come up multiple times, so cleaning it all up.
See [1] for context. AM64 is covered in [2]. There are quite a few other
instances of the same in mailing list[3].
Changes since V1:
* No functional changes
* Commit message update to be verbose as to the rationale.
* Picked up Roger's and Krzysztof's acks.
V1: https://lore.kernel.org/all/20230418165212.1456415-1-nm@ti.com/
[1] https://lore.kernel.org/linux-arm-kernel/81134eb9-2b7d-05bc-3035-a47f020861a8@linaro.org/
[2] https://lore.kernel.org/all/20230414073328.381336-1-nm@ti.com/
[3] https://lore.kernel.org/linux-devicetree/?q=Krzysztof+bootargs
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(-)
--
2.40.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 16+ messages in thread
* [PATCH V2 1/5] arm64: dts: ti: k3-am62x-sk-common: Drop bootargs
2023-04-19 14:12 ` Nishanth Menon
@ 2023-04-19 14:12 ` Nishanth Menon
-1 siblings, 0 replies; 16+ messages in thread
From: Nishanth Menon @ 2023-04-19 14:12 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Vignesh Raghavendra
Cc: linux-kernel, devicetree, linux-arm-kernel, Tero Kristo,
Nishanth Menon, Krzysztof Kozlowski, Roger Quadros
Drop bootargs from the dts. The console arguments are already covered in
stdout-path property and earlycon is a debug property that should be
enabled only when debug is desired and not as default.
Link: https://lore.kernel.org/linux-arm-kernel/81134eb9-2b7d-05bc-3035-a47f020861a8@linaro.org/
Suggested-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: Roger Quadros <rogerq@kernel.org>
Signed-off-by: Nishanth Menon <nm@ti.com>
---
Similar discussions can be seen in https://lore.kernel.org/linux-devicetree/?q=Krzysztof+bootargs
Changes since v1:
- improved commit message
- picked up acks and reviews from Roger and Krzysztof
V1: https://lore.kernel.org/r/20230418165212.1456415-2-nm@ti.com
arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi b/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi
index 976f8303c84f..a61e8648f189 100644
--- a/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi
@@ -25,7 +25,6 @@ aliases {
chosen {
stdout-path = "serial2:115200n8";
- bootargs = "console=ttyS2,115200n8 earlycon=ns16550a,mmio32,0x02800000";
};
memory@80000000 {
--
2.40.0
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH V2 1/5] arm64: dts: ti: k3-am62x-sk-common: Drop bootargs
@ 2023-04-19 14:12 ` Nishanth Menon
0 siblings, 0 replies; 16+ messages in thread
From: Nishanth Menon @ 2023-04-19 14:12 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Vignesh Raghavendra
Cc: linux-kernel, devicetree, linux-arm-kernel, Tero Kristo,
Nishanth Menon, Krzysztof Kozlowski, Roger Quadros
Drop bootargs from the dts. The console arguments are already covered in
stdout-path property and earlycon is a debug property that should be
enabled only when debug is desired and not as default.
Link: https://lore.kernel.org/linux-arm-kernel/81134eb9-2b7d-05bc-3035-a47f020861a8@linaro.org/
Suggested-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: Roger Quadros <rogerq@kernel.org>
Signed-off-by: Nishanth Menon <nm@ti.com>
---
Similar discussions can be seen in https://lore.kernel.org/linux-devicetree/?q=Krzysztof+bootargs
Changes since v1:
- improved commit message
- picked up acks and reviews from Roger and Krzysztof
V1: https://lore.kernel.org/r/20230418165212.1456415-2-nm@ti.com
arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi b/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi
index 976f8303c84f..a61e8648f189 100644
--- a/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi
+++ b/arch/arm64/boot/dts/ti/k3-am62x-sk-common.dtsi
@@ -25,7 +25,6 @@ aliases {
chosen {
stdout-path = "serial2:115200n8";
- bootargs = "console=ttyS2,115200n8 earlycon=ns16550a,mmio32,0x02800000";
};
memory@80000000 {
--
2.40.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH V2 2/5] arm64: dts: ti: k3-am65*: Drop bootargs
2023-04-19 14:12 ` Nishanth Menon
@ 2023-04-19 14:12 ` Nishanth Menon
-1 siblings, 0 replies; 16+ messages in thread
From: Nishanth Menon @ 2023-04-19 14:12 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Vignesh Raghavendra
Cc: linux-kernel, devicetree, linux-arm-kernel, Tero Kristo,
Nishanth Menon, Jan Kiszka, Krzysztof Kozlowski, Roger Quadros
Drop bootargs from the dts. earlycon is a debug property that should be
enabled only when debug is desired and not as default - see referenced
link on discussion on this topic.
Cc: Jan Kiszka <jan.kiszka@siemens.com>
Link: https://lore.kernel.org/linux-arm-kernel/81134eb9-2b7d-05bc-3035-a47f020861a8@linaro.org/
Suggested-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: Roger Quadros <rogerq@kernel.org>
Signed-off-by: Nishanth Menon <nm@ti.com>
---
Similar discussions can be seen in https://lore.kernel.org/linux-devicetree/?q=Krzysztof+bootargs
Changes since v1:
- improved commit message
- picked up acks and reviews from Roger and Krzysztof
V1: https://lore.kernel.org/r/20230418165212.1456415-3-nm@ti.com
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
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH V2 2/5] arm64: dts: ti: k3-am65*: Drop bootargs
@ 2023-04-19 14:12 ` Nishanth Menon
0 siblings, 0 replies; 16+ messages in thread
From: Nishanth Menon @ 2023-04-19 14:12 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Vignesh Raghavendra
Cc: linux-kernel, devicetree, linux-arm-kernel, Tero Kristo,
Nishanth Menon, Jan Kiszka, Krzysztof Kozlowski, Roger Quadros
Drop bootargs from the dts. earlycon is a debug property that should be
enabled only when debug is desired and not as default - see referenced
link on discussion on this topic.
Cc: Jan Kiszka <jan.kiszka@siemens.com>
Link: https://lore.kernel.org/linux-arm-kernel/81134eb9-2b7d-05bc-3035-a47f020861a8@linaro.org/
Suggested-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: Roger Quadros <rogerq@kernel.org>
Signed-off-by: Nishanth Menon <nm@ti.com>
---
Similar discussions can be seen in https://lore.kernel.org/linux-devicetree/?q=Krzysztof+bootargs
Changes since v1:
- improved commit message
- picked up acks and reviews from Roger and Krzysztof
V1: https://lore.kernel.org/r/20230418165212.1456415-3-nm@ti.com
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
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH V2 3/5] arm64: dts: ti: k3-j721e-*: Drop bootargs
2023-04-19 14:12 ` Nishanth Menon
@ 2023-04-19 14:12 ` Nishanth Menon
-1 siblings, 0 replies; 16+ messages in thread
From: Nishanth Menon @ 2023-04-19 14:12 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Vignesh Raghavendra
Cc: linux-kernel, devicetree, linux-arm-kernel, Tero Kristo,
Nishanth Menon, Krzysztof Kozlowski, Roger Quadros
Drop bootargs from the dts. The console arguments are already covered in
stdout-path property and earlycon is a debug property that should be
enabled only when debug is desired and not as default.
Link: https://lore.kernel.org/linux-arm-kernel/81134eb9-2b7d-05bc-3035-a47f020861a8@linaro.org/
Suggested-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: Roger Quadros <rogerq@kernel.org>
Signed-off-by: Nishanth Menon <nm@ti.com>
---
Similar discussions can be seen in https://lore.kernel.org/linux-devicetree/?q=Krzysztof+bootargs
Changes since v1:
- improved commit message
- picked up acks and reviews from Roger and Krzysztof
V1: https://lore.kernel.org/r/20230418165212.1456415-4-nm@ti.com
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
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH V2 3/5] arm64: dts: ti: k3-j721e-*: Drop bootargs
@ 2023-04-19 14:12 ` Nishanth Menon
0 siblings, 0 replies; 16+ messages in thread
From: Nishanth Menon @ 2023-04-19 14:12 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Vignesh Raghavendra
Cc: linux-kernel, devicetree, linux-arm-kernel, Tero Kristo,
Nishanth Menon, Krzysztof Kozlowski, Roger Quadros
Drop bootargs from the dts. The console arguments are already covered in
stdout-path property and earlycon is a debug property that should be
enabled only when debug is desired and not as default.
Link: https://lore.kernel.org/linux-arm-kernel/81134eb9-2b7d-05bc-3035-a47f020861a8@linaro.org/
Suggested-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: Roger Quadros <rogerq@kernel.org>
Signed-off-by: Nishanth Menon <nm@ti.com>
---
Similar discussions can be seen in https://lore.kernel.org/linux-devicetree/?q=Krzysztof+bootargs
Changes since v1:
- improved commit message
- picked up acks and reviews from Roger and Krzysztof
V1: https://lore.kernel.org/r/20230418165212.1456415-4-nm@ti.com
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
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH V2 4/5] arm64: dts: ti: k3-j7200-common-proc-board: Drop bootargs
2023-04-19 14:12 ` Nishanth Menon
@ 2023-04-19 14:12 ` Nishanth Menon
-1 siblings, 0 replies; 16+ messages in thread
From: Nishanth Menon @ 2023-04-19 14:12 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Vignesh Raghavendra
Cc: linux-kernel, devicetree, linux-arm-kernel, Tero Kristo,
Nishanth Menon, Krzysztof Kozlowski, Roger Quadros
Drop bootargs from the dts. The console arguments are already covered in
stdout-path property and earlycon is a debug property that should be
enabled only when debug is desired and not as default.
Link: https://lore.kernel.org/linux-arm-kernel/81134eb9-2b7d-05bc-3035-a47f020861a8@linaro.org/
Suggested-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: Roger Quadros <rogerq@kernel.org>
Signed-off-by: Nishanth Menon <nm@ti.com>
---
Similar discussions can be seen in https://lore.kernel.org/linux-devicetree/?q=Krzysztof+bootargs
Changes since v1:
- improved commit message
- picked up acks and reviews from Roger and Krzysztof
V1: https://lore.kernel.org/r/20230418165212.1456415-5-nm@ti.com
arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts b/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts
index 0d39d6b8cc0c..5506341aae21 100644
--- a/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts
+++ b/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts
@@ -17,7 +17,6 @@ / {
chosen {
stdout-path = "serial2:115200n8";
- bootargs = "console=ttyS2,115200n8 earlycon=ns16550a,mmio32,0x02800000";
};
evm_12v0: fixedregulator-evm12v0 {
--
2.40.0
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH V2 4/5] arm64: dts: ti: k3-j7200-common-proc-board: Drop bootargs
@ 2023-04-19 14:12 ` Nishanth Menon
0 siblings, 0 replies; 16+ messages in thread
From: Nishanth Menon @ 2023-04-19 14:12 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Vignesh Raghavendra
Cc: linux-kernel, devicetree, linux-arm-kernel, Tero Kristo,
Nishanth Menon, Krzysztof Kozlowski, Roger Quadros
Drop bootargs from the dts. The console arguments are already covered in
stdout-path property and earlycon is a debug property that should be
enabled only when debug is desired and not as default.
Link: https://lore.kernel.org/linux-arm-kernel/81134eb9-2b7d-05bc-3035-a47f020861a8@linaro.org/
Suggested-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: Roger Quadros <rogerq@kernel.org>
Signed-off-by: Nishanth Menon <nm@ti.com>
---
Similar discussions can be seen in https://lore.kernel.org/linux-devicetree/?q=Krzysztof+bootargs
Changes since v1:
- improved commit message
- picked up acks and reviews from Roger and Krzysztof
V1: https://lore.kernel.org/r/20230418165212.1456415-5-nm@ti.com
arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts b/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts
index 0d39d6b8cc0c..5506341aae21 100644
--- a/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts
+++ b/arch/arm64/boot/dts/ti/k3-j7200-common-proc-board.dts
@@ -17,7 +17,6 @@ / {
chosen {
stdout-path = "serial2:115200n8";
- bootargs = "console=ttyS2,115200n8 earlycon=ns16550a,mmio32,0x02800000";
};
evm_12v0: fixedregulator-evm12v0 {
--
2.40.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH V2 5/5] arm64: dts: ti: k3-j721s2-common-proc-board: Drop bootargs
2023-04-19 14:12 ` Nishanth Menon
@ 2023-04-19 14:12 ` Nishanth Menon
-1 siblings, 0 replies; 16+ messages in thread
From: Nishanth Menon @ 2023-04-19 14:12 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Vignesh Raghavendra
Cc: linux-kernel, devicetree, linux-arm-kernel, Tero Kristo,
Nishanth Menon, Krzysztof Kozlowski, Roger Quadros
Drop bootargs from the dts. The console arguments are already covered in
stdout-path property and earlycon is a debug property that should be
enabled only when debug is desired and not as default.
Link: https://lore.kernel.org/linux-arm-kernel/81134eb9-2b7d-05bc-3035-a47f020861a8@linaro.org/
Suggested-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: Roger Quadros <rogerq@kernel.org>
Signed-off-by: Nishanth Menon <nm@ti.com>
---
Similar discussions can be seen in https://lore.kernel.org/linux-devicetree/?q=Krzysztof+bootargs
Changes since v1:
- improved commit message
- picked up acks and reviews from Roger and Krzysztof
V1: https://lore.kernel.org/r/20230418165212.1456415-6-nm@ti.com
arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts b/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
index b4b9edfe2d12..1299c715f34c 100644
--- a/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
+++ b/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
@@ -16,7 +16,6 @@ / {
chosen {
stdout-path = "serial2:115200n8";
- bootargs = "console=ttyS2,115200n8 earlycon=ns16550a,mmio32,2880000";
};
aliases {
--
2.40.0
^ permalink raw reply related [flat|nested] 16+ messages in thread
* [PATCH V2 5/5] arm64: dts: ti: k3-j721s2-common-proc-board: Drop bootargs
@ 2023-04-19 14:12 ` Nishanth Menon
0 siblings, 0 replies; 16+ messages in thread
From: Nishanth Menon @ 2023-04-19 14:12 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Vignesh Raghavendra
Cc: linux-kernel, devicetree, linux-arm-kernel, Tero Kristo,
Nishanth Menon, Krzysztof Kozlowski, Roger Quadros
Drop bootargs from the dts. The console arguments are already covered in
stdout-path property and earlycon is a debug property that should be
enabled only when debug is desired and not as default.
Link: https://lore.kernel.org/linux-arm-kernel/81134eb9-2b7d-05bc-3035-a47f020861a8@linaro.org/
Suggested-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Reviewed-by: Roger Quadros <rogerq@kernel.org>
Signed-off-by: Nishanth Menon <nm@ti.com>
---
Similar discussions can be seen in https://lore.kernel.org/linux-devicetree/?q=Krzysztof+bootargs
Changes since v1:
- improved commit message
- picked up acks and reviews from Roger and Krzysztof
V1: https://lore.kernel.org/r/20230418165212.1456415-6-nm@ti.com
arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts b/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
index b4b9edfe2d12..1299c715f34c 100644
--- a/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
+++ b/arch/arm64/boot/dts/ti/k3-j721s2-common-proc-board.dts
@@ -16,7 +16,6 @@ / {
chosen {
stdout-path = "serial2:115200n8";
- bootargs = "console=ttyS2,115200n8 earlycon=ns16550a,mmio32,2880000";
};
aliases {
--
2.40.0
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: [PATCH V2 2/5] arm64: dts: ti: k3-am65*: Drop bootargs
2023-04-19 14:12 ` Nishanth Menon
@ 2023-04-19 14:47 ` Jan Kiszka
-1 siblings, 0 replies; 16+ messages in thread
From: Jan Kiszka @ 2023-04-19 14:47 UTC (permalink / raw)
To: Nishanth Menon, Krzysztof Kozlowski, Rob Herring, Vignesh Raghavendra
Cc: linux-kernel, devicetree, linux-arm-kernel, Tero Kristo,
Krzysztof Kozlowski, Roger Quadros
On 19.04.23 16:12, Nishanth Menon wrote:
> Drop bootargs from the dts. earlycon is a debug property that should be
> enabled only when debug is desired and not as default - see referenced
> link on discussion on this topic.
>
> Cc: Jan Kiszka <jan.kiszka@siemens.com>
> Link: https://lore.kernel.org/linux-arm-kernel/81134eb9-2b7d-05bc-3035-a47f020861a8@linaro.org/
> Suggested-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Reviewed-by: Roger Quadros <rogerq@kernel.org>
> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
> Similar discussions can be seen in https://lore.kernel.org/linux-devicetree/?q=Krzysztof+bootargs
>
> Changes since v1:
> - improved commit message
> - picked up acks and reviews from Roger and Krzysztof
>
> V1: https://lore.kernel.org/r/20230418165212.1456415-3-nm@ti.com
>
> 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 {
For the IOT2050:
Acked-by: Jan Kiszka <jan.kiszka@siemens.com>
Jan
--
Siemens AG, Technology
Competence Center Embedded Linux
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH V2 2/5] arm64: dts: ti: k3-am65*: Drop bootargs
@ 2023-04-19 14:47 ` Jan Kiszka
0 siblings, 0 replies; 16+ messages in thread
From: Jan Kiszka @ 2023-04-19 14:47 UTC (permalink / raw)
To: Nishanth Menon, Krzysztof Kozlowski, Rob Herring, Vignesh Raghavendra
Cc: linux-kernel, devicetree, linux-arm-kernel, Tero Kristo,
Krzysztof Kozlowski, Roger Quadros
On 19.04.23 16:12, Nishanth Menon wrote:
> Drop bootargs from the dts. earlycon is a debug property that should be
> enabled only when debug is desired and not as default - see referenced
> link on discussion on this topic.
>
> Cc: Jan Kiszka <jan.kiszka@siemens.com>
> Link: https://lore.kernel.org/linux-arm-kernel/81134eb9-2b7d-05bc-3035-a47f020861a8@linaro.org/
> Suggested-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Reviewed-by: Roger Quadros <rogerq@kernel.org>
> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
> Similar discussions can be seen in https://lore.kernel.org/linux-devicetree/?q=Krzysztof+bootargs
>
> Changes since v1:
> - improved commit message
> - picked up acks and reviews from Roger and Krzysztof
>
> V1: https://lore.kernel.org/r/20230418165212.1456415-3-nm@ti.com
>
> 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 {
For the IOT2050:
Acked-by: Jan Kiszka <jan.kiszka@siemens.com>
Jan
--
Siemens AG, Technology
Competence Center Embedded Linux
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH V2 0/5] arm64: dts: ti: Drop bootargs
2023-04-19 14:12 ` Nishanth Menon
@ 2023-06-06 17:18 ` Vignesh Raghavendra
-1 siblings, 0 replies; 16+ messages in thread
From: Vignesh Raghavendra @ 2023-06-06 17:18 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Nishanth Menon
Cc: Vignesh Raghavendra, linux-kernel, devicetree, linux-arm-kernel,
Tero Kristo
Hi Nishanth Menon,
On Wed, 19 Apr 2023 09:12:17 -0500, Nishanth Menon wrote:
> The bootargs atm contains (generally) two information: console and
> earlycon - console is already covered by stdout-path and earlycon is
> really meant to be used when debug is needed and not something of a
> default to spin out to all users.
>
> This has come up multiple times, so cleaning it all up.
> See [1] for context. AM64 is covered in [2]. There are quite a few other
> instances of the same in mailing list[3].
>
> [...]
I have applied the following to branch ti-k3-dts-next on [1].
Thank you!
[1/5] arm64: dts: ti: k3-am62x-sk-common: Drop bootargs
commit: d525ef9c7f077bfa8202b4b47ca9242ab936a1dc
[2/5] arm64: dts: ti: k3-am65*: Drop bootargs
commit: da4159a776593f802ee211ad4823f3aed31e5375
[3/5] arm64: dts: ti: k3-j721e-*: Drop bootargs
commit: af2cda7df7769d3353170b6a72d73bc4ea703480
[4/5] arm64: dts: ti: k3-j7200-common-proc-board: Drop bootargs
commit: f920c49f1e46a99c410e2aaf486ffb7ce3d12270
[5/5] arm64: dts: ti: k3-j721s2-common-proc-board: Drop bootargs
commit: cf82a026f54f54c241903add9b9bdbeb2a449db0
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent up the chain during
the next merge window (or sooner if it is a relevant bug fix), however if
problems are discovered then the patch may be dropped or reverted.
You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.
If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.
Please add any relevant lists and maintainers to the CCs when replying
to this mail.
[1] https://git.kernel.org/pub/scm/linux/kernel/git/ti/linux.git
--
Vignesh
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [PATCH V2 0/5] arm64: dts: ti: Drop bootargs
@ 2023-06-06 17:18 ` Vignesh Raghavendra
0 siblings, 0 replies; 16+ messages in thread
From: Vignesh Raghavendra @ 2023-06-06 17:18 UTC (permalink / raw)
To: Krzysztof Kozlowski, Rob Herring, Nishanth Menon
Cc: Vignesh Raghavendra, linux-kernel, devicetree, linux-arm-kernel,
Tero Kristo
Hi Nishanth Menon,
On Wed, 19 Apr 2023 09:12:17 -0500, Nishanth Menon wrote:
> The bootargs atm contains (generally) two information: console and
> earlycon - console is already covered by stdout-path and earlycon is
> really meant to be used when debug is needed and not something of a
> default to spin out to all users.
>
> This has come up multiple times, so cleaning it all up.
> See [1] for context. AM64 is covered in [2]. There are quite a few other
> instances of the same in mailing list[3].
>
> [...]
I have applied the following to branch ti-k3-dts-next on [1].
Thank you!
[1/5] arm64: dts: ti: k3-am62x-sk-common: Drop bootargs
commit: d525ef9c7f077bfa8202b4b47ca9242ab936a1dc
[2/5] arm64: dts: ti: k3-am65*: Drop bootargs
commit: da4159a776593f802ee211ad4823f3aed31e5375
[3/5] arm64: dts: ti: k3-j721e-*: Drop bootargs
commit: af2cda7df7769d3353170b6a72d73bc4ea703480
[4/5] arm64: dts: ti: k3-j7200-common-proc-board: Drop bootargs
commit: f920c49f1e46a99c410e2aaf486ffb7ce3d12270
[5/5] arm64: dts: ti: k3-j721s2-common-proc-board: Drop bootargs
commit: cf82a026f54f54c241903add9b9bdbeb2a449db0
All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent up the chain during
the next merge window (or sooner if it is a relevant bug fix), however if
problems are discovered then the patch may be dropped or reverted.
You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.
If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.
Please add any relevant lists and maintainers to the CCs when replying
to this mail.
[1] https://git.kernel.org/pub/scm/linux/kernel/git/ti/linux.git
--
Vignesh
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2023-06-06 17:19 UTC | newest]
Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-04-19 14:12 [PATCH V2 0/5] arm64: dts: ti: Drop bootargs Nishanth Menon
2023-04-19 14:12 ` Nishanth Menon
2023-04-19 14:12 ` [PATCH V2 1/5] arm64: dts: ti: k3-am62x-sk-common: " Nishanth Menon
2023-04-19 14:12 ` Nishanth Menon
2023-04-19 14:12 ` [PATCH V2 2/5] arm64: dts: ti: k3-am65*: " Nishanth Menon
2023-04-19 14:12 ` Nishanth Menon
2023-04-19 14:47 ` Jan Kiszka
2023-04-19 14:47 ` Jan Kiszka
2023-04-19 14:12 ` [PATCH V2 3/5] arm64: dts: ti: k3-j721e-*: " Nishanth Menon
2023-04-19 14:12 ` Nishanth Menon
2023-04-19 14:12 ` [PATCH V2 4/5] arm64: dts: ti: k3-j7200-common-proc-board: " Nishanth Menon
2023-04-19 14:12 ` Nishanth Menon
2023-04-19 14:12 ` [PATCH V2 5/5] arm64: dts: ti: k3-j721s2-common-proc-board: " Nishanth Menon
2023-04-19 14:12 ` Nishanth Menon
2023-06-06 17:18 ` [PATCH V2 0/5] arm64: dts: ti: " Vignesh Raghavendra
2023-06-06 17:18 ` Vignesh Raghavendra
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.