From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id E4DB3C64ED6 for ; Mon, 27 Feb 2023 12:02:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Date:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=oBArrqPe5KcaOSOTYSXpCMvcP0/1bXtYL08r65hFZb8=; b=ZrEzSX0Y+BUm8Z cdoI8gUCTaKEwE3KYCUzz9kbg+Y2MAiyAo+hpMeNSsZS8a+dmTEYEyrjZj1ZBPlBtpNS7UVg5DHE8 w5S7SyXA+dhKQSBaKHIieBb2vcjBOX6TM/EhevXHwolIwYxz1pfenZZFEoE+n+G59gfLRTC0UgTsj KFbNtDSBO2J+Lq5Jl2CZad6tnhk+pZHukLSVOBPyKWkw75D22t9ZjYy4aIhiJC4VfNyqw6BjiBGZN HSr2OQBGWX4F2ArmA+sKsYCm/iFtpcweHkylFdWRnvOZhdf4G3pbCUzZZIKNO9IQ8hU5TauNH3TST PFIHi2W1vMOIv1/dBAuA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pWcBx-009VeF-Fy; Mon, 27 Feb 2023 12:01:19 +0000 Received: from mx1.tq-group.com ([93.104.207.81]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1pWcBh-009VXl-Nr for linux-arm-kernel@lists.infradead.org; Mon, 27 Feb 2023 12:01:04 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tq-group.com; i=@tq-group.com; q=dns/txt; s=key1; t=1677499262; x=1709035262; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=yraj+9NZxVEy7H/asyQnjEyahz4DMD1ru89T/gslw5c=; b=c68mv0961lPEZgBGS4ZsgE+pZnz6Dmkv4bouLtSagIx0OH8Y+IMg5Fl0 yHty21ocLKQ8qC5Pj2O9a6AeSzKgyydW5f6srO6P1hHJZVK/ZmtQMwXaB j0YDaQ/OsL+LB8QlS640EXL06a6A3cydYljBn1R9MJWtDdHeaqGPGqRlL WS2LkG3f19x7zSJF0SB78jlwnESery1OXacPanTNaZwdAWINwdfV4dIkq zz5COxW3y5w20beWjlQLrUOuYohYni6oXcZiukz2oT4T5uoWVrRZ/9H/w NClY20U0lRj1VS+5fkysl/kYyeSIt1CKM1bcxZyKsP/RRvDdWEWTqLoE4 Q==; X-IronPort-AV: E=Sophos;i="5.97,331,1669071600"; d="scan'208";a="29333946" Received: from unknown (HELO tq-pgp-pr1.tq-net.de) ([192.168.6.15]) by mx1-pgp.tq-group.com with ESMTP; 27 Feb 2023 13:00:57 +0100 Received: from mx1.tq-group.com ([192.168.6.7]) by tq-pgp-pr1.tq-net.de (PGP Universal service); Mon, 27 Feb 2023 13:00:57 +0100 X-PGP-Universal: processed; by tq-pgp-pr1.tq-net.de on Mon, 27 Feb 2023 13:00:57 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tq-group.com; i=@tq-group.com; q=dns/txt; s=key1; t=1677499257; x=1709035257; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=yraj+9NZxVEy7H/asyQnjEyahz4DMD1ru89T/gslw5c=; b=C5/rbEVjsW9kCZtFBxYj3YBrUyNkzk3Mi5Dm8iOf7fbazXfZwEdrhGZ3 C+NgvIOOJBWwTNYSBeS2AgAiIS/9WiRrnQZenGK0S5pm1a869wwTPyNCS cOoKbP03JjcfAzI4OZETut3N0mrVi4vXSEQr16RGEDlJBIVJReMjJR1sy Ji3SN5T+mJDDpaDyUGFCUodt+i3k3K9gKaFGO7rtGnmWWgnZim1ewsGhB oz840wCnnYHJu4GUmgpYv6vID2s1tVrHg390Y9K6Wfwg4T/HUi373SfJe x0Ib4gdQJ2I0ATvkp+xAM4C9AfqoojAMJLcHuV93NDmKiZJxLXqIkiL9j g==; X-IronPort-AV: E=Sophos;i="5.97,331,1669071600"; d="scan'208";a="29333945" Received: from vtuxmail01.tq-net.de ([10.115.0.20]) by mx1.tq-group.com with ESMTP; 27 Feb 2023 13:00:57 +0100 Received: from steina-w.localnet (unknown [10.123.53.21]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)) (No client certificate requested) by vtuxmail01.tq-net.de (Postfix) with ESMTPSA id EDBBD280056; Mon, 27 Feb 2023 13:00:56 +0100 (CET) From: Alexander Stein To: Adam Ford Cc: linux-clk@vger.kernel.org, Marek Vasut , Luca Ceresoli , Abel Vesa , Fabio Estevam , Jacky Bai , Krzysztof Kozlowski , Laurent Pinchart , Lucas Stach , Marco Felsch , Michael Turquette , NXP Linux Team , Peng Fan , Pengutronix Kernel Team , Richard Cochran , Rob Herring , Sascha Hauer , Shawn Guo , Stephen Boyd , 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 Message-ID: <4260396.ejJDZkT8p0@steina-w> Organization: TQ-Systems GmbH In-Reply-To: References: <20230223171114.59164-1-marex@denx.de> <3160821.5fSG56mABF@steina-w> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230227_040102_381675_9176EC58 X-CRM114-Status: GOOD ( 28.48 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Am Montag, 27. Februar 2023, 12:38:10 CET schrieb Adam Ford: > On Mon, Feb 27, 2023 at 3:46 AM Alexander Stein > = > 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 > > > Tested-by: Luca Ceresoli > > > Signed-off-by: Marek Vasut > > > --- > > > Cc: Abel Vesa > > > Cc: Fabio Estevam > > > Cc: Jacky Bai > > > Cc: Krzysztof Kozlowski > > > Cc: Laurent Pinchart > > > Cc: Lucas Stach > > > Cc: Marco Felsch > > > Cc: Michael Turquette > > > Cc: NXP Linux Team > > > Cc: Peng Fan > > > Cc: Pengutronix Kernel Team > > > Cc: Richard Cochran > > > Cc: Rob Herring > > > Cc: Sascha Hauer > > > Cc: Shawn Guo > > > Cc: Stephen Boyd > > > 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 =3D > > = > > ; > > = > > > }; > > > = > > > + pgc_audio: power-domain@5 { > > > + #power-domain- > > = > > cells =3D <0>; > > = > > > + reg =3D > > = > > ; > > = > > > + clocks =3D <&clk > > = > > IMX8MP_CLK_AUDIO_ROOT>, > > = > > > + <&clk > > = > > IMX8MP_CLK_AUDIO_AXI>; > > = > > > + }; > > > + > > > = > > > pgc_gpu2d: power-domain@6 { > > > = > > > #power-domain- > > = > > cells =3D <0>; > > = > > > reg =3D > > = > > ; > > = > > > @@ -1119,6 +1126,152 @@ opp-1000000000 { > > > = > > > }; > > > = > > > }; > > > = > > > + aips5: bus@30c00000 { > > > + compatible =3D "fsl,aips-bus", "simple-bus"; > > > + reg =3D <0x30c00000 0x400000>; > > > + #address-cells =3D <1>; > > > + #size-cells =3D <1>; > > > + ranges; > > > + > > > + spba-bus@30c00000 { > > > + compatible =3D "fsl,spba-bus", "simple- > > = > > bus"; > > = > > > + reg =3D <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 =3D <1>; > > > + #size-cells =3D <1>; > > > + ranges; > > > + > > > + sai1: sai@30c10000 { > > > + compatible =3D "fsl,imx8mp- > > = > > sai", "fsl,imx8mq-sai"; > > = > > > + reg =3D <0x30c10000 0x10000>; > > > + interrupts =3D > = > > IRQ_TYPE_LEVEL_HIGH>; > > = > > > + clocks =3D <&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 =3D "bus", > > = > > "mclk0", "mclk1", "mclk2", "mclk3"; > > = > > > + dmas =3D <&sdma2 0 2 0>, > > = > > <&sdma2 1 2 0>; > > = > > Looking at Documentation/devicetree/bindings/dma/fsl,imx-sdma.yaml shou= ld > > the 2nd cell value be 24? This would be true for all other sai nodes. > > = > > > + dma-names =3D "rx", "tx"; > > > + status =3D "disabled"; > > > + }; > > > + > > > + sai2: sai@30c20000 { > > > + compatible =3D "fsl,imx8mp- > > = > > sai", "fsl,imx8mq-sai"; > > = > > > + reg =3D <0x30c20000 0x10000>; > > > + interrupts =3D > = > > IRQ_TYPE_LEVEL_HIGH>; > > = > > > + clocks =3D <&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 =3D "bus", > > = > > "mclk0", "mclk1", "mclk2", "mclk3"; > > = > > > + dmas =3D <&sdma2 2 2 0>, > > = > > <&sdma2 3 2 0>; > > = > > > + dma-names =3D "rx", "tx"; > > > + status =3D "disabled"; > > > + }; > > > + > > > + sai3: sai@30c30000 { > > > + compatible =3D "fsl,imx8mp- > > = > > sai", "fsl,imx8mq-sai"; > > = > > > + reg =3D <0x30c30000 0x10000>; > > > + interrupts =3D > = > > IRQ_TYPE_LEVEL_HIGH>; > > = > > > + clocks =3D <&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 =3D "bus", > > = > > "mclk0", "mclk1", "mclk2", "mclk3"; > > = > > > + dmas =3D <&sdma2 4 2 0>, > > = > > <&sdma2 5 2 0>; > > = > > > + dma-names =3D "rx", "tx"; > > > + status =3D "disabled"; > > > + }; > > > + > > > + sai5: sai@30c50000 { > > > + compatible =3D "fsl,imx8mp- > > = > > sai", "fsl,imx8mq-sai"; > > = > > > + reg =3D <0x30c50000 0x10000>; > > > + interrupts =3D > = > > IRQ_TYPE_LEVEL_HIGH>; > > = > > > + clocks =3D <&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 =3D "bus", > > = > > "mclk0", "mclk1", "mclk2", "mclk3"; > > = > > > + dmas =3D <&sdma2 8 2 0>, > > = > > <&sdma2 9 2 0>; > > = > > > + dma-names =3D "rx", "tx"; > > > + status =3D "disabled"; > > > + }; > > > + > > > + sai6: sai@30c60000 { > > > + compatible =3D "fsl,imx8mp- > > = > > sai", "fsl,imx8mq-sai"; > > = > > > + reg =3D <0x30c60000 0x10000>; > > > + interrupts =3D > = > > IRQ_TYPE_LEVEL_HIGH>; > > = > > > + clocks =3D <&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 =3D "bus", > > = > > "mclk0", "mclk1", "mclk2", "mclk3"; > > = > > > + dmas =3D <&sdma2 10 2 0>, > > = > > <&sdma2 11 2 0>; > > = > > > + dma-names =3D "rx", "tx"; > > > + status =3D "disabled"; > > > + }; > > > + > > > + sai7: sai@30c80000 { > > > + compatible =3D "fsl,imx8mp- > > = > > sai", "fsl,imx8mq-sai"; > > = > > > + reg =3D <0x30c80000 0x10000>; > > > + interrupts =3D > = > > IRQ_TYPE_LEVEL_HIGH>; > > = > > > + clocks =3D <&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 =3D "bus", > > = > > "mclk0", "mclk1", "mclk2", "mclk3"; > > = > > > + dmas =3D <&sdma2 12 2 0>, > > = > > <&sdma2 13 2 0>; > > = > > > + dma-names =3D "rx", "tx"; > > > + status =3D "disabled"; > > > + }; > > > + }; > > > + > > > + sdma3: dma-controller@30e00000 { > > > + compatible =3D "fsl,imx8mp-sdma", > > = > > "fsl,imx8mq-sdma"; > > = > > > + reg =3D <0x30e00000 0x10000>; > > > + interrupts =3D > = > > IRQ_TYPE_LEVEL_HIGH>; > > = > > > + clocks =3D <&audio_blk_ctrl > > = > > IMX8MP_CLK_AUDIOMIX_SDMA3_ROOT>, > > = > > > + <&clk > > = > > IMX8MP_CLK_AUDIO_ROOT>; > > = > > > + clock-names =3D "ipg", "ahb"; > > > + #dma-cells =3D <3>; > > > + fsl,sdma-ram-script-name =3D "imx/sdma/ > > = > > sdma-imx7d.bin"; > > = > > > + }; > > > + > > > + sdma2: dma-controller@30e10000 { > > > + compatible =3D "fsl,imx8mp-sdma", > > = > > "fsl,imx8mq-sdma"; > > = > > > + reg =3D <0x30e10000 0x10000>; > > > + interrupts =3D > = > > IRQ_TYPE_LEVEL_HIGH>; > > = > > > + clocks =3D <&audio_blk_ctrl > > = > > IMX8MP_CLK_AUDIOMIX_SDMA2_ROOT>, > > = > > > + <&clk > > = > > IMX8MP_CLK_AUDIO_ROOT>; > > = > > > + clock-names =3D "ipg", "ahb"; > > > + #dma-cells =3D <3>; > > > + fsl,sdma-ram-script-name =3D "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 > > = > > Best regards, > > Alexander > > = > > > + #clock-cells =3D <1>; > > > + compatible =3D "fsl,imx8mp-audio-blk- > > = > > ctrl"; > > = > > > + clocks =3D <&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 =3D "ahb", > > > + "sai1", "sai2", > > = > > "sai3", > > = > > > + "sai5", "sai6", > > = > > "sai7"; > > = > > > + power-domains =3D <&pgc_audio>; > > > + power-domain-names =3D "audio"; > > > + reg =3D <0x30e20000 0x10000>; > > > + }; > > > + }; > > > + > > > = > > > aips4: bus@32c00000 { > > > = > > > compatible =3D "fsl,aips-bus", "simple-bus"; > > > reg =3D <0x32c00000 0x400000>; > > = > > -- > > TQ-Systems GmbH | M=FChlstra=DFe 2, Gut Delling | 82229 Seefeld, Germany > > Amtsgericht M=FCnchen, HRB 105018 > > Gesch=E4ftsf=FChrer: Detlef Schneider, R=FCdiger Stahl, Stefan Schneider > > http://www.tq-group.com/ -- = TQ-Systems GmbH | M=FChlstra=DFe 2, Gut Delling | 82229 Seefeld, Germany Amtsgericht M=FCnchen, HRB 105018 Gesch=E4ftsf=FChrer: Detlef Schneider, R=FCdiger 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