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 C7FB3C433F5 for ; Thu, 5 May 2022 15:07:36 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1381092AbiEEPLO (ORCPT ); Thu, 5 May 2022 11:11:14 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34034 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232149AbiEEPLI (ORCPT ); Thu, 5 May 2022 11:11:08 -0400 Received: from esa.microchip.iphmx.com (esa.microchip.iphmx.com [68.232.154.123]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 279F638BDA; Thu, 5 May 2022 08:07:29 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1651763249; x=1683299249; h=message-id:date:mime-version:subject:to:cc:references: from:in-reply-to:content-transfer-encoding; bh=ofOX12TGojhwMN/6KVsscVM33pqSCNg+r9yu6HGPdus=; b=cGLpXgNbtVFx4D+CRdVFagqWeSrVVAncL5Ipx/wq/bQEbvxaINDVv3PL reOProTAjqEUoy9xpNsVDs09UIyfDcC/M8I5/UtashDpe6uM1F3ORwqtj gLeJhiX9taAB15DYOpIrZkZvgFT65T4+fD+Gv6BGp6hjpME1PEu2aTL1w D9CsZMOmLNGpsqN/Vq9593xiQQaQfBYWbWe4d0479eDaASSbRQUxFj3E0 PzQusMaUhCXrnMTx0xCybhc7Hngikd6Fodnt8hKWY61RN1p0+LYJbLJ28 gb4gLHgkH6bEwF2FEqs85RGCbBQW18Hh6l077fyL7WXGg6JAAo13jHyQG Q==; X-IronPort-AV: E=Sophos;i="5.91,201,1647327600"; d="scan'208";a="157932577" Received: from smtpout.microchip.com (HELO email.microchip.com) ([198.175.253.82]) by esa2.microchip.iphmx.com with ESMTP/TLS/AES256-SHA256; 05 May 2022 08:07:28 -0700 Received: from chn-vm-ex03.mchp-main.com (10.10.85.151) by chn-vm-ex02.mchp-main.com (10.10.85.144) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.17; Thu, 5 May 2022 08:07:27 -0700 Received: from [10.159.245.112] (10.10.115.15) by chn-vm-ex03.mchp-main.com (10.10.85.151) with Microsoft SMTP Server id 15.1.2375.17 via Frontend Transport; Thu, 5 May 2022 08:07:25 -0700 Message-ID: Date: Thu, 5 May 2022 17:07:18 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.8.1 Subject: Re: [PATCH v3 6/6] ARM: configs: at91: sama7_defconfig: add MCHP PDMC and DMIC drivers Content-Language: en-US To: Mark Brown , CC: , , , , , , , References: <20220307122202.2251639-1-codrin.ciubotariu@microchip.com> <20220307122202.2251639-7-codrin.ciubotariu@microchip.com> <77c2b348-b7a6-458d-21b6-68c54efc317f@microchip.com> From: Nicolas Ferre Organization: microchip In-Reply-To: Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 05/05/2022 at 17:01, Mark Brown wrote: > On Thu, May 05, 2022 at 02:47:04PM +0000,Codrin.Ciubotariu@microchip.com wrote: >> On 05.05.2022 16:58, Nicolas Ferre wrote: >>> I'm fine with that, but I see that some Kconfig entries "select" this >>> SND_SOC_DMIC directly (amd, intel, mediatek, stm). >>> If it's absolutely needed for PDMC to work, what about doing the same as >>> it would prevent some broken configurations? >> The only way it makes sense to me to have this driver selected somewhere >> is in a sound card driver, used for a specific board, which we know it >> has PDM microphones. Since, for now, we use the simple sound card for >> 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, >> as some of the vendors did, is because, in the future, we might have >> different PDM microphones that might not work with SND_SOC_DMIC and >> might need a different driver. >> I don't have a strong opinion on this. If you think I am overthinking, >> 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. Fair enough, It makes perfect sense to me as it is then. Thanks for the feedback! Best regards, Nicolas -- Nicolas Ferre 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 86B09C433F5 for ; Thu, 5 May 2022 15:08:32 +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 5C67917AA; Thu, 5 May 2022 17:07:40 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa0.perex.cz 5C67917AA DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=alsa-project.org; s=default; t=1651763310; bh=ofOX12TGojhwMN/6KVsscVM33pqSCNg+r9yu6HGPdus=; h=Date:Subject:To:References:From:In-Reply-To:Cc:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=bhNhxWIjfFCGxxYEuLx5/dSV/o/4fFkkimtiATcWAM4WodMg52Co444z4vVgv/AE0 24MUvWhzhHkCzkIvFwxGTrEHSwRJoQ57Oex4DcYsKHEQA5JgE5LRkq/kmluCa/0XF+ kqgNkj/EaI2W35lT7mDtWtxgGVCTKlUpDSLaATRI= Received: from alsa1.perex.cz (localhost.localdomain [127.0.0.1]) by alsa1.perex.cz (Postfix) with ESMTP id F28B1F8016A; Thu, 5 May 2022 17:07:39 +0200 (CEST) Received: by alsa1.perex.cz (Postfix, from userid 50401) id 33392F8016B; Thu, 5 May 2022 17:07:38 +0200 (CEST) Received: from esa.microchip.iphmx.com (esa.microchip.iphmx.com [68.232.154.123]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by alsa1.perex.cz (Postfix) with ESMTPS id 88FE1F800D3 for ; Thu, 5 May 2022 17:07:31 +0200 (CEST) DKIM-Filter: OpenDKIM Filter v2.11.0 alsa1.perex.cz 88FE1F800D3 Authentication-Results: alsa1.perex.cz; dkim=pass (2048-bit key) header.d=microchip.com header.i=@microchip.com header.b="E6n5+P2T" DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1651763253; x=1683299253; h=message-id:date:mime-version:subject:to:cc:references: from:in-reply-to:content-transfer-encoding; bh=ofOX12TGojhwMN/6KVsscVM33pqSCNg+r9yu6HGPdus=; b=E6n5+P2TYgD/FlPpm8pXqrSCt1S2DeaoFwJRZjEislZdMMYcjVM6kfQO E1fFsPgMb6RDYU7K/S3fLF8Su4rvUWmMjUccbss9u+Ii/2BsxZdtIFYFW TSgPSX5PqDLgFI7OwW/oLWYnCZAu7cbczHbsrXjty60PLmUe4vTYXNqlJ J0Ho36SAPK/fMd+d69dxzqBvyYNonskEy8w4j/sXU3/CHYjZIKxhuQZ+h x7207+tWeLZ1hpHnMNd2oiDKRwDyvuKPFNoU6D+LFxPpPgnBP5ULp9x3O 1oLWnfKqckjmbQz3loa9x5zVGHr9G5PSH+4OgJTFnxKnveQHQDvnc+BI+ g==; X-IronPort-AV: E=Sophos;i="5.91,201,1647327600"; d="scan'208";a="157932577" Received: from smtpout.microchip.com (HELO email.microchip.com) ([198.175.253.82]) by esa2.microchip.iphmx.com with ESMTP/TLS/AES256-SHA256; 05 May 2022 08:07:28 -0700 Received: from chn-vm-ex03.mchp-main.com (10.10.85.151) by chn-vm-ex02.mchp-main.com (10.10.85.144) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.17; Thu, 5 May 2022 08:07:27 -0700 Received: from [10.159.245.112] (10.10.115.15) by chn-vm-ex03.mchp-main.com (10.10.85.151) with Microsoft SMTP Server id 15.1.2375.17 via Frontend Transport; Thu, 5 May 2022 08:07:25 -0700 Message-ID: Date: Thu, 5 May 2022 17:07:18 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.8.1 Subject: Re: [PATCH v3 6/6] ARM: configs: at91: sama7_defconfig: add MCHP PDMC and DMIC drivers Content-Language: en-US To: Mark Brown , References: <20220307122202.2251639-1-codrin.ciubotariu@microchip.com> <20220307122202.2251639-7-codrin.ciubotariu@microchip.com> <77c2b348-b7a6-458d-21b6-68c54efc317f@microchip.com> From: Nicolas Ferre Organization: microchip In-Reply-To: Content-Type: text/plain; charset="UTF-8"; format=flowed Content-Transfer-Encoding: 7bit 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, 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" On 05/05/2022 at 17:01, Mark Brown wrote: > On Thu, May 05, 2022 at 02:47:04PM +0000,Codrin.Ciubotariu@microchip.com wrote: >> On 05.05.2022 16:58, Nicolas Ferre wrote: >>> I'm fine with that, but I see that some Kconfig entries "select" this >>> SND_SOC_DMIC directly (amd, intel, mediatek, stm). >>> If it's absolutely needed for PDMC to work, what about doing the same as >>> it would prevent some broken configurations? >> The only way it makes sense to me to have this driver selected somewhere >> is in a sound card driver, used for a specific board, which we know it >> has PDM microphones. Since, for now, we use the simple sound card for >> 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, >> as some of the vendors did, is because, in the future, we might have >> different PDM microphones that might not work with SND_SOC_DMIC and >> might need a different driver. >> I don't have a strong opinion on this. If you think I am overthinking, >> 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. Fair enough, It makes perfect sense to me as it is then. Thanks for the feedback! Best regards, Nicolas -- Nicolas Ferre 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 330D3C433EF for ; Thu, 5 May 2022 15:08: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-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:From:References:CC:To:Subject: MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=B5/yxFjbnqaNv6B7tvmKnCiDqUUM+ngewVneBUe12zQ=; b=wvpwpYHIor3Iss +x80eKLQgcj5BgzFmHTRPVu7JsD6YYvBksImTBdod8TOGAdeqkPKEuJwHiMfANnkaSeby7FH4ZplW x1Y+tSUz4phdn+k2Gja0WemONxN5V41nlANfE5HhOAyMHlTiDBIyEqp6zhdYNuPCclMdnVQFxaTRe v7gNDOYdy7PCfQ+KDF2Kmwpq484u0BYXe3+FztnlmWnLuG9eRYi19qHFCzSqJezOt9PlC409wiU9n RJEHy8duIju5eXt5XtmJ0l72HaM9o7qW/IFwQ7caMP+kc/ennyxwbwzv/QZSs8BUCUVwqhaJQR68W mbJaO6vn+E6475iXaXnw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nmd4m-00GZ6Z-Se; Thu, 05 May 2022 15:07:33 +0000 Received: from esa.microchip.iphmx.com ([68.232.154.123]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nmd4j-00GZ5B-HV for linux-arm-kernel@lists.infradead.org; Thu, 05 May 2022 15:07:30 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=microchip.com; i=@microchip.com; q=dns/txt; s=mchp; t=1651763249; x=1683299249; h=message-id:date:mime-version:subject:to:cc:references: from:in-reply-to:content-transfer-encoding; bh=ofOX12TGojhwMN/6KVsscVM33pqSCNg+r9yu6HGPdus=; b=cGLpXgNbtVFx4D+CRdVFagqWeSrVVAncL5Ipx/wq/bQEbvxaINDVv3PL reOProTAjqEUoy9xpNsVDs09UIyfDcC/M8I5/UtashDpe6uM1F3ORwqtj gLeJhiX9taAB15DYOpIrZkZvgFT65T4+fD+Gv6BGp6hjpME1PEu2aTL1w D9CsZMOmLNGpsqN/Vq9593xiQQaQfBYWbWe4d0479eDaASSbRQUxFj3E0 PzQusMaUhCXrnMTx0xCybhc7Hngikd6Fodnt8hKWY61RN1p0+LYJbLJ28 gb4gLHgkH6bEwF2FEqs85RGCbBQW18Hh6l077fyL7WXGg6JAAo13jHyQG Q==; X-IronPort-AV: E=Sophos;i="5.91,201,1647327600"; d="scan'208";a="157932577" Received: from smtpout.microchip.com (HELO email.microchip.com) ([198.175.253.82]) by esa2.microchip.iphmx.com with ESMTP/TLS/AES256-SHA256; 05 May 2022 08:07:28 -0700 Received: from chn-vm-ex03.mchp-main.com (10.10.85.151) by chn-vm-ex02.mchp-main.com (10.10.85.144) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.17; Thu, 5 May 2022 08:07:27 -0700 Received: from [10.159.245.112] (10.10.115.15) by chn-vm-ex03.mchp-main.com (10.10.85.151) with Microsoft SMTP Server id 15.1.2375.17 via Frontend Transport; Thu, 5 May 2022 08:07:25 -0700 Message-ID: Date: Thu, 5 May 2022 17:07:18 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.8.1 Subject: Re: [PATCH v3 6/6] ARM: configs: at91: sama7_defconfig: add MCHP PDMC and DMIC drivers Content-Language: en-US To: Mark Brown , CC: , , , , , , , References: <20220307122202.2251639-1-codrin.ciubotariu@microchip.com> <20220307122202.2251639-7-codrin.ciubotariu@microchip.com> <77c2b348-b7a6-458d-21b6-68c54efc317f@microchip.com> From: Nicolas Ferre Organization: microchip In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220505_080729_695503_C18DFFAD X-CRM114-Status: GOOD ( 18.80 ) 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-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 05/05/2022 at 17:01, Mark Brown wrote: > On Thu, May 05, 2022 at 02:47:04PM +0000,Codrin.Ciubotariu@microchip.com wrote: >> On 05.05.2022 16:58, Nicolas Ferre wrote: >>> I'm fine with that, but I see that some Kconfig entries "select" this >>> SND_SOC_DMIC directly (amd, intel, mediatek, stm). >>> If it's absolutely needed for PDMC to work, what about doing the same as >>> it would prevent some broken configurations? >> The only way it makes sense to me to have this driver selected somewhere >> is in a sound card driver, used for a specific board, which we know it >> has PDM microphones. Since, for now, we use the simple sound card for >> 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, >> as some of the vendors did, is because, in the future, we might have >> different PDM microphones that might not work with SND_SOC_DMIC and >> might need a different driver. >> I don't have a strong opinion on this. If you think I am overthinking, >> 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. Fair enough, It makes perfect sense to me as it is then. Thanks for the feedback! Best regards, Nicolas -- Nicolas Ferre _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel