Re: [PATCH v3 6/6] ARM: configs: at91: sama7_defconfig: add MCHP PDMC and DMIC drivers

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 05/05/2022 at 17:01, Mark Brown wrote:
On Thu, May 05, 2022 at 02:47:04PM +0000,Codrin.Ciubotariu@xxxxxxxxxxxxx  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



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux