Re: [PATCH alsa-lib 0/5] Add generic exception mechanism for non-standard control-names

[Hans de Goede <hdegoede@redhat.com>]

Hi Jaroslav,

On 5/4/21 10:53 AM, Jaroslav Kysela wrote:
> Dne 03. 05. 21 v 22:52 Hans de Goede napsal(a):
>> Hi All,
>>
>> This series seems to have fallen through the cracks,
>> so here is a resend of it.
>>
>> Regards,
> 
> Thank you, Hans. The problem with this implementation is that it's really card
> specific. Also, ASoC codec drivers have usually ID names based on registers so
> the mapping for the user is problematic anyway (the functionality is different
> from the name or not related to the name). I'm actually evaluating another
> solution which is more flexible:
> 
> 1) add control remap plugin to allow the control ID remapping in the
> alsa-lib's control API, so we can mangle those identifiers there (already
> implemented)
> 
> 2) add local and global alsa-lib configurations per UCM card specified in the
> UCM configuration files; the configurations may be for both control and PCM
> devices (restrict or set specific parameters)

Ok, thank you for working on this.

> I will notify you when I finish my tests.

Yes, please let me know when you've something ready to test, then I'll take
a look at adding the necessary bits for the bycr-rt5640 and cht-bsw-rt567
UCM profiles, as some control renaming is necessary to make sure that
the hw-volume control on these devices also correctly controls the
hw mute controls (which in turn are necessary for both full muting and
for mute LED control).

Regards,

Hans


>> Hans de Goede (5):
>>   mixer: simple - Add generic exception mechanism for non-standard
>>     control-names
>>   mixer: simple - Move handling of 3D Control - Depth controls to the
>>     exceptions list
>>   mixer: simple - Add exceptions for non " Volume" suffixed capture
>>     vol-ctls used in ASoC realtek codec drivers
>>   mixer: simple - Add exceptions for some capture-vol-ctls which have a
>>     " Volume" suffix
>>   mixer: simple - Add exceptions for some Playback Switches with a "
>>     Channel Switch" suffix
>>
>>  src/mixer/simple_none.c | 74 +++++++++++++++++++++++++----------------
>>  1 file changed, 46 insertions(+), 28 deletions(-)
>>
> 
>