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 D8F4EC5475B for ; Fri, 8 Mar 2024 19:06:41 +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:MIME-Version:In-Reply-To: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=wMBUsUxfQwfMqdZuKHk0lj+NbDJVZsrfKXahCUQrsuQ=; b=kgF+5OLcftAlVY wV6/8ij3w64Hym4r4dfMlizU0capRWP+AygnohRl1cz0RsxQTYV9UkKj7nIbF3mhKTMHjSky7cb2L t0xlxV9BNJpokwb7a+FcrHQfIeKcffIjtK8S4EhoV6KR/kAEK0R/xhmWfkX8aonlabcSnnyw3Uban 979l+197zkplC6VaqOJOGaOTsqXX8psQtTRz1j9jdCCBkqAOwi1324cUD1x4CJSmav4iYtJNC2xdY SOFZ0ndCD8prYkyTPVzxIfqedColsujt1AzUnXb56VYN3cm4jH8RMgSN+8ecbeCkzqgzD2HD/sr1V iuTByuQ0UvzOxYtql7gA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rifY2-0000000AwWg-3fEO; Fri, 08 Mar 2024 19:06:26 +0000 Received: from mail-he1eur01on0607.outbound.protection.outlook.com ([2a01:111:f400:fe1e::607] helo=EUR01-HE1-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1rifY0-0000000AwVQ-2GLW for linux-arm-kernel@lists.infradead.org; Fri, 08 Mar 2024 19:06:25 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=NDzgwGRSLlgKursziPnQ5CCEnicIvOTIheUskAwWIwSI501jHA8V2OA0AXPzMr1IHKrwKFWoWjQQv7pnbqO71yZnb/wvJkzHEqYC/KEKXVTVVmkHBqFYzHFQqq7ftB2538A4RiK+KI7VVqYYJ91UQj26ygofJrjdhcBwYBbSa6WFRXb1miRhVj1B2Z3RaY6oH2VHVlQncpa6ToQQx0SLfGZuDJZ8mqAJkulFr5yqqlhxSz/W3KYnKjBIjifQx3X6gZVLtitS1sxZXNrVxepYOp5T2VH4EXngxVJxZxRLi82ug2FQYX2xKOU1qMIU0YBeR16owkR5l76Wx8vvnDAz3w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=VJ2JuimjhDWtLpWqIZUi1mBn36cUBrg/OUvsNtfl0+M=; b=aeUmp4nLU4jXLAH7H9lGIek6zcA8zBJqc/kQX1Uk9aYd1R97FxI0Rgw3Y20TP245qhuoEdZ9CXGTCh+JmqDJrhYzKDoc9nOew3CH3nwxNbl3zwHiIOGJr/tcPbVE/utTGiMazFBHNefzDka2P0BfEnr/aRmxMrw5ggYrxhZdRwf+D31Iy0KUkEynkJ2Rkrr8ufAmagUIHMAUzW03+Cu2avrLeprzjg3eV+wDfaoFl0qxwiu6k2mHlUhhXFXJg6ZjYub3sOIFGxyIXPwPE2BtHA9ODfw6dlxuDBmQEC5kNBNuWxOYFuXJEDUfAG0r2upODUCADEym25h40Fxv0uluIQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nxp.com; dmarc=pass action=none header.from=nxp.com; dkim=pass header.d=nxp.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=VJ2JuimjhDWtLpWqIZUi1mBn36cUBrg/OUvsNtfl0+M=; b=SyAUd7GSMSLPrtNJuKwNTj5xzbMpCLGGtkxJDJRNxUpFUHBpZjsU06rvJPq8RZ+PL5RVotLNOMniv403Ts7hicPtn6rG9ZseHgL/nLqdW789VdhY5ZhtHimgwxCjfQcYyfRXawef6goJt5s3xcH9snPfn/0olQYN9xC6r78BXCA= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nxp.com; Received: from PAXPR04MB9642.eurprd04.prod.outlook.com (2603:10a6:102:240::14) by PAWPR04MB9742.eurprd04.prod.outlook.com (2603:10a6:102:389::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7362.26; Fri, 8 Mar 2024 19:06:16 +0000 Received: from PAXPR04MB9642.eurprd04.prod.outlook.com ([fe80::3168:91:27c6:edf6]) by PAXPR04MB9642.eurprd04.prod.outlook.com ([fe80::3168:91:27c6:edf6%3]) with mapi id 15.20.7362.024; Fri, 8 Mar 2024 19:06:16 +0000 Date: Fri, 8 Mar 2024 14:06:08 -0500 From: Frank Li To: Conor Dooley Cc: Liam Girdwood , Mark Brown , Rob Herring , 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, Conor Dooley Subject: Re: [PATCH v6 3/4] ASoC: dt-bindings: fsl-sai: allow only one dma-names Message-ID: References: <20240308-asrc_8qxp-v6-0-e08f6d030e09@nxp.com> <20240308-asrc_8qxp-v6-3-e08f6d030e09@nxp.com> <20240308-croon-goofball-797d091e981a@spud> Content-Disposition: inline In-Reply-To: <20240308-croon-goofball-797d091e981a@spud> X-ClientProxiedBy: SJ0PR13CA0227.namprd13.prod.outlook.com (2603:10b6:a03:2c1::22) To PAXPR04MB9642.eurprd04.prod.outlook.com (2603:10a6:102:240::14) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: PAXPR04MB9642:EE_|PAWPR04MB9742:EE_ X-MS-Office365-Filtering-Correlation-Id: 1bba068c-314e-48ce-9712-08dc3fa2d48d X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: vSfuTM/CYxf8vY4QoIC4KZMcuj6n3YQ/7Rj+9xJ6X8Hdc81uAXDMcvH6iWksEnusl8JPlSI6oxH1Vlz2PyO0e7LhBMOYdcLgb3SkHOoJF883beYE7JUpJ7i6vCjEgeIpyArCk9MwtXJZJ413NxQ2w4P4AxvIGd50uvPQlEwh/GU3WC2nyW3XHe1jh3HRkYjS//xpCZpUpgjWNBIpyyyHeUYHo2mthvFBWyyKopvTYkJE75OqzVaQ35h1LCpQGi6nbRVSRHEtMcoouSP3w9ijUnFfqxUuGZF/5bndXBjuYI61NiLxPXw+p+o8ncMjskRry5oKZ25DkgDTeUfqkgQh6kOyWbpuBZWT0hp+yfITpMZMN6FUooABI8HU/8PuTH60Sv03s2knfV5jliTUjaEL8NI4xX3z7I0oAf0mw2RYMXA8YqWjo4deK2ln47gqb5mx+XNzeI6w6snc4bZ4M0bhIf2MIqMRGat/EYYmfMPFQrSpih4x0VbHtTI3A6XtUL/mz5pw1p2o3pDqdPMuzc/SWartTB46/8NsE09YxBkzDCD9Mw2iUjdPEgoi2skB/7an4lK9aqCPBUO4J+K0BPREUkS1TqXOOndqoyfsb2JhVQy2X5pWpUAIY2MfogjPTPBXZF/GX9zLi7IaEnYoMGSQ7/TVj08zBYcRWaoWvzDROBODo9PGXEx312zaKuNvRqyieodoNSwOjq2Fhklzx+waV6k2cjAoI8KBO5hsi3BoQRg= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:PAXPR04MB9642.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(52116005)(1800799015)(376005)(7416005)(38350700005);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?71uXvJDfNhtqeCvjLSZel90Hpi5SEOtIVFl21/EvtHjamN7Zb/i4vucfrVvM?= =?us-ascii?Q?cLB5qzJGjtscmzbw1L4xPPRMHIIEqZc+I8b9vS63YQMCuGgzG3b076Xg8Yhq?= =?us-ascii?Q?IzoNCjg3kB+W+zbDcAyjOL6czMiNvqy0QlISfp2c5Cwa1UapvuAYncjNQsnU?= =?us-ascii?Q?Tys8s+iwQA3XiXuNu6rWIOwMn3NLNWI0x5JmPxjWOm0oMHYg3PIj1kC6L18U?= =?us-ascii?Q?G9U9EzIULUuBIjXmeHVhDSsDLHKdaQvefu7XsGaAS7VuS0fywwgFvlyW0Kj5?= =?us-ascii?Q?x6j5LSoJ0Qk6l6qd02cvV8Lk/uyeRvKYSuZrG8/3bsUYi0jP5D4w7t/+KYrG?= =?us-ascii?Q?Q6WQW8+Slt8TFssYuvfN9e5pOEEix1b1TNC/w28sFc5cBNY/PmCFFlb099G+?= =?us-ascii?Q?r5Tm7hZL87BIWpnw4IlTPB9fXj0TrchzSJjLWpequJC10RnHsZhFR7B2NWvU?= =?us-ascii?Q?uJ2I1m/maeREqj6sLj0QXqp5Pkk/RufnS/3rFl/9XweABwjN1PG0q8WSeqxL?= =?us-ascii?Q?ogfyM043UyYMslPS/2digt3TTfxsNLW9mkkziL5a7VIxJupg2shcKQPhvU+C?= =?us-ascii?Q?yZzK1BDS+2YulFaiSJf0WPzu6JktZY7ibvo35PGedRTcGAazCRy5sCTxJQuC?= =?us-ascii?Q?Erpdu8tpFrbi+chKbYrJOIcCIp8geKvg7eU85vi58UQ97ongNQMRSMEUQKNC?= =?us-ascii?Q?OqHPfmwSIOH0sbk3yUxK0e79/3u6KK+XbqP1e5OF5fPaJVDghGxyEDmH/a22?= =?us-ascii?Q?Fy9t2FRSYWQ7eWwmK/srzPN5QSdrF6nRUK4iXyasHvCWTD5DnRbjvZy6FKi7?= =?us-ascii?Q?RhQjT+2oyTBgNaCVvNe3lRH4+EcGw/x8oK3SwnapiEkW8d8uhBxARZV4zKyB?= =?us-ascii?Q?IPCmoGiE/wlebdt6wgBtqc4VFAhvp2/NyLHcW019e6ugcCvszssQaLp5eN4z?= =?us-ascii?Q?9QrVNQlQzDdRywF5YJySMhpG2ztMNpo9GwV66t+pAmQu3eyDihenwKHuq2t/?= =?us-ascii?Q?zGDniqTzHJVhkQnW3mABGOvLCLpTrUCGGBC65xo3F3XKAbiksW8nZ3bTDl5k?= =?us-ascii?Q?JukkSI/htRYu0pTYzYGvSOAe475wHz4Ui+6OK+HMGx7o8YfPSF7Ka3cPWsLU?= =?us-ascii?Q?5voYr+3ZyoAO31glsNTATHdV8LZjGlBZuYwCTDyB9MK67G8pnH3OzKFnL46G?= =?us-ascii?Q?fmMd2YBAZLxit8lhqFhR/p4o+C0gMbReANslx+UMW2cihGXrUKaTkwwyePFT?= =?us-ascii?Q?9kvsDV8/VosVl3KrF4s1bdS1rLXXSpZ4FeKTj/XcA4oc+n8LcRQQWukvlSyt?= =?us-ascii?Q?A8g2WA0kciisJ/FmFDB21qVyOqQBHaCi9CVr+d6kFDNMf7fT1VFu3gOmVsAb?= =?us-ascii?Q?WotJgZpWuS7jTzXscw1/vmVLXW2VVdSppO5xCli6nkqc+dsE7DiPqGdSgv/U?= =?us-ascii?Q?1YDkpJqDhKD/flIxbVyV6DPi5aClVOPEclToM+HhKUJ4CHzIbscFTbKnK1Jn?= =?us-ascii?Q?RB5u35LXzZGWKjwov52yTC7J5RGiGLW781qPzEHF8eFqqOT70D8o8dWTkGEt?= =?us-ascii?Q?Gv08kQEGCVqKtTT7QWyWC3ZQ/C2BsEmotMSpRMiC?= X-OriginatorOrg: nxp.com X-MS-Exchange-CrossTenant-Network-Message-Id: 1bba068c-314e-48ce-9712-08dc3fa2d48d X-MS-Exchange-CrossTenant-AuthSource: PAXPR04MB9642.eurprd04.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 Mar 2024 19:06:16.4110 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 2jTUhd05jsWcDcFFO2v8ZkPMR3Y2O1s+tdD+QK6UaY4eaecUynxQ7O4qwN4/FfMPkMUZjq2/c/M3mW/xzNn+Aw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAWPR04MB9742 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240308_110624_735763_DA883926 X-CRM114-Status: GOOD ( 26.30 ) 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 Fri, Mar 08, 2024 at 04:58:16PM +0000, Conor Dooley wrote: > On Fri, Mar 08, 2024 at 10:30:52AM -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 > > Please drop my ack from this, this isn't the patch I acked originally > and we were having a conversation as recently as yesterday on v4 about > this patch because Rob didn't like this approach. His suggestion is > better than the one I gave on v4 that you have used here. Why do you think Rob don't like this approach? He just said this is 3rd method. And it is simple enough and match all restriction. Frank Li > > Please give it a few days between resubmissions, and do not send new > versions of a series every day. It is very confusing to have different > conversions on three versions of this patch in my inbox all at the same > time. Okay, I think it is just simple fix and almost everything already close to settle down. Frank > > Thanks, > Conor. > > > --- > > Documentation/devicetree/bindings/sound/fsl,sai.yaml | 8 +++----- > > 1 file changed, 3 insertions(+), 5 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/sound/fsl,sai.yaml b/Documentation/devicetree/bindings/sound/fsl,sai.yaml > > index 2456d958adeef..93e7737a49a7b 100644 > > --- a/Documentation/devicetree/bindings/sound/fsl,sai.yaml > > +++ b/Documentation/devicetree/bindings/sound/fsl,sai.yaml > > @@ -81,15 +81,13 @@ 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 > > + maxItems: 2 > > items: > > - - const: rx > > - - const: tx > > + enum: [ rx, tx ] > > > > interrupts: > > items: > > > > -- > > 2.34.1 > > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel