[PATCH] ASoC: Intel: Skylake: Add alternative topology binary name
Pierre-Louis Bossart
pierre-louis.bossart at linux.intel.com
Thu Mar 26 16:26:08 CET 2020
On 3/26/20 10:00 AM, Gorski, Mateusz wrote:
>
>> That's progress but is this complete?
>>
>> skl->i2s_dev->name is the name of the machine driver, I don't see the
>> part where this is modified to deal with the number of dmics?
>>
>> In your topology patches, the names are: hda_dsp_noDMIC
>> hda_dsp_DMIC_2ch hda_dsp_DMIC_4ch
>>
>> How would the relevant file be found based on the number of DMICs on
>> the platform? I must be missing something here?
>>
>
> The intention of this patch was not to deal with the different DMIC
> configurations problem. It only simplifies the topology binary
> selection. As you mentioned in one of previous mails, currently there
> are two options:
>
> - name based on NHLT, which is pretty complicated, especially for end
> user, and is also depending on things like OEM name so the same topology
> will need to be renamed multiple times different devices (additionally,
> there are laptops on the market that do not have NHLT table at all)
>
> - dfw_sst.bin, which is only a fallback binary name and should not be
> actually used because it could be misleading for users/distro intergrators
>
> This change adds the third option, which is, in my opinion, the right
> way to deal with this problem. This name is simpler, does not depend on
> existence of NHLT, and makes life easier for users/distros.
Right, and that's fine to avoid the NHLT-name and dfw_sst.bin, no issue
here. The point is to go one step further and require ZERO configuration
from users.
> And as for the mentioned DMIC confguration problem - I am doing a
> research to find the simplest way to deal with this.
Just append the number of mics detected to the topology file name?
Asking users to copy/symlink hda_dsp_DMIC_2ch.tplg as hda-dsp.tplg
doesn't really help, you can make things simpler.
More information about the Alsa-devel
mailing list