On Mon, Apr 24, 2023 at 10:21:08AM +0200, Alexander Stein wrote: > Add PWM device and the corresponding clock gating device in adma subsystem. > > Signed-off-by: Alexander Stein <alexander.stein@xxxxxxxxxxxxxxx> Looks good to me. I will pick it up after dt-bindings one gets accepted/picked first. Shawn > --- > * New in v2 > > .../arm64/boot/dts/freescale/imx8-ss-dma.dtsi | 25 +++++++++++++++++++ > 1 file changed, 25 insertions(+) > > diff --git a/arch/arm64/boot/dts/freescale/imx8-ss-dma.dtsi b/arch/arm64/boot/dts/freescale/imx8-ss-dma.dtsi > index 2dce8f2ee3ea..7d5f96c99020 100644 > --- a/arch/arm64/boot/dts/freescale/imx8-ss-dma.dtsi > +++ b/arch/arm64/boot/dts/freescale/imx8-ss-dma.dtsi > @@ -124,6 +124,19 @@ lpuart3: serial@5a090000 { > status = "disabled"; > }; > > + adma_pwm: pwm@5a190000 { > + compatible = "fsl,imx8qxp-pwm", "fsl,imx27-pwm"; > + reg = <0x5a190000 0x1000>; > + interrupts = <GIC_SPI 127 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&adma_pwm_lpcg 1>, > + <&adma_pwm_lpcg 0>; > + clock-names = "ipg", "per"; > + assigned-clocks = <&clk IMX_SC_R_LCD_0_PWM_0 IMX_SC_PM_CLK_PER>; > + assigned-clock-rates = <24000000>; > + #pwm-cells = <2>; > + power-domains = <&pd IMX_SC_R_LCD_0_PWM_0>; > + }; > + > spi0_lpcg: clock-controller@5a400000 { > compatible = "fsl,imx8qxp-lpcg"; > reg = <0x5a400000 0x10000>; > @@ -220,6 +233,18 @@ uart3_lpcg: clock-controller@5a490000 { > power-domains = <&pd IMX_SC_R_UART_3>; > }; > > + adma_pwm_lpcg: clock-controller@5a590000 { > + compatible = "fsl,imx8qxp-lpcg"; > + reg = <0x5a590000 0x10000>; > + #clock-cells = <1>; > + clocks = <&clk IMX_SC_R_LCD_0_PWM_0 IMX_SC_PM_CLK_PER>, > + <&dma_ipg_clk>; > + clock-indices = <IMX_LPCG_CLK_0>, <IMX_LPCG_CLK_4>; > + clock-output-names = "adma_pwm_lpcg_clk", > + "adma_pwm_lpcg_ipg_clk"; > + power-domains = <&pd IMX_SC_R_LCD_0_PWM_0>; > + }; > + > i2c0: i2c@5a800000 { > reg = <0x5a800000 0x4000>; > interrupts = <GIC_SPI 220 IRQ_TYPE_LEVEL_HIGH>; > -- > 2.34.1 >