On Thu, Jul 04, 2019 at 06:04:43PM +0800, fugang.duan@xxxxxxx wrote: > From: Fugang Duan <fugang.duan@xxxxxxx> > > Add imx8qxp lpuart baud clock. > > Signed-off-by: Fugang Duan <fugang.duan@xxxxxxx> > --- > arch/arm64/boot/dts/freescale/imx8qxp.dtsi | 23 +++++++++++++++-------- > 1 file changed, 15 insertions(+), 8 deletions(-) > > diff --git a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > index 05fa0b7..4402b2e 100644 > --- a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi > @@ -30,6 +30,9 @@ > mmc2 = &usdhc3; > mu1 = &lsio_mu1; > serial0 = &adma_lpuart0; > + serial1 = &adma_lpuart1; > + serial2 = &adma_lpuart2; > + serial3 = &adma_lpuart3; This is not about adding baud clock, right? Please either mention the change in the commit log, or have it as a separate patch. Shawn > }; > > cpus { > @@ -209,8 +212,9 @@ > reg = <0x5a060000 0x1000>; > interrupts = <GIC_SPI 225 IRQ_TYPE_LEVEL_HIGH>; > interrupt-parent = <&gic>; > - clocks = <&adma_lpcg IMX_ADMA_LPCG_UART0_BAUD_CLK>; > - clock-names = "ipg"; > + clocks = <&adma_lpcg IMX_ADMA_LPCG_UART0_IPG_CLK>, > + <&adma_lpcg IMX_ADMA_LPCG_UART0_BAUD_CLK>; > + clock-names = "ipg", "baud"; > power-domains = <&pd IMX_SC_R_UART_0>; > status = "disabled"; > }; > @@ -220,8 +224,9 @@ > reg = <0x5a070000 0x1000>; > interrupts = <GIC_SPI 226 IRQ_TYPE_LEVEL_HIGH>; > interrupt-parent = <&gic>; > - clocks = <&adma_lpcg IMX_ADMA_LPCG_UART1_BAUD_CLK>; > - clock-names = "ipg"; > + clocks = <&adma_lpcg IMX_ADMA_LPCG_UART1_IPG_CLK>, > + <&adma_lpcg IMX_ADMA_LPCG_UART1_BAUD_CLK>; > + clock-names = "ipg", "baud"; > power-domains = <&pd IMX_SC_R_UART_1>; > status = "disabled"; > }; > @@ -231,8 +236,9 @@ > reg = <0x5a080000 0x1000>; > interrupts = <GIC_SPI 227 IRQ_TYPE_LEVEL_HIGH>; > interrupt-parent = <&gic>; > - clocks = <&adma_lpcg IMX_ADMA_LPCG_UART2_BAUD_CLK>; > - clock-names = "ipg"; > + clocks = <&adma_lpcg IMX_ADMA_LPCG_UART2_IPG_CLK>, > + <&adma_lpcg IMX_ADMA_LPCG_UART2_BAUD_CLK>; > + clock-names = "ipg", "baud"; > power-domains = <&pd IMX_SC_R_UART_2>; > status = "disabled"; > }; > @@ -242,8 +248,9 @@ > reg = <0x5a090000 0x1000>; > interrupts = <GIC_SPI 228 IRQ_TYPE_LEVEL_HIGH>; > interrupt-parent = <&gic>; > - clocks = <&adma_lpcg IMX_ADMA_LPCG_UART3_BAUD_CLK>; > - clock-names = "ipg"; > + clocks = <&adma_lpcg IMX_ADMA_LPCG_UART3_IPG_CLK>, > + <&adma_lpcg IMX_ADMA_LPCG_UART3_BAUD_CLK>; > + clock-names = "ipg", "baud"; > power-domains = <&pd IMX_SC_R_UART_3>; > status = "disabled"; > }; > -- > 2.7.4 >