From: Alexander Stein <alexander.stein@ew.tq-group.com>
To: Adam Ford <aford173@gmail.com>
Cc: linux-clk@vger.kernel.org, Marek Vasut <marex@denx.de>,
Luca Ceresoli <luca.ceresoli@bootlin.com>,
Abel Vesa <abelvesa@kernel.org>,
Fabio Estevam <festevam@gmail.com>, Jacky Bai <ping.bai@nxp.com>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
Lucas Stach <l.stach@pengutronix.de>,
Marco Felsch <m.felsch@pengutronix.de>,
Michael Turquette <mturquette@baylibre.com>,
NXP Linux Team <linux-imx@nxp.com>, Peng Fan <peng.fan@nxp.com>,
Pengutronix Kernel Team <kernel@pengutronix.de>,
Richard Cochran <richardcochran@gmail.com>,
Rob Herring <robh+dt@kernel.org>,
Sascha Hauer <s.hauer@pengutronix.de>,
Shawn Guo <shawnguo@kernel.org>, Stephen Boyd <sboyd@kernel.org>,
devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v4 4/5] arm64: dts: imx8mp: Add SAI, SDMA, AudioMIX
Date: Mon, 27 Feb 2023 13:00:56 +0100 [thread overview]
Message-ID: <4260396.ejJDZkT8p0@steina-w> (raw)
In-Reply-To: <CAHCN7xLoV1wNSPBoxwK89Mj3JWZoEYKEd2MJTUZdDbL03k5Axw@mail.gmail.com>
Am Montag, 27. Februar 2023, 12:38:10 CET schrieb Adam Ford:
> On Mon, Feb 27, 2023 at 3:46 AM Alexander Stein
>
> <alexander.stein@ew.tq-group.com> wrote:
> > 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@bootlin.com>
> > > Tested-by: Luca Ceresoli <luca.ceresoli@bootlin.com>
> > > Signed-off-by: Marek Vasut <marex@denx.de>
> > > ---
> > > Cc: Abel Vesa <abelvesa@kernel.org>
> > > Cc: Fabio Estevam <festevam@gmail.com>
> > > Cc: Jacky Bai <ping.bai@nxp.com>
> > > Cc: Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
> > > Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> > > Cc: Lucas Stach <l.stach@pengutronix.de>
> > > Cc: Marco Felsch <m.felsch@pengutronix.de>
> > > Cc: Michael Turquette <mturquette@baylibre.com>
> > > Cc: NXP Linux Team <linux-imx@nxp.com>
> > > Cc: Peng Fan <peng.fan@nxp.com>
> > > Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
> > > Cc: Richard Cochran <richardcochran@gmail.com>
> > > Cc: Rob Herring <robh+dt@kernel.org>
> > > Cc: Sascha Hauer <s.hauer@pengutronix.de>
> > > Cc: Shawn Guo <shawnguo@kernel.org>
> > > Cc: Stephen Boyd <sboyd@kernel.org>
> > > Cc: devicetree@vger.kernel.org
> > > Cc: linux-arm-kernel@lists.infradead.org
> > > Cc: linux-clk@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@3 {
> > >
> > > reg =
> >
> > <IMX8MP_POWER_DOMAIN_USB2_PHY>;
> >
> > > };
> > >
> > > + pgc_audio: power-domain@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@6 {
> > >
> > > #power-domain-
> >
> > cells = <0>;
> >
> > > reg =
> >
> > <IMX8MP_POWER_DOMAIN_GPU2D>;
> >
> > > @@ -1119,6 +1126,152 @@ opp-1000000000 {
> > >
> > > };
> > >
> > > };
> > >
> > > + aips5: bus@30c00000 {
> > > + compatible = "fsl,aips-bus", "simple-bus";
> > > + reg = <0x30c00000 0x400000>;
> > > + #address-cells = <1>;
> > > + #size-cells = <1>;
> > > + ranges;
> > > +
> > > + spba-bus@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.
>
> The SPBA buses are listed in the TRM, and they appear to be mostly
> covering serial devices (UART, SPI, SAI, etc), but some are excluded.
> As to why it is this way is more of a question for NXP. Check out
> chapter 2 of the TRM for the AIPS memory maps.
Thanks, I did find those AIPS parts. But the issue regarding SPBA is not so
clear. Apparently the spba compatible is checked by the SDMA driver.
Best regards,
Alexander
> adam
>
> > > + #address-cells = <1>;
> > > + #size-cells = <1>;
> > > + ranges;
> > > +
> > > + sai1: sai@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@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@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@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@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@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@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@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@30e20000 {
> >
> > Shouldn't the node name be 'clock-controller@30e20000' as mentioned in the
> > bindings?
> > But for the whole series:
> > Testes-by: Alexander Stein <alexander.stein@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@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/
--
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/
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2023-02-27 12:02 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-02-23 17:11 [PATCH v4 1/5] clk: Introduce devm_clk_hw_register_gate_parent_data() Marek Vasut
2023-02-23 17:11 ` [PATCH v4 2/5] clk: imx: imx8mp: Add audiomix block control Marek Vasut
2023-02-24 16:38 ` Luca Ceresoli
2023-02-23 17:11 ` [PATCH v4 3/5] dt-bindings: clock: " Marek Vasut
2023-02-23 17:11 ` [PATCH v4 4/5] arm64: dts: imx8mp: Add SAI, SDMA, AudioMIX Marek Vasut
2023-02-27 9:08 ` Alexander Stein
2023-02-27 11:38 ` Adam Ford
2023-02-27 12:00 ` Alexander Stein [this message]
2023-02-27 13:53 ` Marek Vasut
2023-02-23 17:11 ` [PATCH v4 5/5] arm64: dts: imx8mp: Add analog audio output on i.MX8MP EVK Marek Vasut
2023-02-23 17:28 ` Fabio Estevam
2023-02-23 17:39 ` Marek Vasut
2023-02-24 12:59 ` Marco Felsch
2023-02-24 0:24 ` [PATCH v4 1/5] clk: Introduce devm_clk_hw_register_gate_parent_data() Adam Ford
2023-02-27 8:38 ` Alexander Stein
2023-02-27 13:47 ` Marek Vasut
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4260396.ejJDZkT8p0@steina-w \
--to=alexander.stein@ew.tq-group.com \
--cc=abelvesa@kernel.org \
--cc=aford173@gmail.com \
--cc=devicetree@vger.kernel.org \
--cc=festevam@gmail.com \
--cc=kernel@pengutronix.de \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=l.stach@pengutronix.de \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-imx@nxp.com \
--cc=luca.ceresoli@bootlin.com \
--cc=m.felsch@pengutronix.de \
--cc=marex@denx.de \
--cc=mturquette@baylibre.com \
--cc=peng.fan@nxp.com \
--cc=ping.bai@nxp.com \
--cc=richardcochran@gmail.com \
--cc=robh+dt@kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=sboyd@kernel.org \
--cc=shawnguo@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox