Hi Frank, thanks for the patch. Am Mittwoch, 28. Februar 2024, 20:54:58 CET schrieb Frank Li: > From: Alice Guo <alice.guo@xxxxxxx> > > Adding lpuart device in cm40 subsystem. > > Signed-off-by: Alice Guo <alice.guo@xxxxxxx> > Reviewed-by: Peng Fan <peng.fan@xxxxxxx> > Signed-off-by: Frank Li <Frank.Li@xxxxxxx> > --- > arch/arm64/boot/dts/freescale/imx8-ss-cm40.dtsi | 25 +++++++++++++++++++++++++ > 1 file changed, 25 insertions(+) > > diff --git a/arch/arm64/boot/dts/freescale/imx8-ss-cm40.dtsi b/arch/arm64/boot/dts/freescale/imx8-ss-cm40.dtsi > index b1d626862ddf8..ecca5ada224b7 100644 > --- a/arch/arm64/boot/dts/freescale/imx8-ss-cm40.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx8-ss-cm40.dtsi > @@ -64,4 +64,29 @@ cm40_intmux: intmux@37400000 { > power-domains = <&pd IMX_SC_R_M4_0_INTMUX>; > status = "disabled"; > }; > + > + cm40_lpuart: serial@37220000 { > + compatible = "fsl,imx8qxp-lpuart"; > + reg = <0x37220000 0x1000>; > + interrupts = <7 IRQ_TYPE_LEVEL_HIGH>; > + interrupt-parent = <&cm40_intmux>; With interrupt-parent set in Patch 1 for the whole subsystem, this line is not needed anymore. Best regards, Alexander > + clocks = <&cm40_uart_lpcg 1>, <&cm40_uart_lpcg 0>; > + clock-names = "ipg", "baud"; > + assigned-clocks = <&clk IMX_SC_R_M4_0_UART IMX_SC_PM_CLK_PER>; > + assigned-clock-rates = <24000000>; > + power-domains = <&pd IMX_SC_R_M4_0_UART>; > + status = "disabled"; > + }; > + > + cm40_uart_lpcg: clock-controller@37620000 { > + compatible = "fsl,imx8qxp-lpcg"; > + reg = <0x37620000 0x1000>; > + #clock-cells = <1>; > + clocks = <&clk IMX_SC_R_M4_0_UART IMX_SC_PM_CLK_PER>, > + <&cm40_ipg_clk>; > + clock-indices = <IMX_LPCG_CLK_0>, <IMX_LPCG_CLK_1>; > + clock-output-names = "cm40_lpcg_uart_clk", > + "cm40_lpcg_uart_ipg_clk"; > + power-domains = <&pd IMX_SC_R_M4_0_UART>; > + }; > }; > > -- TQ-Systems GmbH | Mühlstraße 2, Gut Delling | 82229 Seefeld, Germany Amtsgericht München, HRB 105018 Geschäftsführer: Detlef Schneider, Rüdiger Stahl, Stefan Schneider http://www.tq-group.com/