Re: [PATCH] ALSA: hda: intel-nhlt: verify config type

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

 



On Tue, 02 Mar 2021 01:01:46 +0100,
Pierre-Louis Bossart wrote:
> 
> Multiple bug reports report issues with the SOF and SST drivers when
> dealing with single microphone cases.
> 
> We currently read the DMIC array information unconditionally but we
> don't check that the configuration type is actually a mic array.
> 
> When the DMIC link does not rely on a mic array configuration, the
> recommendation is to check the format information to infer the maximum
> number of channels, and map this to the number of microphones.
> 
> This leaves a potential for a mismatch between actual microphones
> available in hardware and what the ACPI table contains, but we have no
> other source of information.
> 
> Note that single microphone configurations can alternatively be
> handled with a 'mic array' configuration along with a 'vendor-defined'
> geometry.
> 
> Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=201251
> BugLink: https://github.com/thesofproject/linux/issues/2725
> Fixes: 7a33ea70e1868 ('ALSA: hda: intel-nhlt: handle NHLT VENDOR_DEFINED DMIC geometry')
> Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@xxxxxxxxxxxxxxx>
> Reviewed-by: Guennadi Liakhovetski <guennadi.liakhovetski@xxxxxxxxx>
> Reviewed-by: Rander Wang <rander.wang@xxxxxxxxx>
> Reviewed-by: Kai Vehmanen <kai.vehmanen@xxxxxxxxxxxxxxx>

Thanks, applied.


Takashi



[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