Received-SPF: pass (google.com: domain of linux-kernel+bounces-91211-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Date: Mon, 4 Mar 2024 20:13:06 +0000
From: Conor Dooley <conor@kernel.org>
To: Frank Li <Frank.li@nxp.com>
Cc: Liam Girdwood <lgirdwood@gmail.com>, Mark Brown <broonie@kernel.org>,
	Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Conor Dooley <conor+dt@kernel.org>, Shawn Guo <shawnguo@kernel.org>,
	Sascha Hauer <s.hauer@pengutronix.de>,
	Pengutronix Kernel Team <kernel@pengutronix.de>,
	Fabio Estevam <festevam@gmail.com>,
	Shengjiu Wang <shengjiu.wang@nxp.com>, linux-sound@vger.kernel.org,
	devicetree@vger.kernel.org, imx@lists.linux.dev,
	linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 3/4] ASoC: dt-bindings: fsl-sai: allow only one
 dma-names
Message-ID: <20240304-cape-saloon-80f241bfd6df@spud>
References: <20240227-asrc_8qxp-v2-0-521bcc7eb1c0@nxp.com>
 <20240227-asrc_8qxp-v2-3-521bcc7eb1c0@nxp.com>
 <20240229-husband-penalty-8c1ab0f57f55@spud>
 <20240229-rundown-isotope-954ba9ea4c57@spud>
 <ZeDdMJlxBL4SGkws@lizhi-Precision-Tower-5810>
 <20240301-crudeness-resale-3c0a1228850d@spud>
 <ZeIGXEJ3l4tgjmxT@lizhi-Precision-Tower-5810>
 <20240301-deluxe-tiptoeing-741af7d620b9@spud>
 <ZeIPmGG7+5cKZkO3@lizhi-Precision-Tower-5810>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
	protocol="application/pgp-signature"; boundary="G/bhWJ7YMFFq+jSj"
Content-Disposition: inline
In-Reply-To: <ZeIPmGG7+5cKZkO3@lizhi-Precision-Tower-5810>


--G/bhWJ7YMFFq+jSj
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Fri, Mar 01, 2024 at 12:25:44PM -0500, Frank Li wrote:
> On Fri, Mar 01, 2024 at 05:07:07PM +0000, Conor Dooley wrote:
> > On Fri, Mar 01, 2024 at 11:46:20AM -0500, Frank Li wrote:
> > > On Fri, Mar 01, 2024 at 04:05:25PM +0000, Conor Dooley wrote:
> > > > On Thu, Feb 29, 2024 at 02:38:24PM -0500, Frank Li wrote:
> > > > > On Thu, Feb 29, 2024 at 06:57:29PM +0000, Conor Dooley wrote:
> > > > > > On Thu, Feb 29, 2024 at 06:55:58PM +0000, Conor Dooley wrote:
> > > > > > > On Tue, Feb 27, 2024 at 03:54:11PM -0500, Frank Li wrote:
> > > > > > > > Some sai only connect one direction. So allow only "rx" or =
"tx" for
> > > > > > > > dma-names.
> > > > > > >=20
> > > > > > > Which sai? Can you restrict this per compatible please, so th=
at someone
> > > > > > > cannot add 2 dmas for ones where only the tx is supported.
> > > > > > >=20
> > > > > > > |  dmas:
> > > > > > > |    minItems: 1
> > > > > > > |    items:
> > > > > > > |      - description: DMA controller phandle and request line=
 for RX
> > > > > > > |      - description: DMA controller phandle and request line=
 for TX
> > > > > > >=20
> > > > > > > The binding already allows only one, but it documents that th=
e first dma
> > > > > > > is always the RX dma, and that doesn't change with this patch=
=2E.
> > > > > >=20
> > > > > > I said "doesn't change" - but I don't think you can change this
> > > > > > trivially, as something could rely on the first dma being the r=
x one.
> > > > > > You'd have to check that there is nothing using these using ind=
ices
> > > > > > rather than names before making any changes here.
> > > > >=20
> > > > > Linux driver and dts with tx only work well. Only issue is dtb_ch=
eck will
> > > > > report error. I want to eliminate these DTB_CHECK warning.
> > > >=20
> > > > Linux is not the only user of these bindings, citing linux as your
> > > > evidence here is only sufficient if no other users exist. Do they?
> > >=20
> > > But, 'dmas' should be common property for all these bindings? I don't=
 think
> > > they use 'descriptions:' property, which should guide dts writer to w=
rite
> > > dts file. actually words 'DMA controller phandle and request line' ju=
st
> > > nonsense words. let 'regs', it'd better descript at 'reg-names' inste=
ad
> > > of 'regs' if reg-names exist. Only meansful words is "RX" and "TX", w=
hich
> > > already show at "dma-names".
> >=20
> > None of this matters. If there's a documented order for these, which
> > there is, software is not obligated to use the names and can rely on the
> > order alone. You need to check that there are no other users which will
> > be broken by your proposed change.
>=20
> As my best knowledge, only linux use this binding.

If you've checked and there are no users, that's fine. Please mention
in the commit message that there are no users that will be affected as
justification for the ABI change.

--G/bhWJ7YMFFq+jSj
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZeYrUgAKCRB4tDGHoIJi
0kXGAP0ZXQDdSBHtYgiW6KEtRSeHnNW+Qzp4Xi6aG6/WngSB7wEAkXcAGBBLZxUQ
3gj8XZWrnoAYzbHNua6AJZOzWa0FEQY=
=OdYx
-----END PGP SIGNATURE-----

--G/bhWJ7YMFFq+jSj--