On Tue, Mar 05, 2024 at 10:54:58AM -0500, Frank Li wrote: > Add lpuart1 and cm40 uart. > > Signed-off-by: Frank Li <Frank.Li@xxxxxxx> > --- > arch/arm64/boot/dts/freescale/imx8dxl-evk.dts | 37 +++++++++++++++++++++++++++ > 1 file changed, 37 insertions(+) > > diff --git a/arch/arm64/boot/dts/freescale/imx8dxl-evk.dts b/arch/arm64/boot/dts/freescale/imx8dxl-evk.dts > index 2123d431e0613..737bacacc2da1 100644 > --- a/arch/arm64/boot/dts/freescale/imx8dxl-evk.dts > +++ b/arch/arm64/boot/dts/freescale/imx8dxl-evk.dts > @@ -16,6 +16,8 @@ aliases { > mmc0 = &usdhc1; > mmc1 = &usdhc2; > serial0 = &lpuart0; > + serial1 = &lpuart1; > + serial6 = &cm40_lpuart; > }; > > chosen { > @@ -51,6 +53,16 @@ linux,cma { > }; > }; > > + m2_uart1_sel: fixedregulator-101 { Why do we not name the node regulator-xxx just like other fixed regulators? Also not sure what suffix -101 means. Can we name it like regulator-m2uart1sel or something? Shawn > + compatible = "regulator-fixed"; > + regulator-min-microvolt = <3300000>; > + regulator-max-microvolt = <3300000>; > + regulator-name = "m2_uart1_sel"; > + gpio = <&pca6416_1 6 GPIO_ACTIVE_HIGH>; > + enable-active-high; > + regulator-always-on; > + }; > + > mux3_en: regulator-0 { > compatible = "regulator-fixed"; > regulator-min-microvolt = <3300000>; > @@ -340,6 +352,12 @@ &lpuart0 { > status = "okay"; > }; > > +&lpuart1 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_lpuart1>; > + status = "okay"; > +}; > + > &flexcan2 { > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_flexcan2>; > @@ -354,6 +372,16 @@ &flexcan3 { > status = "okay"; > }; > > +&cm40_intmux { > + status = "disabled"; > +}; > + > +&cm40_lpuart { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_cm40_lpuart>; > + status = "disabled"; > +}; > + > &lsio_gpio4 { > status = "okay"; > }; > @@ -595,6 +623,15 @@ IMX8DXL_UART0_TX_ADMA_UART0_TX 0x06000020 > >; > }; > > + pinctrl_lpuart1: lpuart1grp { > + fsl,pins = < > + IMX8DXL_UART1_TX_ADMA_UART1_TX 0x06000020 > + IMX8DXL_UART1_RX_ADMA_UART1_RX 0x06000020 > + IMX8DXL_UART1_RTS_B_ADMA_UART1_RTS_B 0x06000020 > + IMX8DXL_UART1_CTS_B_ADMA_UART1_CTS_B 0x06000020 > + >; > + }; > + > pinctrl_usdhc1: usdhc1grp { > fsl,pins = < > IMX8DXL_EMMC0_CLK_CONN_EMMC0_CLK 0x06000041 > > -- > 2.34.1 >