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 CA9C2CDB465 for ; Thu, 19 Oct 2023 13:32:50 +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=2dFAQ48WcQ6ZejoyAzYD9kEIvfBrz9QkOzehsDQcSp0=; b=suZmkftVXle6W9G5RbFbN57p88 crryhSBX/857Aj8KlZvWi1jhVfp4BWlvg4lAkvtsIGfUdxOstJsi7SpZSJvk2Z8JgKBV8maXxMLr4 DKWU/ie5zUikx30IQ8cfoKBg5Lf0qIB+skaHTYV3vab3eYqZmW68REnqZaZi1TXoS1oZ0OSvarVyF FUEBRYa4sFgwPl9v1JdJ3nRaUZt/xb1wfnc0pMK/VZ2WQgvVy/+2N912oAVO3LhV9O1LieBYEmY47 abbHMgoFXcQie0paXi7oyw3WpD4NoXH7yAAFvRuzbw0Vu655a/NF3EPt30Js1qwxgQ+s7XlH9uXeB 7EtMFrMA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qtT8Q-00HWuC-0A; Thu, 19 Oct 2023 13:32:22 +0000 Received: from dfw.source.kernel.org ([139.178.84.217]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qtT8M-00HWtX-1A for linux-arm-kernel@lists.infradead.org; Thu, 19 Oct 2023 13:32:19 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 4119F61344; Thu, 19 Oct 2023 13:32:16 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id C4F5CC433C8; Thu, 19 Oct 2023 13:32:12 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1697722335; bh=uIth58KOnu0F2t7Gf0oSpsZ5JWiWr+pq+5yI3KITDA4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=X17d+diE3S+X/LEd5hs1P1DF8P9iQVp3EhubvjOpHCldFPZHMdV19ZKrYCMXR9TwT m24rPnTJRkMnhgpACX9lyx1G81dewsPRKPNB/vvSuffFcPNWFMqrh3TQx7aDNJ1lUP TGtXl0QmjvKnJyxO53jjPIrkG2sIZFhybhGhxlooU0HfBaYzIPvqsSk3S+00raaJB4 sktv1FtlDFNEGhyk2ttH573tK4bonpYiYJL/wn9F9YKEfE4xffH09AjiSb/njm6eKU Aat69T6/wax74wtjrUAjebIq2787Gnb1y/lF2DopgQJtloDI2Hc9gUbBjcNd2y8kpg hjsOZIeB5J7mQ== Date: Thu, 19 Oct 2023 14:32:10 +0100 From: Conor Dooley To: Peng Fan Cc: "Peng Fan (OSS)" , "robh+dt@kernel.org" , "krzysztof.kozlowski+dt@linaro.org" , "conor+dt@kernel.org" , "shawnguo@kernel.org" , "s.hauer@pengutronix.de" , "sboyd@kernel.org" , "abelvesa@kernel.org" , "kernel@pengutronix.de" , "estevam@gmail.com" , dl-linux-imx , "devicetree@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "inux-clk@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH 1/2] dt-bindings: clock: support i.MX93 Anatop Message-ID: <20231019-president-gallantly-1b509e6eb2eb@spud> References: <20231018084414.1950241-1-peng.fan@oss.nxp.com> <20231018-promenade-budding-3e228f241eb7@spud> MIME-Version: 1.0 In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20231019_063218_488508_9F777861 X-CRM114-Status: GOOD ( 26.40 ) 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="===============6485738016868828911==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============6485738016868828911== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="VtVv58AsgCqDexvh" Content-Disposition: inline --VtVv58AsgCqDexvh Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Oct 19, 2023 at 10:05:54AM +0000, Peng Fan wrote: > > Subject: Re: [PATCH 1/2] dt-bindings: clock: support i.MX93 Anatop > >=20 > > On Wed, Oct 18, 2023 at 04:44:13PM +0800, Peng Fan (OSS) wrote: > > > From: Peng Fan > > > > > > Support i.MX93 Anatop which generates PLL and feeds into CCM. > >=20 > > What is "CCM". How do you "generate" a PLL? >=20 > =20 > CCM: Clock Controller Module. > I may use produces PLL No, almost certainly it _is_ a PLL and produces a clock. > > > Signed-off-by: Peng Fan > > > --- > > > .../bindings/clock/fsl,imx93-anatop.yaml | 41 +++++++++++++++++= ++ > > > 1 file changed, 41 insertions(+) > > > create mode 100644 > > > Documentation/devicetree/bindings/clock/fsl,imx93-anatop.yaml > > > > > > diff --git > > > a/Documentation/devicetree/bindings/clock/fsl,imx93-anatop.yaml > > > b/Documentation/devicetree/bindings/clock/fsl,imx93-anatop.yaml > > > new file mode 100644 > > > index 000000000000..9585c9e4ee40 > > > --- /dev/null > > > +++ b/Documentation/devicetree/bindings/clock/fsl,imx93-anatop.yaml > > > @@ -0,0 +1,41 @@ > > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) %YAML 1.2 > > > +--- > > > +$id: http://devicetree.org/schemas/clock/fsl,imx93-anatop.yaml# > > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > > + > > > +title: NXP i.MX93 Anatop Module > >=20 > > I see there are also "anatop regulators" on some imx SoCs, just calling= this > > "anatop module" does not seem sufficiently descriptive. >=20 > Analog Module? How do you think? It'd probably more be descriptive, but really putting the PLL bit in the title would be helpful. I assume "anatop" is what this thing is called in the RTL for the device. Cheers, Conor. > =20 > >=20 > > > + > > > +maintainers: > > > + - Peng Fan > > > + > > > +description: | > >=20 > > Drop the |, you don't need it. >=20 > Ok. >=20 > >=20 > > > + NXP i.MX93 anatop PLL module which generates PLL to CCM root. > > > + > > > +properties: > > > + compatible: > > > + items: > > > + - const: fsl,imx93-anatop > > > + > > > + reg: > > > + maxItems: 1 > > > + > > > + '#clock-cells': > > > + const: 1 > > > + > > > +required: > > > + - compatible > > > + - reg > > > + - '#clock-cells' > > > + > > > +additionalProperties: false > > > + > > > +examples: > > > + - | > > > + anatop: clock-controller@44480000 { > >=20 > > and the label is is not used, so should be dropped. >=20 > Ok. >=20 > Thanks, > Peng. >=20 > >=20 > > Thanks, > > Conor. > >=20 > > > + compatible =3D "fsl,imx93-anatop"; > > > + reg =3D <0x44480000 0x2000>; > > > + #clock-cells =3D <1>; > > > + }; > > > + > > > +... > > > -- > > > 2.37.1 > > > --VtVv58AsgCqDexvh Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZTEv2gAKCRB4tDGHoIJi 0vTbAQCdtgf6jyJoIvRHR06DyWuImL0J27qLZptQv3FbnevShgD/X1Hf19mwJFfU rHS1cw91RIoJ2A48wjGbSFPtGnwd2gk= =F942 -----END PGP SIGNATURE----- --VtVv58AsgCqDexvh-- --===============6485738016868828911== 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 --===============6485738016868828911==--