* [PATCH 2/3] arm64: dts: ti: k3-am642-phyboard-electra-x27-gpio1-spi1-uart3: Fix schema warnings
2025-11-26 8:44 [PATCH 1/3] arm64: dts: ti: k3-am642-phyboard-electra-peb-c-010: Fix icssg-prueth schema warning Wadim Egorov
@ 2025-11-26 8:44 ` Wadim Egorov
2025-11-26 13:28 ` Nishanth Menon
2025-11-26 8:44 ` [PATCH 3/3] arm64: dts: ti: k3-am62-lp-sk-nand: Rename pinctrls to fix " Wadim Egorov
2025-11-26 13:29 ` [PATCH 1/3] arm64: dts: ti: k3-am642-phyboard-electra-peb-c-010: Fix icssg-prueth schema warning Nishanth Menon
2 siblings, 1 reply; 6+ messages in thread
From: Wadim Egorov @ 2025-11-26 8:44 UTC (permalink / raw)
To: nm, vigneshr, kristo, robh, krzk+dt, conor+dt
Cc: linux-arm-kernel, linux-kernel, devicetree, upstream
Rename pinctrl nodes to comply with naming conventions required by
pinctrl-single schema. Also, replace invalid integer assignment in
SPI node with a boolean to align with omap-spi schema.
Signed-off-by: Wadim Egorov <w.egorov@phytec.de>
---
.../k3-am642-phyboard-electra-x27-gpio1-spi1-uart3.dtso | 8 ++++----
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am642-phyboard-electra-x27-gpio1-spi1-uart3.dtso b/arch/arm64/boot/dts/ti/k3-am642-phyboard-electra-x27-gpio1-spi1-uart3.dtso
index 996c42ec4253..bea8efa3e909 100644
--- a/arch/arm64/boot/dts/ti/k3-am642-phyboard-electra-x27-gpio1-spi1-uart3.dtso
+++ b/arch/arm64/boot/dts/ti/k3-am642-phyboard-electra-x27-gpio1-spi1-uart3.dtso
@@ -20,13 +20,13 @@ aliases {
};
&main_pmx0 {
- main_gpio1_exp_header_gpio_pins_default: main-gpio1-exp-header-gpio-pins-default {
+ main_gpio1_exp_header_gpio_pins_default: main-gpio1-exp-header-gpio-default-pins {
pinctrl-single,pins = <
AM64X_IOPAD(0x0220, PIN_INPUT, 7) /* (D14) SPI1_CS1.GPIO1_48 */
>;
};
- main_spi1_pins_default: main-spi1-pins-default {
+ main_spi1_pins_default: main-spi1-default-pins {
pinctrl-single,pins = <
AM64X_IOPAD(0x0224, PIN_INPUT, 0) /* (C14) SPI1_CLK */
AM64X_IOPAD(0x021C, PIN_OUTPUT, 0) /* (B14) SPI1_CS0 */
@@ -35,7 +35,7 @@ AM64X_IOPAD(0x022C, PIN_INPUT, 0) /* (A15) SPI1_D1 */
>;
};
- main_uart3_pins_default: main-uart3-pins-default {
+ main_uart3_pins_default: main-uart3-default-pins {
pinctrl-single,pins = <
AM64X_IOPAD(0x0048, PIN_INPUT, 2) /* (U20) GPMC0_AD3.UART3_RXD */
AM64X_IOPAD(0x004c, PIN_OUTPUT, 2) /* (U18) GPMC0_AD4.UART3_TXD */
@@ -52,7 +52,7 @@ &main_gpio1 {
&main_spi1 {
pinctrl-names = "default";
pinctrl-0 = <&main_spi1_pins_default>;
- ti,pindir-d0-out-d1-in = <1>;
+ ti,pindir-d0-out-d1-in;
status = "okay";
};
--
2.48.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH 2/3] arm64: dts: ti: k3-am642-phyboard-electra-x27-gpio1-spi1-uart3: Fix schema warnings
2025-11-26 8:44 ` [PATCH 2/3] arm64: dts: ti: k3-am642-phyboard-electra-x27-gpio1-spi1-uart3: Fix schema warnings Wadim Egorov
@ 2025-11-26 13:28 ` Nishanth Menon
0 siblings, 0 replies; 6+ messages in thread
From: Nishanth Menon @ 2025-11-26 13:28 UTC (permalink / raw)
To: Wadim Egorov
Cc: vigneshr, kristo, robh, krzk+dt, conor+dt, linux-arm-kernel,
linux-kernel, devicetree, upstream
On 09:44-20251126, Wadim Egorov wrote:
> Rename pinctrl nodes to comply with naming conventions required by
> pinctrl-single schema. Also, replace invalid integer assignment in
> SPI node with a boolean to align with omap-spi schema.
>
Fixes: 638ab30ce4c6 ("arm64: dts: ti: am64-phyboard-electra: Add DT overlay for X27 connector")
?
> Signed-off-by: Wadim Egorov <w.egorov@phytec.de>
> ---
> .../k3-am642-phyboard-electra-x27-gpio1-spi1-uart3.dtso | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am642-phyboard-electra-x27-gpio1-spi1-uart3.dtso b/arch/arm64/boot/dts/ti/k3-am642-phyboard-electra-x27-gpio1-spi1-uart3.dtso
> index 996c42ec4253..bea8efa3e909 100644
> --- a/arch/arm64/boot/dts/ti/k3-am642-phyboard-electra-x27-gpio1-spi1-uart3.dtso
> +++ b/arch/arm64/boot/dts/ti/k3-am642-phyboard-electra-x27-gpio1-spi1-uart3.dtso
> @@ -20,13 +20,13 @@ aliases {
> };
>
> &main_pmx0 {
> - main_gpio1_exp_header_gpio_pins_default: main-gpio1-exp-header-gpio-pins-default {
> + main_gpio1_exp_header_gpio_pins_default: main-gpio1-exp-header-gpio-default-pins {
> pinctrl-single,pins = <
> AM64X_IOPAD(0x0220, PIN_INPUT, 7) /* (D14) SPI1_CS1.GPIO1_48 */
> >;
> };
>
> - main_spi1_pins_default: main-spi1-pins-default {
> + main_spi1_pins_default: main-spi1-default-pins {
> pinctrl-single,pins = <
> AM64X_IOPAD(0x0224, PIN_INPUT, 0) /* (C14) SPI1_CLK */
> AM64X_IOPAD(0x021C, PIN_OUTPUT, 0) /* (B14) SPI1_CS0 */
> @@ -35,7 +35,7 @@ AM64X_IOPAD(0x022C, PIN_INPUT, 0) /* (A15) SPI1_D1 */
> >;
> };
>
> - main_uart3_pins_default: main-uart3-pins-default {
> + main_uart3_pins_default: main-uart3-default-pins {
> pinctrl-single,pins = <
> AM64X_IOPAD(0x0048, PIN_INPUT, 2) /* (U20) GPMC0_AD3.UART3_RXD */
> AM64X_IOPAD(0x004c, PIN_OUTPUT, 2) /* (U18) GPMC0_AD4.UART3_TXD */
> @@ -52,7 +52,7 @@ &main_gpio1 {
> &main_spi1 {
> pinctrl-names = "default";
> pinctrl-0 = <&main_spi1_pins_default>;
> - ti,pindir-d0-out-d1-in = <1>;
> + ti,pindir-d0-out-d1-in;
> status = "okay";
> };
>
> --
> 2.48.1
>
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
https://ti.com/opensource
^ permalink raw reply [flat|nested] 6+ messages in thread
* [PATCH 3/3] arm64: dts: ti: k3-am62-lp-sk-nand: Rename pinctrls to fix schema warnings
2025-11-26 8:44 [PATCH 1/3] arm64: dts: ti: k3-am642-phyboard-electra-peb-c-010: Fix icssg-prueth schema warning Wadim Egorov
2025-11-26 8:44 ` [PATCH 2/3] arm64: dts: ti: k3-am642-phyboard-electra-x27-gpio1-spi1-uart3: Fix schema warnings Wadim Egorov
@ 2025-11-26 8:44 ` Wadim Egorov
2025-11-26 13:27 ` Nishanth Menon
2025-11-26 13:29 ` [PATCH 1/3] arm64: dts: ti: k3-am642-phyboard-electra-peb-c-010: Fix icssg-prueth schema warning Nishanth Menon
2 siblings, 1 reply; 6+ messages in thread
From: Wadim Egorov @ 2025-11-26 8:44 UTC (permalink / raw)
To: nm, vigneshr, kristo, robh, krzk+dt, conor+dt
Cc: linux-arm-kernel, linux-kernel, devicetree, upstream
Rename pinctrl nodes to comply with naming conventions required by
pinctrl-single schema.
Signed-off-by: Wadim Egorov <w.egorov@phytec.de>
---
arch/arm64/boot/dts/ti/k3-am62-lp-sk-nand.dtso | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/ti/k3-am62-lp-sk-nand.dtso b/arch/arm64/boot/dts/ti/k3-am62-lp-sk-nand.dtso
index 173ac60723b6..b4daa674eaa1 100644
--- a/arch/arm64/boot/dts/ti/k3-am62-lp-sk-nand.dtso
+++ b/arch/arm64/boot/dts/ti/k3-am62-lp-sk-nand.dtso
@@ -14,7 +14,7 @@ &mcasp1 {
};
&main_pmx0 {
- gpmc0_pins_default: gpmc0-pins-default {
+ gpmc0_pins_default: gpmc0-default-pins {
pinctrl-single,pins = <
AM62X_IOPAD(0x003c, PIN_INPUT, 0) /* (K19) GPMC0_AD0 */
AM62X_IOPAD(0x0040, PIN_INPUT, 0) /* (L19) GPMC0_AD1 */
--
2.48.1
^ permalink raw reply related [flat|nested] 6+ messages in thread* Re: [PATCH 3/3] arm64: dts: ti: k3-am62-lp-sk-nand: Rename pinctrls to fix schema warnings
2025-11-26 8:44 ` [PATCH 3/3] arm64: dts: ti: k3-am62-lp-sk-nand: Rename pinctrls to fix " Wadim Egorov
@ 2025-11-26 13:27 ` Nishanth Menon
0 siblings, 0 replies; 6+ messages in thread
From: Nishanth Menon @ 2025-11-26 13:27 UTC (permalink / raw)
To: Wadim Egorov
Cc: vigneshr, kristo, robh, krzk+dt, conor+dt, linux-arm-kernel,
linux-kernel, devicetree, upstream
On 09:44-20251126, Wadim Egorov wrote:
> Rename pinctrl nodes to comply with naming conventions required by
> pinctrl-single schema.
>
Thanks for fixing.
Could we add:
Fixes: e569152274fec ("arm64: dts: ti: am62-lp-sk: Add overlay for NAND expansion card")
If I am not mistaken?
> Signed-off-by: Wadim Egorov <w.egorov@phytec.de>
> ---
> arch/arm64/boot/dts/ti/k3-am62-lp-sk-nand.dtso | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am62-lp-sk-nand.dtso b/arch/arm64/boot/dts/ti/k3-am62-lp-sk-nand.dtso
> index 173ac60723b6..b4daa674eaa1 100644
> --- a/arch/arm64/boot/dts/ti/k3-am62-lp-sk-nand.dtso
> +++ b/arch/arm64/boot/dts/ti/k3-am62-lp-sk-nand.dtso
> @@ -14,7 +14,7 @@ &mcasp1 {
> };
>
> &main_pmx0 {
> - gpmc0_pins_default: gpmc0-pins-default {
> + gpmc0_pins_default: gpmc0-default-pins {
> pinctrl-single,pins = <
> AM62X_IOPAD(0x003c, PIN_INPUT, 0) /* (K19) GPMC0_AD0 */
> AM62X_IOPAD(0x0040, PIN_INPUT, 0) /* (L19) GPMC0_AD1 */
> --
> 2.48.1
>
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
https://ti.com/opensource
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH 1/3] arm64: dts: ti: k3-am642-phyboard-electra-peb-c-010: Fix icssg-prueth schema warning
2025-11-26 8:44 [PATCH 1/3] arm64: dts: ti: k3-am642-phyboard-electra-peb-c-010: Fix icssg-prueth schema warning Wadim Egorov
2025-11-26 8:44 ` [PATCH 2/3] arm64: dts: ti: k3-am642-phyboard-electra-x27-gpio1-spi1-uart3: Fix schema warnings Wadim Egorov
2025-11-26 8:44 ` [PATCH 3/3] arm64: dts: ti: k3-am62-lp-sk-nand: Rename pinctrls to fix " Wadim Egorov
@ 2025-11-26 13:29 ` Nishanth Menon
2 siblings, 0 replies; 6+ messages in thread
From: Nishanth Menon @ 2025-11-26 13:29 UTC (permalink / raw)
To: Wadim Egorov
Cc: vigneshr, kristo, robh, krzk+dt, conor+dt, linux-arm-kernel,
linux-kernel, devicetree, upstream
On 09:44-20251126, Wadim Egorov wrote:
> Reduce length of dma-names and dmas properties for icssg1-ethernet
> node to comply with ti,icssg-prueth schema constraints. The previous
> entries exceeded the allowed count and triggered dtschema warnings
> during validation.
>
Fixes: e53fbf955ea7 ("arm64: dts: ti: k3-am642-phyboard-electra: Add PEB-C-010 Overlay")
Perhaps?
> Signed-off-by: Wadim Egorov <w.egorov@phytec.de>
> ---
> .../boot/dts/ti/k3-am642-phyboard-electra-peb-c-010.dtso | 7 ++-----
> 1 file changed, 2 insertions(+), 5 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am642-phyboard-electra-peb-c-010.dtso b/arch/arm64/boot/dts/ti/k3-am642-phyboard-electra-peb-c-010.dtso
> index 7fc73cfacadb..1176a52d560b 100644
> --- a/arch/arm64/boot/dts/ti/k3-am642-phyboard-electra-peb-c-010.dtso
> +++ b/arch/arm64/boot/dts/ti/k3-am642-phyboard-electra-peb-c-010.dtso
> @@ -30,13 +30,10 @@ icssg1-ethernet {
> <&main_pktdma 0xc206 15>, /* egress slice 1 */
> <&main_pktdma 0xc207 15>, /* egress slice 1 */
> <&main_pktdma 0x4200 15>, /* ingress slice 0 */
> - <&main_pktdma 0x4201 15>, /* ingress slice 1 */
> - <&main_pktdma 0x4202 0>, /* mgmnt rsp slice 0 */
> - <&main_pktdma 0x4203 0>; /* mgmnt rsp slice 1 */
> + <&main_pktdma 0x4201 15>; /* ingress slice 1 */
> dma-names = "tx0-0", "tx0-1", "tx0-2", "tx0-3",
> "tx1-0", "tx1-1", "tx1-2", "tx1-3",
> - "rx0", "rx1",
> - "rxmgm0", "rxmgm1";
> + "rx0", "rx1";
>
> firmware-name = "ti-pruss/am65x-sr2-pru0-prueth-fw.elf",
> "ti-pruss/am65x-sr2-rtu0-prueth-fw.elf",
> --
> 2.48.1
>
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
https://ti.com/opensource
^ permalink raw reply [flat|nested] 6+ messages in thread