From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 9DA1B1F19E for ; Mon, 2 Oct 2023 19:21:07 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 0B629C433C8; Mon, 2 Oct 2023 19:21:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1696274467; bh=WeIQDK9C+Lg7sfwz7aMXeYG+hNY0EwrPIT8ROstC1zE=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=ujxy3EA76/Hrc6fXpKGNxLtcvwKse/DIua954gfHgtf92hdzm5adhY3WNGPkgeSew xMIrKJLzdjfrFVbB0+SMTcLXRBgfebAbR+jRf9J8ECj/85zQJGi3/akE58aKiZM7yV Urh41IF6V9IB1e75adR1zz2JK4FOIVTgjulExHKMAt6hBym8Yd3602xKrcqGJuDfZM J67zMDGbmS1HOLGbacowB7DHavlAHMa5bi1orU35l34rXz/uB0qqqg6PIlGTlRKP39 jV0YK6rbSw/AR8Vp1UW4ZU+Q+rDNbH0fqQDtG4GHofC1HuoJyRSKMYK2BNZhgpFJre qRto3ILnFFLVg== Date: Mon, 2 Oct 2023 20:21:02 +0100 From: Conor Dooley To: "Balas, Eliza" Cc: Rob Herring , Krzysztof Kozlowski , Conor Dooley , Derek Kiernan , Dragan Cvetic , Arnd Bergmann , Greg Kroah-Hartman , "linux-kernel@vger.kernel.org" , "devicetree@vger.kernel.org" Subject: Re: [PATCH v2 1/2] dt-bindings: misc: adi,axi-tdd: Add device-tree binding for TDD engine Message-ID: <20231002-aggregate-scanner-3ca2ae69cf0c@spud> References: <20230928092804.22612-1-eliza.balas@analog.com> <20230928092804.22612-2-eliza.balas@analog.com> <20231002163244.GA1886905-robh@kernel.org> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="89n6cpjAet9oXeM3" Content-Disposition: inline In-Reply-To: --89n6cpjAet9oXeM3 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Oct 02, 2023 at 04:46:26PM +0000, Balas, Eliza wrote: > > -----Original Message----- > > From: Rob Herring > > Sent: Monday, October 2, 2023 19:33 > > To: Balas, Eliza > > Cc: Krzysztof Kozlowski ; Conor Dool= ey ; Derek Kiernan > > ; Dragan Cvetic ; Arnd Be= rgmann ; Greg Kroah-Hartman > > ; linux-kernel@vger.kernel.org; devicetree@= vger.kernel.org > > Subject: Re: [PATCH v2 1/2] dt-bindings: misc: adi,axi-tdd: Add device-= tree binding for TDD engine > >=20 > > [External] > >=20 > > On Thu, Sep 28, 2023 at 12:28:03PM +0300, Eliza Balas wrote: > > > Add device tree documentation for the AXI TDD Core. > > > The generic TDD controller is in essence a waveform generator > > > capable of addressing RF applications which require Time Division > > > Duplexing, as well as controlling other modules of general > > > applications through its dedicated 32 channel outputs. > > > > > > The reason of creating the generic TDD controller was to reduce > > > the naming confusion around the existing repurposed TDD core > > > built for AD9361, as well as expanding its number of output > > > channels for systems which require more than six controlling signals. > > > > > > Signed-off-by: Eliza Balas > > > --- > > > .../devicetree/bindings/misc/adi,axi-tdd.yaml | 65 +++++++++++++++++= ++ > > > MAINTAINERS | 7 ++ > > > 2 files changed, 72 insertions(+) > > > create mode 100644 Documentation/devicetree/bindings/misc/adi,axi-td= d.yaml > > > > > > diff --git a/Documentation/devicetree/bindings/misc/adi,axi-tdd.yaml = b/Documentation/devicetree/bindings/misc/adi,axi-tdd.yaml > > > new file mode 100644 > > > index 000000000000..8938da801b95 > > > --- /dev/null > > > +++ b/Documentation/devicetree/bindings/misc/adi,axi-tdd.yaml > > > @@ -0,0 +1,65 @@ > > > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) > > > +# Copyright 2023 Analog Devices Inc. > > > +%YAML 1.2 > > > +--- > > > +$id: https://urldefense.com/v3/__http://devicetree.org/schemas/misc/= adi,axi- > > tdd.yaml*__;Iw!!A3Ni8CS0y2Y!5Cxus2huppjhkiJZLWTpJEgA0IXdLZx4t0fS9J1yt0x= gjp9g3Y1N5PbZ7pAcIkKU4WPbS_TR9yjTKw$ > > > +$schema: https://urldefense.com/v3/__http://devicetree.org/meta- > > schemas/core.yaml*__;Iw!!A3Ni8CS0y2Y!5Cxus2huppjhkiJZLWTpJEgA0IXdLZx4t0= fS9J1yt0xgjp9g3Y1N5PbZ7pAcIkKU4WPbS_RK8aQ9xw$ > > > + > > > +title: Analog Devices AXI TDD Core > > > + > > > +maintainers: > > > + - Eliza Balas > > > + > > > +description: | > > > + The TDD controller is a waveform generator capable of addressing RF > > > + applications which require Time Division Duplexing, as well as con= trolling > > > + other modules of general applications through its dedicated 32 cha= nnel > > > + outputs. It solves the synchronization issue when transmitting and= receiving > > > + multiple frames of data through multiple buffers. > > > + The TDD IP core is part of the Analog Devices hdl reference design= s and has > > > + the following features: > > > + * Up to 32 independent output channels > > > + * Start/stop time values per channel > > > + * Enable and polarity bit values per channel > > > + * 32 bit-max internal reference counter > > > + * Initial startup delay before waveform generation > > > + * Configurable frame length and number of frames per burst > > > + * 3 sources of synchronization: external, internal and software = generated > > > + For more information see the wiki: > > > + https://wiki.analog.com/resources/fpga/docs/axi_tdd > > > + > > > +properties: > > > + compatible: > > > + enum: > > > + - adi,axi-tdd-2.00.a > >=20 > > Where does this version number come from? I looked at the above link and > > see versions such as '2021_R2', '2019_r2', etc. I didn't dig deeper > > whether there's some per IP version. > >=20 > > If you want to use version numbers, please document the versioning > > scheme. For example, see > > Documentation/devicetree/bindings/sifive/sifive-blocks-ip-versioning.tx= t. > >=20 > > Rob >=20 > The version refers to the IP version. The version of the IP is also > specified in its VERSION register (there is a drop down to expand the > register map on the wiki page) which is verified by the driver during > probe. "2021_R2" refers to the compatible tool version used for > creating the FPGAIP Core. If you have version registers in these IPs, what benefit does version numbers in the compatible string bring? Rather than using the version numbers to validate what the DT gave you, which not the kernel's job IMO, why not just use the information from the register to determine the version? Cheers, Conor. --89n6cpjAet9oXeM3 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZRsYHgAKCRB4tDGHoIJi 0hNTAQDQKnn2rQiKo7g3PzTiisgKqyTMNHeg9GOBd8i9KiUhPAEAixAZhoF4bcci ysMUNIYsa5zOPzAy4Vd24Y4FftL65wQ= =zP/g -----END PGP SIGNATURE----- --89n6cpjAet9oXeM3--