| Message ID | 20230424082108.26512-2-alexander.stein@ew.tq-group.com |
|---|---|
| State | Handled Elsewhere |
| Headers | show |
| Series | [v2,1/2] dt-bindings: pwm: imx: add i.MX8QXP compatible | expand |
Hi, Am Montag, 24. April 2023, 10:21:08 CEST schrieb Alexander Stein: > Add PWM device and the corresponding clock gating device in adma subsystem. Gentle ping. Best regards, Alexander > Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com> > --- > * 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>;
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@ew.tq-group.com> 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 >
Hi Shawn, Am Sonntag, 4. Juni 2023, 11:50:38 CEST schrieb Shawn Guo: > 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@ew.tq-group.com> > > Looks good to me. I will pick it up after dt-bindings one gets > accepted/picked first. Did this get missed accidentally? Patch 1 was picked by Thierry already. Best regards, Alexander > 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>;
On Thu, Aug 17, 2023 at 10:03:20AM +0200, Alexander Stein wrote: > Hi Shawn, > > Am Sonntag, 4. Juni 2023, 11:50:38 CEST schrieb Shawn Guo: > > 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@ew.tq-group.com> > > > > Looks good to me. I will pick it up after dt-bindings one gets > > accepted/picked first. > > Did this get missed accidentally? Patch 1 was picked by Thierry already. Sorry, indeed! Applied, thanks! Shawn
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>;
Add PWM device and the corresponding clock gating device in adma subsystem. Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com> --- * New in v2 .../arm64/boot/dts/freescale/imx8-ss-dma.dtsi | 25 +++++++++++++++++++ 1 file changed, 25 insertions(+)