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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 95CAFC433FE for ; Thu, 5 May 2022 15:01:42 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1380356AbiEEPFU (ORCPT ); Thu, 5 May 2022 11:05:20 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55014 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233555AbiEEPFR (ORCPT ); Thu, 5 May 2022 11:05:17 -0400 Received: from ams.source.kernel.org (ams.source.kernel.org [IPv6:2604:1380:4601:e00::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id DF0F622280; Thu, 5 May 2022 08:01:36 -0700 (PDT) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id 99E5DB82B7A; Thu, 5 May 2022 15:01:35 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 14047C385A4; Thu, 5 May 2022 15:01:31 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1651762894; bh=UUh4IAYvnUeRl0+FGz9aUqMk1V1C/5Mv1s6gce1m4PY=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=S8Y4NKxoe5tibidDLQ1YsIqidmaB/VrEldK+Y9cbMH5jS/+MM5rgrPY10yHnZuKpA Vx9PHYyx0ctjGVYN0AW2U+dj9sQF9nYeW19rmlBAaro91ZcvrnCEvQGSa4AOhWe4F+ AORKwgkynESpVtUsLkKVDuj91k0fG9nqnFkW+X74VADC85jI/SoExoUCj45dBUkPpw Xi936/6T6rBsBBkB/rBLDpE8NcvgzDUFSfZGqCKN4nIruGHpACikgCGzflWOu6DwGI kpfzOf2U/DAZwBv3TrR1gbINOSQxTB6NUvESyu3p4jsN3Y180ej0joxdrI7Y7frWDd SCwA7xo4UFLIw== Date: Thu, 5 May 2022 16:01:28 +0100 From: Mark Brown To: Codrin.Ciubotariu@microchip.com Cc: Nicolas.Ferre@microchip.com, alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, lars@metafoo.de, perex@perex.cz, tiwai@suse.com, robh+dt@kernel.org Subject: Re: [PATCH v3 6/6] ARM: configs: at91: sama7_defconfig: add MCHP PDMC and DMIC drivers Message-ID: References: <20220307122202.2251639-1-codrin.ciubotariu@microchip.com> <20220307122202.2251639-7-codrin.ciubotariu@microchip.com> <77c2b348-b7a6-458d-21b6-68c54efc317f@microchip.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="V6OcCftqYTAkRMmU" Content-Disposition: inline In-Reply-To: <77c2b348-b7a6-458d-21b6-68c54efc317f@microchip.com> X-Cookie: Real programs don't eat cache. Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --V6OcCftqYTAkRMmU Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, May 05, 2022 at 02:47:04PM +0000, Codrin.Ciubotariu@microchip.com w= rote: > On 05.05.2022 16:58, Nicolas Ferre wrote: > > I'm fine with that, but I see that some Kconfig entries "select" this= =20 > > SND_SOC_DMIC directly (amd, intel, mediatek, stm). > > If it's absolutely needed for PDMC to work, what about doing the same a= s=20 > > it would prevent some broken configurations? > The only way it makes sense to me to have this driver selected somewhere= =20 > is in a sound card driver, used for a specific board, which we know it=20 > has PDM microphones. Since, for now, we use the simple sound card for=20 > our audio interfaces, we have no place to add this select. > The reason I do not like to add this select under the controller driver,= =20 > as some of the vendors did, is because, in the future, we might have=20 > different PDM microphones that might not work with SND_SOC_DMIC and=20 > might need a different driver. > I don't have a strong opinion on this. If you think I am overthinking,=20 > please let me know and I will change this. It's unlikely but possible that there could be some other device connected (eg, you could have a DSP or something that generates PDM output). If the driver doesn't directly instantiate the DMIC itself then it's probably reasonable for it to be user controllable if the DMIC driver is there. --V6OcCftqYTAkRMmU Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmJz5scACgkQJNaLcl1U h9BoAQf/RxATTm9oBfzFYiQDmMZnY6nSvVtROk9XiCIp4BRvt4ezstNodFEGlLKQ fWhTWfMmr7rGF9Syj1Zb6W/2Fahc/R04bml6dXcDHBKeW5a9Tnslg4U0brWHd2oT SeE9ZVLQzoFgUcCh6DrwxpYIkfbPrZwrBS0YNHHgjw3LAc2voYJiNm8njcOXop9i +kpz1RZs60mULxSPGoNOr/9HNfNc2H3lZcSG6ZkdjkiXMJ4TjedzB5YpaOFGrYLN hCy9jHw/mzCFq9EnV8ubaTI05RFwIusQd0doLtRxlgrrS94+mwNqHggZLnFK95Z/ BnnQHFiY+49FUwRU4NPZhfcFayakzQ== =TVpY -----END PGP SIGNATURE----- --V6OcCftqYTAkRMmU-- 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 A808AC433F5 for ; Thu, 5 May 2022 15:02:37 +0000 (UTC) Received: from alsa1.perex.cz (alsa1.perex.cz [207.180.221.201]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by alsa0.perex.cz (Postfix) with ESMTPS id 92D721798; Thu, 5 May 2022 17:01:45 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz 92D721798 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1651762955; bh=UUh4IAYvnUeRl0+FGz9aUqMk1V1C/5Mv1s6gce1m4PY=; h=Date:From:To:Subject:References:In-Reply-To:Cc:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=TiADbsWUx/OqiCMS6TiGI7HjJd0vHowDuK7a1/1uj7tc6Z8ctTmnnuDubjT7GzNoG t72Nx1eu7KJlg9wVd5xrtwFseTGogpmaXwN8Dv9uLuxUqLfXkpsuH1GNWIchDcQmPC g/I+et8cOTF4iv0XZTWAIG2lQ3mkU8DH+tgkK8Po= Received: from alsa1.perex.cz (localhost.localdomain [127.0.0.1]) by alsa1.perex.cz (Postfix) with ESMTP id 34C90F800D3; Thu, 5 May 2022 17:01:45 +0200 (CEST) Received: by alsa1.perex.cz (Postfix, from userid 50401) id 5FB02F8016B; Thu, 5 May 2022 17:01:44 +0200 (CEST) Received: from ams.source.kernel.org (ams.source.kernel.org [IPv6:2604:1380:4601:e00::1]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by alsa1.perex.cz (Postfix) with ESMTPS id 495A9F800D3 for ; Thu, 5 May 2022 17:01:36 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz 495A9F800D3 Authentication-Results: alsa1.perex.cz; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="S8Y4NKxo" Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id A1783B82D82; Thu, 5 May 2022 15:01:35 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 14047C385A4; Thu, 5 May 2022 15:01:31 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1651762894; bh=UUh4IAYvnUeRl0+FGz9aUqMk1V1C/5Mv1s6gce1m4PY=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=S8Y4NKxoe5tibidDLQ1YsIqidmaB/VrEldK+Y9cbMH5jS/+MM5rgrPY10yHnZuKpA Vx9PHYyx0ctjGVYN0AW2U+dj9sQF9nYeW19rmlBAaro91ZcvrnCEvQGSa4AOhWe4F+ AORKwgkynESpVtUsLkKVDuj91k0fG9nqnFkW+X74VADC85jI/SoExoUCj45dBUkPpw Xi936/6T6rBsBBkB/rBLDpE8NcvgzDUFSfZGqCKN4nIruGHpACikgCGzflWOu6DwGI kpfzOf2U/DAZwBv3TrR1gbINOSQxTB6NUvESyu3p4jsN3Y180ej0joxdrI7Y7frWDd SCwA7xo4UFLIw== Date: Thu, 5 May 2022 16:01:28 +0100 From: Mark Brown To: Codrin.Ciubotariu@microchip.com Subject: Re: [PATCH v3 6/6] ARM: configs: at91: sama7_defconfig: add MCHP PDMC and DMIC drivers Message-ID: References: <20220307122202.2251639-1-codrin.ciubotariu@microchip.com> <20220307122202.2251639-7-codrin.ciubotariu@microchip.com> <77c2b348-b7a6-458d-21b6-68c54efc317f@microchip.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="V6OcCftqYTAkRMmU" Content-Disposition: inline In-Reply-To: <77c2b348-b7a6-458d-21b6-68c54efc317f@microchip.com> X-Cookie: Real programs don't eat cache. Cc: devicetree@vger.kernel.org, alsa-devel@alsa-project.org, lars@metafoo.de, tiwai@suse.com, Nicolas.Ferre@microchip.com, linux-kernel@vger.kernel.org, robh+dt@kernel.org, linux-arm-kernel@lists.infradead.org X-BeenThere: alsa-devel@alsa-project.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: "Alsa-devel mailing list for ALSA developers - http://www.alsa-project.org" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: "Alsa-devel" --V6OcCftqYTAkRMmU Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, May 05, 2022 at 02:47:04PM +0000, Codrin.Ciubotariu@microchip.com w= rote: > On 05.05.2022 16:58, Nicolas Ferre wrote: > > I'm fine with that, but I see that some Kconfig entries "select" this= =20 > > SND_SOC_DMIC directly (amd, intel, mediatek, stm). > > If it's absolutely needed for PDMC to work, what about doing the same a= s=20 > > it would prevent some broken configurations? > The only way it makes sense to me to have this driver selected somewhere= =20 > is in a sound card driver, used for a specific board, which we know it=20 > has PDM microphones. Since, for now, we use the simple sound card for=20 > our audio interfaces, we have no place to add this select. > The reason I do not like to add this select under the controller driver,= =20 > as some of the vendors did, is because, in the future, we might have=20 > different PDM microphones that might not work with SND_SOC_DMIC and=20 > might need a different driver. > I don't have a strong opinion on this. If you think I am overthinking,=20 > please let me know and I will change this. It's unlikely but possible that there could be some other device connected (eg, you could have a DSP or something that generates PDM output). If the driver doesn't directly instantiate the DMIC itself then it's probably reasonable for it to be user controllable if the DMIC driver is there. --V6OcCftqYTAkRMmU Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmJz5scACgkQJNaLcl1U h9BoAQf/RxATTm9oBfzFYiQDmMZnY6nSvVtROk9XiCIp4BRvt4ezstNodFEGlLKQ fWhTWfMmr7rGF9Syj1Zb6W/2Fahc/R04bml6dXcDHBKeW5a9Tnslg4U0brWHd2oT SeE9ZVLQzoFgUcCh6DrwxpYIkfbPrZwrBS0YNHHgjw3LAc2voYJiNm8njcOXop9i +kpz1RZs60mULxSPGoNOr/9HNfNc2H3lZcSG6ZkdjkiXMJ4TjedzB5YpaOFGrYLN hCy9jHw/mzCFq9EnV8ubaTI05RFwIusQd0doLtRxlgrrS94+mwNqHggZLnFK95Z/ BnnQHFiY+49FUwRU4NPZhfcFayakzQ== =TVpY -----END PGP SIGNATURE----- --V6OcCftqYTAkRMmU-- 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 3D03CC433F5 for ; Thu, 5 May 2022 15:03:00 +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-Type:Cc: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=NxYsgaoAbD11+FcJOwRiM1+bppzJALPgIzXXM5SzexA=; b=wJJoQAVbgPO0od8y2GPGMfDoeL EnXS+YtqiDniCqRGya34ZTTb4EVis/Qgk51hG2R7RcgSxzsXqeeD73S/a4H1bbXVuiOJ5VhchyK1M AvURCVptudSlWTn4gaK9h6HM9ROuYQ6d2T4vAGltuzqUzDAI71MrA40tkv17OHAlQGOQtAE4mGXjY Dg7nurYmCmGzkwxJ/yJK1AZa56Qazkazi4MGlzmlXbBmby+1dhz90u0q/FakmL0taDizHWrjiDBht cvsR0oWDK6yDUGCv6tzqXIYgTICxkJsBqfKLjPIhyVXJBH3iQZq9XsIW6MuXVTAidvSmvSZybApet EvMAwgCA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nmcz6-00GWcT-Op; Thu, 05 May 2022 15:01:41 +0000 Received: from ams.source.kernel.org ([2604:1380:4601:e00::1]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nmcz3-00GWaM-FG for linux-arm-kernel@lists.infradead.org; Thu, 05 May 2022 15:01:39 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ams.source.kernel.org (Postfix) with ESMTPS id A1783B82D82; Thu, 5 May 2022 15:01:35 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 14047C385A4; Thu, 5 May 2022 15:01:31 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1651762894; bh=UUh4IAYvnUeRl0+FGz9aUqMk1V1C/5Mv1s6gce1m4PY=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=S8Y4NKxoe5tibidDLQ1YsIqidmaB/VrEldK+Y9cbMH5jS/+MM5rgrPY10yHnZuKpA Vx9PHYyx0ctjGVYN0AW2U+dj9sQF9nYeW19rmlBAaro91ZcvrnCEvQGSa4AOhWe4F+ AORKwgkynESpVtUsLkKVDuj91k0fG9nqnFkW+X74VADC85jI/SoExoUCj45dBUkPpw Xi936/6T6rBsBBkB/rBLDpE8NcvgzDUFSfZGqCKN4nIruGHpACikgCGzflWOu6DwGI kpfzOf2U/DAZwBv3TrR1gbINOSQxTB6NUvESyu3p4jsN3Y180ej0joxdrI7Y7frWDd SCwA7xo4UFLIw== Date: Thu, 5 May 2022 16:01:28 +0100 From: Mark Brown To: Codrin.Ciubotariu@microchip.com Subject: Re: [PATCH v3 6/6] ARM: configs: at91: sama7_defconfig: add MCHP PDMC and DMIC drivers Message-ID: References: <20220307122202.2251639-1-codrin.ciubotariu@microchip.com> <20220307122202.2251639-7-codrin.ciubotariu@microchip.com> <77c2b348-b7a6-458d-21b6-68c54efc317f@microchip.com> MIME-Version: 1.0 In-Reply-To: <77c2b348-b7a6-458d-21b6-68c54efc317f@microchip.com> X-Cookie: Real programs don't eat cache. X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220505_080137_870983_9E499D09 X-CRM114-Status: GOOD ( 23.66 ) 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: , Cc: devicetree@vger.kernel.org, alsa-devel@alsa-project.org, lars@metafoo.de, tiwai@suse.com, linux-kernel@vger.kernel.org, robh+dt@kernel.org, perex@perex.cz, linux-arm-kernel@lists.infradead.org Content-Type: multipart/mixed; boundary="===============7924770931864579691==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============7924770931864579691== Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="V6OcCftqYTAkRMmU" Content-Disposition: inline --V6OcCftqYTAkRMmU Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, May 05, 2022 at 02:47:04PM +0000, Codrin.Ciubotariu@microchip.com w= rote: > On 05.05.2022 16:58, Nicolas Ferre wrote: > > I'm fine with that, but I see that some Kconfig entries "select" this= =20 > > SND_SOC_DMIC directly (amd, intel, mediatek, stm). > > If it's absolutely needed for PDMC to work, what about doing the same a= s=20 > > it would prevent some broken configurations? > The only way it makes sense to me to have this driver selected somewhere= =20 > is in a sound card driver, used for a specific board, which we know it=20 > has PDM microphones. Since, for now, we use the simple sound card for=20 > our audio interfaces, we have no place to add this select. > The reason I do not like to add this select under the controller driver,= =20 > as some of the vendors did, is because, in the future, we might have=20 > different PDM microphones that might not work with SND_SOC_DMIC and=20 > might need a different driver. > I don't have a strong opinion on this. If you think I am overthinking,=20 > please let me know and I will change this. It's unlikely but possible that there could be some other device connected (eg, you could have a DSP or something that generates PDM output). If the driver doesn't directly instantiate the DMIC itself then it's probably reasonable for it to be user controllable if the DMIC driver is there. --V6OcCftqYTAkRMmU Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmJz5scACgkQJNaLcl1U h9BoAQf/RxATTm9oBfzFYiQDmMZnY6nSvVtROk9XiCIp4BRvt4ezstNodFEGlLKQ fWhTWfMmr7rGF9Syj1Zb6W/2Fahc/R04bml6dXcDHBKeW5a9Tnslg4U0brWHd2oT SeE9ZVLQzoFgUcCh6DrwxpYIkfbPrZwrBS0YNHHgjw3LAc2voYJiNm8njcOXop9i +kpz1RZs60mULxSPGoNOr/9HNfNc2H3lZcSG6ZkdjkiXMJ4TjedzB5YpaOFGrYLN hCy9jHw/mzCFq9EnV8ubaTI05RFwIusQd0doLtRxlgrrS94+mwNqHggZLnFK95Z/ BnnQHFiY+49FUwRU4NPZhfcFayakzQ== =TVpY -----END PGP SIGNATURE----- --V6OcCftqYTAkRMmU-- --===============7924770931864579691== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============7924770931864579691==--