[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20251126132825.axrkcjj7gsexn6br@lankiness>
Date: Wed, 26 Nov 2025 07:28:25 -0600
From: Nishanth Menon <nm@...com>
To: Wadim Egorov <w.egorov@...tec.de>
CC: <vigneshr@...com>, <kristo@...nel.org>, <robh@...nel.org>,
<krzk+dt@...nel.org>, <conor+dt@...nel.org>,
<linux-arm-kernel@...ts.infradead.org>, <linux-kernel@...r.kernel.org>,
<devicetree@...r.kernel.org>, <upstream@...ts.phytec.de>
Subject: Re: [PATCH 2/3] arm64: dts: ti:
k3-am642-phyboard-electra-x27-gpio1-spi1-uart3: Fix schema warnings
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@...tec.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
Powered by blists - more mailing lists