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 89242C54E41 for ; Wed, 6 Mar 2024 20:55:42 +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:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=L9xtYQQL8yD9gnfdqaa86wnSX3QKghYSvd7czrIiTy8=; b=i95TT/HZRvlVA6 DeDXdn0z/7gTTyBrpi4/OLXmWD6TljOB18ajQaHI2z35vnq9hmT9M4SFO2Xq4eDJXw81IJSlbgAT5 pegNkqCteKbMcdAQeQV7rUybqR4nRV01wxO+dEoeJwJjynk+WDlYZpGCfjOLcsEE+tTkQzCgOHelQ dJDHQT1Esy2VMYtpMhHRp+53xAtvzmg8dFunV12uApWMPc8NbB0n+K6frBawHDpTXO1OFXyVswHRe sxtcTtcwDFd4oB5ekb+t8mI6/RMdlsCo8rEtGkAbC8lT+VP36gmqaCIMqm0FrmtIRu/rfxHA+sh/v 49XOXbq64DpSyHK/ylpw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rhyIV-00000001ruK-1Bks; Wed, 06 Mar 2024 20:55:31 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1rhyIS-00000001rtp-3RKT for linux-arm-kernel@lists.infradead.org; Wed, 06 Mar 2024 20:55:30 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by dfw.source.kernel.org (Postfix) with ESMTP id 720C861B1F; Wed, 6 Mar 2024 20:55:27 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id B9114C433F1; Wed, 6 Mar 2024 20:55:26 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1709758526; bh=8wpIf1u7ZRpALOfHZ4AX6o9s1PVu3WXuDX1nZ0s4XSc=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=mXF5lxxx+5t7+86g9lXRI/nNc8B4ygRS3Pb/kEY2lndTQUNMt4NdY7rUvEMCXmXQl vxmyt4AOGSH2J4BP3G5owS+NwYYzYVNAi7VZYYC94rGw+DYdQFHddWlgmPeVdHI8e5 TO7VYXRFl7FeITijsC6W/rxrVtrmpB3/n7h3O0uJGKcoF7CRFZnbC5m2BZD+1zwpuh zt5ZPUwn/ak239yElwk+gNSlGGsBVgGtYpsdxO76K4qrOOJlgOJrZVidGFIn2F+MVm w0zqPA++4MN6nCQbqOK5WuxVp7d/V16f5iYor8Lvr8kR2Chbaj4VvihYTvX3vsDAx6 hUYzM5k4d/n0Q== Date: Wed, 6 Mar 2024 14:55:24 -0600 From: Rob Herring To: Frank Li Cc: Conor Dooley , Liam Girdwood , Mark Brown , Krzysztof Kozlowski , Conor Dooley , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , Shengjiu Wang , 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 v4 3/4] ASoC: dt-bindings: fsl-sai: allow only one dma-names Message-ID: <20240306205524.GB587561-robh@kernel.org> References: <20240305-asrc_8qxp-v4-0-c61b98046591@nxp.com> <20240305-asrc_8qxp-v4-3-c61b98046591@nxp.com> <20240306-pebble-grope-88fdaa95a87c@spud> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240306_125529_005252_09A97978 X-CRM114-Status: GOOD ( 25.01 ) 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="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Wed, Mar 06, 2024 at 02:25:53PM -0500, Frank Li wrote: > On Wed, Mar 06, 2024 at 06:45:13PM +0000, Conor Dooley wrote: > > On Tue, Mar 05, 2024 at 12:33:04PM -0500, Frank Li wrote: > > > Some sai only connect one direction dma (rx/tx) in SOC. For example: > > > imx8qxp sai5 only connect tx dma channel. So allow only one "rx" or "tx" > > > for dma-names. > > > > > > Remove description under dmas because no user use index to get dma channel. > > > All user use 'dma-names' to get correct dma channel. dma-names already in > > > 'required' list. > > > > Acked-by: Conor Dooley > > > > > Signed-off-by: Frank Li > > > --- > > > Documentation/devicetree/bindings/sound/fsl,sai.yaml | 13 ++++++------- > > > 1 file changed, 6 insertions(+), 7 deletions(-) > > > > > > diff --git a/Documentation/devicetree/bindings/sound/fsl,sai.yaml b/Documentation/devicetree/bindings/sound/fsl,sai.yaml > > > index 2456d958adeef..6f551c68d33db 100644 > > > --- a/Documentation/devicetree/bindings/sound/fsl,sai.yaml > > > +++ b/Documentation/devicetree/bindings/sound/fsl,sai.yaml > > > @@ -81,15 +81,14 @@ properties: > > > > > > dmas: > > > minItems: 1 > > > - items: > > > - - description: DMA controller phandle and request line for RX > > > - - description: DMA controller phandle and request line for TX > > > + maxItems: 2 > > > > > > dma-names: > > > - minItems: 1 > > > - items: > > > - - const: rx > > > - - const: tx > > > + oneOf: > > > + - items: > > > + - const: rx > > > + - const: tx > > > + - enum: [ rx, tx ] > > > > I'm not entirely sure if this was Rob's suggestion, I got the impression > > he was suggesting that in the two items case we'd not care about the > > order. But while I think this is different to that suggestion it's also > > not wrong. > > I log this at cover-letter. b4 can't support write change log at every > patch yet. It never will (probably). That's because it doesn't need to. You can just do it with git. When you edit the commit message, then after the tags, Add '---' and put whatever you want after. That works as long as the commit is applied from a patch as 'git am' will drop it. > Rob's suggest was not work. dt-binding check complain too long > if there are two dma-names = "rx", "tx". So I'm wrong or you didn't have it correct? No way to tell with your explanation. Let me give you the exact schema: dma-names: minItems: 1 items: - enum: [ rx, tx ] - const: tx This says we can have 1 or 2 entries. The first entry can be either rx or tx. The 2nd entry must be tx. That's what you want. However, '"tx", "tx"' is allowed with the above, but we enforce items to be unique elsewhere. Or I thought we did, but we relaxed '.*-names$' at some point. I'm going to fix that now. Rob _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel