[v1] arm64: dts: ti: k3-am625-beagleplay: Use UART name in pinmux name

[PATCH] arm64: dts: ti: k3-am625-beagleplay: Use UART name in pinmux name

Posted by Andrew Davis 2 years, 8 months ago

The main_uart0 may not always be the console, but it will always be
the UART0 in MAIN domain. Name the pinmux node to match. This makes
it consistent with all other TI SoC based boards.

Signed-off-by: Andrew Davis <afd@ti.com>
---
 arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts b/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts
index cb46c38ce2cc1..29c51705c2f17 100644
--- a/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts
+++ b/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts
@@ -397,7 +397,7 @@ AM62X_IOPAD(0x01a8, PIN_INPUT, 7) /* (D20) MCASP0_AFSX.GPIO1_12 */
 		>;
 	};
 
-	console_pins_default: console-pins-default {
+	main_uart0_pins_default: main-uart0-pins-default {
 		pinctrl-single,pins = <
 			AM62X_IOPAD(0x01c8, PIN_INPUT, 0) /* (D14) UART0_RXD */
 			AM62X_IOPAD(0x01cc, PIN_OUTPUT, 0) /* (E14) UART0_TXD */
@@ -736,7 +736,7 @@ wlcore: wlcore@2 {
 
 &main_uart0 {
 	pinctrl-names = "default";
-	pinctrl-0 = <&console_pins_default>;
+	pinctrl-0 = <&main_uart0_pins_default>;
 	status = "okay";
 };
 
-- 
2.39.2

Re: [PATCH] arm64: dts: ti: k3-am625-beagleplay: Use UART name in pinmux name

Posted by Robert Nelson 2 years, 8 months ago

Andew, since we are touching this section, do you want to match with
@Tony Lindgren rename changes here in omap...

https://www.spinics.net/lists/linux-omap/msg166406.html

Regards,

On Thu, Jun 1, 2023 at 1:56 PM Andrew Davis <afd@ti.com> wrote:
>
> The main_uart0 may not always be the console, but it will always be
> the UART0 in MAIN domain. Name the pinmux node to match. This makes
> it consistent with all other TI SoC based boards.
>
> Signed-off-by: Andrew Davis <afd@ti.com>
> ---
>  arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts b/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts
> index cb46c38ce2cc1..29c51705c2f17 100644
> --- a/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts
> +++ b/arch/arm64/boot/dts/ti/k3-am625-beagleplay.dts
> @@ -397,7 +397,7 @@ AM62X_IOPAD(0x01a8, PIN_INPUT, 7) /* (D20) MCASP0_AFSX.GPIO1_12 */
>                 >;
>         };
>
> -       console_pins_default: console-pins-default {
> +       main_uart0_pins_default: main-uart0-pins-default {
>                 pinctrl-single,pins = <
>                         AM62X_IOPAD(0x01c8, PIN_INPUT, 0) /* (D14) UART0_RXD */
>                         AM62X_IOPAD(0x01cc, PIN_OUTPUT, 0) /* (E14) UART0_TXD */
> @@ -736,7 +736,7 @@ wlcore: wlcore@2 {
>
>  &main_uart0 {
>         pinctrl-names = "default";
> -       pinctrl-0 = <&console_pins_default>;
> +       pinctrl-0 = <&main_uart0_pins_default>;
>         status = "okay";
>  };
>
> --
> 2.39.2
>


-- 
Robert Nelson
https://rcn-ee.com/