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 alsa0.perex.cz (alsa0.perex.cz [77.48.224.243]) (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 E9CF8EE4996 for ; Mon, 21 Aug 2023 14:08:04 +0000 (UTC) Received: from alsa1.perex.cz (alsa1.perex.cz [207.180.221.201]) (using TLSv1.2 with cipher ADH-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by alsa0.perex.cz (Postfix) with ESMTPS id CFB1A83B; Mon, 21 Aug 2023 16:07:11 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz CFB1A83B DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1692626881; bh=jskCnmfcQS5EFi06b4333cYtIRY89QRwP0GiuGw2lz8=; h=Date:From:To:Cc:Subject:References:In-Reply-To:List-Id: List-Archive:List-Help:List-Owner:List-Post:List-Subscribe: List-Unsubscribe:From; b=OjDzU3XLl9GLm+eIgwYfl75gn0Fs9TQUqzz9dcGP14Ad8iVmT57K6HSMBrPBUl8hh JfCs/SFIN+mWeh38ZUFLLACgnppJ64PG6uJLxx2iOS1EuotFe6TmbLrJitCYPZtccM JSJN5InwKmZ6VP+NFT0G85QAaCM6JuHiMjKgQR8M= Received: by alsa1.perex.cz (Postfix, from userid 50401) id 9E8C3F80199; Mon, 21 Aug 2023 16:06:41 +0200 (CEST) Received: from mailman-core.alsa-project.org (mailman-core.alsa-project.org [10.254.200.10]) by alsa1.perex.cz (Postfix) with ESMTP id C0372F800F8; Mon, 21 Aug 2023 16:06:40 +0200 (CEST) Received: by alsa1.perex.cz (Postfix, from userid 50401) id B6D20F80199; Mon, 21 Aug 2023 16:06:35 +0200 (CEST) Received: from dfw.source.kernel.org (dfw.source.kernel.org [139.178.84.217]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by alsa1.perex.cz (Postfix) with ESMTPS id 8703DF800F8 for ; Mon, 21 Aug 2023 16:06:28 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz 8703DF800F8 Authentication-Results: alsa1.perex.cz; dkim=pass (2048-bit key, unprotected) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256 header.s=k20201202 header.b=OPZBm3I8 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 8FDE761849; Mon, 21 Aug 2023 14:06:25 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 64BCCC433C7; Mon, 21 Aug 2023 14:06:23 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1692626785; bh=jskCnmfcQS5EFi06b4333cYtIRY89QRwP0GiuGw2lz8=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=OPZBm3I8NJHoZf7lNV4qSpa5xDGS9Hj6d9J+HQ41DwnpKpgECp2/T0IvOCJNncVya KUOCkzttOoxWUVf7H+69kgVBsKZOKqz22xOIyUo7v+iP0131btrysBz0SMywPcyYhE r1LIquZBVqSczHC4FriOFXStKLlwWUx/900G6VzQA/QFZ5AJ+cAqk8rJs5nNpuXjH/ tj3V7Cvm/SvDWo97FwWn3Xcjh5X78b0j+JKRlXkGx1lUIhLdZYMK68JF7e8T4ZvBiR 5QCv5J7Z3bgb6+r7F0T8wA32Q/jw4GdjpCsRAZe18V0ZmeVy9c/BFOAhNVhnSzL53g +nsYCO0dsNUdA== Received: (nullmailer pid 1479558 invoked by uid 1000); Mon, 21 Aug 2023 14:06:22 -0000 Date: Mon, 21 Aug 2023 09:06:22 -0500 From: Rob Herring To: Adam Ford Cc: Krzysztof Kozlowski , linux-arm-kernel@lists.infradead.org, aford@beaconembedded.com, Liam Girdwood , Mark Brown , Krzysztof Kozlowski , Conor Dooley , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Shengjiu Wang , alsa-devel@alsa-project.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/3] ASoC: dt-bindings: fsl_easrc: Add support for imx8mp-easrc Message-ID: <20230821140622.GA1473812-robh@kernel.org> References: <20230820175655.206723-1-aford173@gmail.com> <6089b46d-a580-af52-0cac-84b46fe11e6c@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Message-ID-Hash: JGGGZY7U462NKTG4R4ZL7RM5AJDEU273 X-Message-ID-Hash: JGGGZY7U462NKTG4R4ZL7RM5AJDEU273 X-MailFrom: SRS0=A38r=EG=robh_at_kernel.org=rob@kernel.org X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; emergency; loop; banned-address; member-moderation; header-match-alsa-devel.alsa-project.org-0; header-match-alsa-devel.alsa-project.org-1; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header X-Mailman-Version: 3.3.8 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" Archived-At: List-Archive: <> List-Help: List-Owner: List-Post: List-Subscribe: List-Unsubscribe: On Sun, Aug 20, 2023 at 04:05:16PM -0500, Adam Ford wrote: > On Sun, Aug 20, 2023 at 3:33 PM Krzysztof Kozlowski > wrote: > > > > On 20/08/2023 22:32, Krzysztof Kozlowski wrote: > > > On 20/08/2023 19:56, Adam Ford wrote: > > >> The i.MX8MP appears to have the same easrc support as the Nano, so > > >> add imx8mp as an option with a fallback to imx8mn. > > >> > > >> Signed-off-by: Adam Ford > > >> > > >> diff --git a/Documentation/devicetree/bindings/sound/fsl,easrc.yaml b/Documentation/devicetree/bindings/sound/fsl,easrc.yaml > > >> index bdde68a1059c..2d53b3b10f2c 100644 > > >> --- a/Documentation/devicetree/bindings/sound/fsl,easrc.yaml > > >> +++ b/Documentation/devicetree/bindings/sound/fsl,easrc.yaml > > >> @@ -14,7 +14,11 @@ properties: > > >> pattern: "^easrc@.*" > > >> > > >> compatible: > > >> - const: fsl,imx8mn-easrc > > >> + oneOf: > > >> + - items: > > >> + - enum: > > >> + - fsl,imx8mp-easrc > > >> + - const: fsl,imx8mn-easrc > > > > > > You need here also const for fsl,imx8mn-easrc, otherwise you do not > > > allow it alone. Test it for fsl,imx8mn-easrc DTS - you will notice warnings. > > > > Actually, I see now Rob's report... you did not have to test DTS even. > > It was enough to test your change and this test was missing :(. Please > > test your changes before sending. > > For what it's worth, I did run 'make dt_binding_check', but I didn't > run it with the extra flags from Rob's e-mail. The tool didn't return > any errors. The error is not related to the '-m' (undocumented compatible) warning. It is as Krzysztof said. Rob 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 843A2EE4993 for ; Mon, 21 Aug 2023 14:06:53 +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=FJMOqxAbc9sUVId0hyVLdeARw6bbCT4aPVZyNDWDVcw=; b=ijb3FGhMM2/RFd 9knN7ov7I4s5gv2wdPCvZjiD4NropnJaRYKnenrKwI8U8omaT+SQ2iqDM9MNkwobX82+Ggp/Oa/rF Fku+dQ56BMTl41vq4D7NcW1JrWuM/xhkx81ducyymdbkicObnGf7pew3YLCS4EHzExQonX/3UcVRs Cs/Tz/gNPCN/72XDQtqtw6fPnDpZKFZxpYWxoQY2KziHV7qfnLKZpiWAj0gOixOWYpzAUkx01afxz d8/1q5Br4x5TbO2YlrN3moYDdMlLj1t3OKyQBYIQjj87G7r5odam7Y5rJxoqZxne350J3q7UmHAvo NTdgW+b/lxRb9vyh4/mg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qY5Y5-00E938-1o; Mon, 21 Aug 2023 14:06:29 +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 1qY5Y2-00E92M-1d for linux-arm-kernel@lists.infradead.org; Mon, 21 Aug 2023 14:06:27 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id 8FDE761849; Mon, 21 Aug 2023 14:06:25 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 64BCCC433C7; Mon, 21 Aug 2023 14:06:23 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1692626785; bh=jskCnmfcQS5EFi06b4333cYtIRY89QRwP0GiuGw2lz8=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=OPZBm3I8NJHoZf7lNV4qSpa5xDGS9Hj6d9J+HQ41DwnpKpgECp2/T0IvOCJNncVya KUOCkzttOoxWUVf7H+69kgVBsKZOKqz22xOIyUo7v+iP0131btrysBz0SMywPcyYhE r1LIquZBVqSczHC4FriOFXStKLlwWUx/900G6VzQA/QFZ5AJ+cAqk8rJs5nNpuXjH/ tj3V7Cvm/SvDWo97FwWn3Xcjh5X78b0j+JKRlXkGx1lUIhLdZYMK68JF7e8T4ZvBiR 5QCv5J7Z3bgb6+r7F0T8wA32Q/jw4GdjpCsRAZe18V0ZmeVy9c/BFOAhNVhnSzL53g +nsYCO0dsNUdA== Received: (nullmailer pid 1479558 invoked by uid 1000); Mon, 21 Aug 2023 14:06:22 -0000 Date: Mon, 21 Aug 2023 09:06:22 -0500 From: Rob Herring To: Adam Ford Cc: Krzysztof Kozlowski , linux-arm-kernel@lists.infradead.org, aford@beaconembedded.com, Liam Girdwood , Mark Brown , Krzysztof Kozlowski , Conor Dooley , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Shengjiu Wang , alsa-devel@alsa-project.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/3] ASoC: dt-bindings: fsl_easrc: Add support for imx8mp-easrc Message-ID: <20230821140622.GA1473812-robh@kernel.org> References: <20230820175655.206723-1-aford173@gmail.com> <6089b46d-a580-af52-0cac-84b46fe11e6c@linaro.org> 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-20230821_070626_654894_DDBF72BD X-CRM114-Status: GOOD ( 21.82 ) 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="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org T24gU3VuLCBBdWcgMjAsIDIwMjMgYXQgMDQ6MDU6MTZQTSAtMDUwMCwgQWRhbSBGb3JkIHdyb3Rl Ogo+IE9uIFN1biwgQXVnIDIwLCAyMDIzIGF0IDM6MzPigK9QTSBLcnp5c3p0b2YgS296bG93c2tp Cj4gPGtyenlzenRvZi5rb3psb3dza2lAbGluYXJvLm9yZz4gd3JvdGU6Cj4gPgo+ID4gT24gMjAv MDgvMjAyMyAyMjozMiwgS3J6eXN6dG9mIEtvemxvd3NraSB3cm90ZToKPiA+ID4gT24gMjAvMDgv MjAyMyAxOTo1NiwgQWRhbSBGb3JkIHdyb3RlOgo+ID4gPj4gVGhlIGkuTVg4TVAgYXBwZWFycyB0 byBoYXZlIHRoZSBzYW1lIGVhc3JjIHN1cHBvcnQgYXMgdGhlIE5hbm8sIHNvCj4gPiA+PiBhZGQg aW14OG1wIGFzIGFuIG9wdGlvbiB3aXRoIGEgZmFsbGJhY2sgdG8gaW14OG1uLgo+ID4gPj4KPiA+ ID4+IFNpZ25lZC1vZmYtYnk6IEFkYW0gRm9yZCA8YWZvcmQxNzNAZ21haWwuY29tPgo+ID4gPj4K PiA+ID4+IGRpZmYgLS1naXQgYS9Eb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3Mvc291 bmQvZnNsLGVhc3JjLnlhbWwgYi9Eb2N1bWVudGF0aW9uL2RldmljZXRyZWUvYmluZGluZ3Mvc291 bmQvZnNsLGVhc3JjLnlhbWwKPiA+ID4+IGluZGV4IGJkZGU2OGExMDU5Yy4uMmQ1M2IzYjEwZjJj IDEwMDY0NAo+ID4gPj4gLS0tIGEvRG9jdW1lbnRhdGlvbi9kZXZpY2V0cmVlL2JpbmRpbmdzL3Nv dW5kL2ZzbCxlYXNyYy55YW1sCj4gPiA+PiArKysgYi9Eb2N1bWVudGF0aW9uL2RldmljZXRyZWUv YmluZGluZ3Mvc291bmQvZnNsLGVhc3JjLnlhbWwKPiA+ID4+IEBAIC0xNCw3ICsxNCwxMSBAQCBw cm9wZXJ0aWVzOgo+ID4gPj4gICAgICBwYXR0ZXJuOiAiXmVhc3JjQC4qIgo+ID4gPj4KPiA+ID4+ ICAgIGNvbXBhdGlibGU6Cj4gPiA+PiAtICAgIGNvbnN0OiBmc2wsaW14OG1uLWVhc3JjCj4gPiA+ PiArICAgIG9uZU9mOgo+ID4gPj4gKyAgICAgIC0gaXRlbXM6Cj4gPiA+PiArICAgICAgICAgIC0g ZW51bToKPiA+ID4+ICsgICAgICAgICAgICAgIC0gZnNsLGlteDhtcC1lYXNyYwo+ID4gPj4gKyAg ICAgICAgICAtIGNvbnN0OiBmc2wsaW14OG1uLWVhc3JjCj4gPiA+Cj4gPiA+IFlvdSBuZWVkIGhl cmUgYWxzbyBjb25zdCBmb3IgZnNsLGlteDhtbi1lYXNyYywgb3RoZXJ3aXNlIHlvdSBkbyBub3QK PiA+ID4gYWxsb3cgaXQgYWxvbmUuIFRlc3QgaXQgZm9yIGZzbCxpbXg4bW4tZWFzcmMgRFRTIC0g eW91IHdpbGwgbm90aWNlIHdhcm5pbmdzLgo+ID4KPiA+IEFjdHVhbGx5LCBJIHNlZSBub3cgUm9i J3MgcmVwb3J0Li4uIHlvdSBkaWQgbm90IGhhdmUgdG8gdGVzdCBEVFMgZXZlbi4KPiA+IEl0IHdh cyBlbm91Z2ggdG8gdGVzdCB5b3VyIGNoYW5nZSBhbmQgdGhpcyB0ZXN0IHdhcyBtaXNzaW5nIDoo LiBQbGVhc2UKPiA+IHRlc3QgeW91ciBjaGFuZ2VzIGJlZm9yZSBzZW5kaW5nLgo+IAo+IEZvciB3 aGF0IGl0J3Mgd29ydGgsIEkgZGlkIHJ1biAnbWFrZSBkdF9iaW5kaW5nX2NoZWNrJywgYnV0IEkg ZGlkbid0Cj4gcnVuIGl0IHdpdGggdGhlIGV4dHJhIGZsYWdzIGZyb20gUm9iJ3MgZS1tYWlsLiAg VGhlIHRvb2wgZGlkbid0IHJldHVybgo+IGFueSBlcnJvcnMuCgpUaGUgZXJyb3IgaXMgbm90IHJl bGF0ZWQgdG8gdGhlICctbScgKHVuZG9jdW1lbnRlZCBjb21wYXRpYmxlKSB3YXJuaW5nLiAKSXQg aXMgYXMgS3J6eXN6dG9mIHNhaWQuCgpSb2IKCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fCmxpbnV4LWFybS1rZXJuZWwgbWFpbGluZyBsaXN0CmxpbnV4LWFy bS1rZXJuZWxAbGlzdHMuaW5mcmFkZWFkLm9yZwpodHRwOi8vbGlzdHMuaW5mcmFkZWFkLm9yZy9t YWlsbWFuL2xpc3RpbmZvL2xpbnV4LWFybS1rZXJuZWwK