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 173C3C04FFE for ; Tue, 14 May 2024 18:06:40 +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-Type: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:Cc:To:From:Date: Reply-To:Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date :Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=RpQy0I62Umb2RqalA14xd0ldEg+HcLwbRIypXLLvpT4=; b=tsIVvCStxUqU9IlJmAzZAWORfq d9W3oYWWtoiPMXzYYR3UI8yQTsAdyjFFtJ9VFtv9QNG3ED53XZ4fErqQUZvJbsEkE38KMo+AWpq/u g2Z3PTY6Qo4koyNP4ALoWIFY4HDaDJNWjcWFn64GuaShhpEST+ZkiM0OQfcdBxNKsvDO0U8jL03hW YiOKF/nzp8scgb8WoHiRqUSQZslnQ80Vli4l5jKXbU9ya6CLzQTiFSF42Z/LjkNIOuV+6mRX2cmle yHb5dJD9fq1j2qAsGycMEKOoZ0tztTluPGoTNk1jYjz/6HoJgDL8WMBX57X1Meer6R9dm0l3kUskb h4TBc7Ng==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1s6wXh-0000000GmGK-3K1C; Tue, 14 May 2024 18:06:25 +0000 Received: from sin.source.kernel.org ([2604:1380:40e1:4800::1]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1s6wXe-0000000GmFU-35Lv for linux-arm-kernel@lists.infradead.org; Tue, 14 May 2024 18:06:24 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sin.source.kernel.org (Postfix) with ESMTP id F26D9CE1184; Tue, 14 May 2024 18:06:20 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 6C8A9C2BD10; Tue, 14 May 2024 18:06:16 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1715709980; bh=1D7baXBKgZDMtYJArXbSDEg75oiAzjU9n1AllDE1/jA=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=dtDmOqPd0y5D56goykEk4q+29o4G52cyn17EUlae8b9AAKGV/nQugPcFxLxjdajys uSXbTVMi7rbcYOeRNcm4gcgwFkce/xHJRVOqOjMBbBD90CyI7QXzOs/VuyGbIjMgWI qgYOthRVzQHQUpg+21SGMtO3IBIWsbFFXfVePq+Sa/nx2Y8TEoYU/CSqDXgN/1mWzL It3E0eMmYN3ByX6MzxqjzBYPvqeq2M0REk019lGl4VeAoBBbldeervtdSe2tnUq/cp x2uepyY1P2EpET69f2+Yf9cZCAXEg5479aYZM7dO9zS3GJPot2JSOWd+9QEVS0HcwR JzT1N25quhF4g== Date: Tue, 14 May 2024 19:06:14 +0100 From: Conor Dooley To: Shengjiu Wang Cc: abelvesa@kernel.org, peng.fan@nxp.com, mturquette@baylibre.com, sboyd@kernel.org, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, shawnguo@kernel.org, s.hauer@pengutronix.de, kernel@pengutronix.de, festevam@gmail.com, marex@denx.de, linux-clk@vger.kernel.org, imx@lists.linux.dev, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, p.zabel@pengutronix.de, shengjiu.wang@gmail.com Subject: Re: [PATCH v3 3/6] dt-bindings: clock: imx8mp: Add reset-controller sub-node Message-ID: <20240514-campus-sibling-21cdf4c78366@spud> References: <1715679210-9588-1-git-send-email-shengjiu.wang@nxp.com> <1715679210-9588-4-git-send-email-shengjiu.wang@nxp.com> MIME-Version: 1.0 In-Reply-To: <1715679210-9588-4-git-send-email-shengjiu.wang@nxp.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240514_110623_220329_C0DDD959 X-CRM114-Status: GOOD ( 19.58 ) 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: multipart/mixed; boundary="===============5322261408636166010==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============5322261408636166010== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="/fS/DhJwdDL+x8Fz" Content-Disposition: inline --/fS/DhJwdDL+x8Fz Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, May 14, 2024 at 05:33:27PM +0800, Shengjiu Wang wrote: > The Audio Block Control contains clock distribution and gating > controls, as well as reset handling to several of the AUDIOMIX > peripherals. Especially the reset controls for Enhanced Audio > Return Channel (EARC) PHY and Controller. >=20 > So Audio Block Control is a Multi-Function Devices. >=20 > Add reset-controller sub node which is a reset provider for EARC. > Add compatible string "syscon", "simple-mfd" which make Audio > Block Control device support reset-controller sub-node. >=20 > Signed-off-by: Shengjiu Wang > --- > .../bindings/clock/imx8mp-audiomix.yaml | 17 +++++++++++++++-- > 1 file changed, 15 insertions(+), 2 deletions(-) >=20 > diff --git a/Documentation/devicetree/bindings/clock/imx8mp-audiomix.yaml= b/Documentation/devicetree/bindings/clock/imx8mp-audiomix.yaml > index 0a6dc1a6e122..a403ace4d11f 100644 > --- a/Documentation/devicetree/bindings/clock/imx8mp-audiomix.yaml > +++ b/Documentation/devicetree/bindings/clock/imx8mp-audiomix.yaml > @@ -15,7 +15,10 @@ description: | > =20 > properties: > compatible: > - const: fsl,imx8mp-audio-blk-ctrl > + items: > + - const: fsl,imx8mp-audio-blk-ctrl > + - const: syscon > + - const: simple-mfd > =20 > reg: > maxItems: 1 > @@ -44,6 +47,11 @@ properties: > ID in its "clocks" phandle cell. See include/dt-bindings/clock/imx= 8mp-clock.h > for the full list of i.MX8MP IMX8MP_CLK_AUDIOMIX_ clock IDs. > =20 > + reset-controller: > + type: object > + $ref: /schemas/reset/fsl,imx8mp-audiomix-reset.yaml# > + description: The child reset devices of AudioMIX Block Control. Why not just set #reset-cells =3D <1> in the existing node? IIRC it was already suggested to you to do that and use auxdev to set up the reset driver. Cheers, Conor. > + > required: > - compatible > - reg > @@ -60,7 +68,7 @@ examples: > #include > =20 > clock-controller@30e20000 { > - compatible =3D "fsl,imx8mp-audio-blk-ctrl"; > + compatible =3D "fsl,imx8mp-audio-blk-ctrl", "syscon", "simple-mf= d"; > reg =3D <0x30e20000 0x10000>; > #clock-cells =3D <1>; > clocks =3D <&clk IMX8MP_CLK_AUDIO_ROOT>, > @@ -74,6 +82,11 @@ examples: > "sai1", "sai2", "sai3", > "sai5", "sai6", "sai7"; > power-domains =3D <&pgc_audio>; > + > + reset-controller { > + compatible =3D "fsl,imx8mp-audiomix-reset"; > + #reset-cells =3D <1>; > + }; > }; > =20 > ... > --=20 > 2.34.1 >=20 --/fS/DhJwdDL+x8Fz Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZkOoFgAKCRB4tDGHoIJi 0mnkAQCfYx3ESpxuBfcV+o4yFqeGhopQBeQemIeoiZ7FFQmbeQEAt8BwyiI9fAVf T8+mnW1Tx66ZdvmZsib64ALlolQyUQ8= =XWvb -----END PGP SIGNATURE----- --/fS/DhJwdDL+x8Fz-- --===============5322261408636166010== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============5322261408636166010==--