[PATCH v4 4/5] arm64: dts: imx8mp: Add SAI, SDMA, AudioMIX
Alexander Stein
alexander.stein at ew.tq-group.com
Mon Feb 27 01:08:24 PST 2023
Hi Marek,
thanks for respinning this series.
Am Donnerstag, 23. Februar 2023, 18:11:13 CET schrieb Marek Vasut:
> Add all SAI nodes, SDMA2 and SDMA3 nodes, and AudioMIX node. This is
> needed to get audio operational on i.MX8MP .
>
> Reviewed-by: Luca Ceresoli <luca.ceresoli at bootlin.com>
> Tested-by: Luca Ceresoli <luca.ceresoli at bootlin.com>
> Signed-off-by: Marek Vasut <marex at denx.de>
> ---
> Cc: Abel Vesa <abelvesa at kernel.org>
> Cc: Fabio Estevam <festevam at gmail.com>
> Cc: Jacky Bai <ping.bai at nxp.com>
> Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt at linaro.org>
> Cc: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> Cc: Lucas Stach <l.stach at pengutronix.de>
> Cc: Marco Felsch <m.felsch at pengutronix.de>
> Cc: Michael Turquette <mturquette at baylibre.com>
> Cc: NXP Linux Team <linux-imx at nxp.com>
> Cc: Peng Fan <peng.fan at nxp.com>
> Cc: Pengutronix Kernel Team <kernel at pengutronix.de>
> Cc: Richard Cochran <richardcochran at gmail.com>
> Cc: Rob Herring <robh+dt at kernel.org>
> Cc: Sascha Hauer <s.hauer at pengutronix.de>
> Cc: Shawn Guo <shawnguo at kernel.org>
> Cc: Stephen Boyd <sboyd at kernel.org>
> Cc: devicetree at vger.kernel.org
> Cc: linux-arm-kernel at lists.infradead.org
> Cc: linux-clk at vger.kernel.org
> ---
> V2: - Add AUDIO_AXI clock to audio gpc
> - Use IMX8MP_CLK_AUDIOMIX_SDMA2_ROOT for SDMA2 IPG clock
> V3: Rename audio_ahb to plain ahb
> V4: - Add RB/TB from Luca
> - Rebase on next 20230223
> ---
> arch/arm64/boot/dts/freescale/imx8mp.dtsi | 153 ++++++++++++++++++++++
> 1 file changed, 153 insertions(+)
>
> diff --git a/arch/arm64/boot/dts/freescale/imx8mp.dtsi
> b/arch/arm64/boot/dts/freescale/imx8mp.dtsi index
> ca8093ee4d0e4..56d2a6377a5a6 100644
> --- a/arch/arm64/boot/dts/freescale/imx8mp.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8mp.dtsi
> @@ -571,6 +571,13 @@ pgc_usb2_phy: power-domain at 3 {
> reg =
<IMX8MP_POWER_DOMAIN_USB2_PHY>;
> };
>
> + pgc_audio: power-domain at 5 {
> + #power-domain-
cells = <0>;
> + reg =
<IMX8MP_POWER_DOMAIN_AUDIOMIX>;
> + clocks = <&clk
IMX8MP_CLK_AUDIO_ROOT>,
> + <&clk
IMX8MP_CLK_AUDIO_AXI>;
> + };
> +
> pgc_gpu2d: power-domain at 6 {
> #power-domain-
cells = <0>;
> reg =
<IMX8MP_POWER_DOMAIN_GPU2D>;
> @@ -1119,6 +1126,152 @@ opp-1000000000 {
> };
> };
>
> + aips5: bus at 30c00000 {
> + compatible = "fsl,aips-bus", "simple-bus";
> + reg = <0x30c00000 0x400000>;
> + #address-cells = <1>;
> + #size-cells = <1>;
> + ranges;
> +
> + spba-bus at 30c00000 {
> + compatible = "fsl,spba-bus", "simple-
bus";
> + reg = <0x30c00000 0x100000>;
Is there an SPBA bus for every AIPS? I'm wondering why some aips do have an
spba below and some do not.
> + #address-cells = <1>;
> + #size-cells = <1>;
> + ranges;
> +
> + sai1: sai at 30c10000 {
> + compatible = "fsl,imx8mp-
sai", "fsl,imx8mq-sai";
> + reg = <0x30c10000 0x10000>;
> + interrupts = <GIC_SPI 95
IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SAI1_IPG>,
> + <&clk
IMX8MP_CLK_DUMMY>,
> + <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SAI1_MCLK1>,
> + <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SAI1_MCLK2>,
> + <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SAI1_MCLK3>;
> + clock-names = "bus",
"mclk0", "mclk1", "mclk2", "mclk3";
> + dmas = <&sdma2 0 2 0>,
<&sdma2 1 2 0>;
Looking at Documentation/devicetree/bindings/dma/fsl,imx-sdma.yaml should the
2nd cell value be 24? This would be true for all other sai nodes.
> + dma-names = "rx", "tx";
> + status = "disabled";
> + };
> +
> + sai2: sai at 30c20000 {
> + compatible = "fsl,imx8mp-
sai", "fsl,imx8mq-sai";
> + reg = <0x30c20000 0x10000>;
> + interrupts = <GIC_SPI 96
IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SAI2_IPG>,
> + <&clk
IMX8MP_CLK_DUMMY>,
> + <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SAI2_MCLK1>,
> + <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SAI2_MCLK2>,
> + <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SAI2_MCLK3>;
> + clock-names = "bus",
"mclk0", "mclk1", "mclk2", "mclk3";
> + dmas = <&sdma2 2 2 0>,
<&sdma2 3 2 0>;
> + dma-names = "rx", "tx";
> + status = "disabled";
> + };
> +
> + sai3: sai at 30c30000 {
> + compatible = "fsl,imx8mp-
sai", "fsl,imx8mq-sai";
> + reg = <0x30c30000 0x10000>;
> + interrupts = <GIC_SPI 50
IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SAI3_IPG>,
> + <&clk
IMX8MP_CLK_DUMMY>,
> + <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SAI3_MCLK1>,
> + <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SAI3_MCLK2>,
> + <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SAI3_MCLK3>;
> + clock-names = "bus",
"mclk0", "mclk1", "mclk2", "mclk3";
> + dmas = <&sdma2 4 2 0>,
<&sdma2 5 2 0>;
> + dma-names = "rx", "tx";
> + status = "disabled";
> + };
> +
> + sai5: sai at 30c50000 {
> + compatible = "fsl,imx8mp-
sai", "fsl,imx8mq-sai";
> + reg = <0x30c50000 0x10000>;
> + interrupts = <GIC_SPI 90
IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SAI5_IPG>,
> + <&clk
IMX8MP_CLK_DUMMY>,
> + <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SAI5_MCLK1>,
> + <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SAI5_MCLK2>,
> + <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SAI5_MCLK3>;
> + clock-names = "bus",
"mclk0", "mclk1", "mclk2", "mclk3";
> + dmas = <&sdma2 8 2 0>,
<&sdma2 9 2 0>;
> + dma-names = "rx", "tx";
> + status = "disabled";
> + };
> +
> + sai6: sai at 30c60000 {
> + compatible = "fsl,imx8mp-
sai", "fsl,imx8mq-sai";
> + reg = <0x30c60000 0x10000>;
> + interrupts = <GIC_SPI 90
IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SAI6_IPG>,
> + <&clk
IMX8MP_CLK_DUMMY>,
> + <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SAI6_MCLK1>,
> + <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SAI6_MCLK2>,
> + <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SAI6_MCLK3>;
> + clock-names = "bus",
"mclk0", "mclk1", "mclk2", "mclk3";
> + dmas = <&sdma2 10 2 0>,
<&sdma2 11 2 0>;
> + dma-names = "rx", "tx";
> + status = "disabled";
> + };
> +
> + sai7: sai at 30c80000 {
> + compatible = "fsl,imx8mp-
sai", "fsl,imx8mq-sai";
> + reg = <0x30c80000 0x10000>;
> + interrupts = <GIC_SPI 111
IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SAI7_IPG>,
> + <&clk
IMX8MP_CLK_DUMMY>,
> + <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SAI7_MCLK1>,
> + <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SAI7_MCLK2>,
> + <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SAI7_MCLK3>;
> + clock-names = "bus",
"mclk0", "mclk1", "mclk2", "mclk3";
> + dmas = <&sdma2 12 2 0>,
<&sdma2 13 2 0>;
> + dma-names = "rx", "tx";
> + status = "disabled";
> + };
> + };
> +
> + sdma3: dma-controller at 30e00000 {
> + compatible = "fsl,imx8mp-sdma",
"fsl,imx8mq-sdma";
> + reg = <0x30e00000 0x10000>;
> + interrupts = <GIC_SPI 34
IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SDMA3_ROOT>,
> + <&clk
IMX8MP_CLK_AUDIO_ROOT>;
> + clock-names = "ipg", "ahb";
> + #dma-cells = <3>;
> + fsl,sdma-ram-script-name = "imx/sdma/
sdma-imx7d.bin";
> + };
> +
> + sdma2: dma-controller at 30e10000 {
> + compatible = "fsl,imx8mp-sdma",
"fsl,imx8mq-sdma";
> + reg = <0x30e10000 0x10000>;
> + interrupts = <GIC_SPI 103
IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&audio_blk_ctrl
IMX8MP_CLK_AUDIOMIX_SDMA2_ROOT>,
> + <&clk
IMX8MP_CLK_AUDIO_ROOT>;
> + clock-names = "ipg", "ahb";
> + #dma-cells = <3>;
> + fsl,sdma-ram-script-name = "imx/sdma/
sdma-imx7d.bin";
> + };
> +
> + audio_blk_ctrl: blk-ctrl at 30e20000 {
Shouldn't the node name be 'clock-controller at 30e20000' as mentioned in the
bindings?
But for the whole series:
Testes-by: Alexander Stein <alexander.stein at ew.tq-group.com>
Best regards,
Alexander
> + #clock-cells = <1>;
> + compatible = "fsl,imx8mp-audio-blk-
ctrl";
> + clocks = <&clk IMX8MP_CLK_AUDIO_ROOT>,
> + <&clk IMX8MP_CLK_SAI1>,
> + <&clk IMX8MP_CLK_SAI2>,
> + <&clk IMX8MP_CLK_SAI3>,
> + <&clk IMX8MP_CLK_SAI5>,
> + <&clk IMX8MP_CLK_SAI6>,
> + <&clk IMX8MP_CLK_SAI7>;
> + clock-names = "ahb",
> + "sai1", "sai2",
"sai3",
> + "sai5", "sai6",
"sai7";
> + power-domains = <&pgc_audio>;
> + power-domain-names = "audio";
> + reg = <0x30e20000 0x10000>;
> + };
> + };
> +
> aips4: bus at 32c00000 {
> compatible = "fsl,aips-bus", "simple-bus";
> reg = <0x32c00000 0x400000>;
--
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/
More information about the linux-arm-kernel
mailing list