Hi,
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:
- 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)
- 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)
p.s.
Note that the first patch in this series also fixes a regression, quoting from the commit message:
This also fixes the "Capture Volume" and "Capture Switch" exceptions no longer working after commit 86b9c67774bc ("mixer: simple - Unify simple_none: base_len() exception handling") because they were moved to after the suffix checking, so they would be treated as CTL_GLOBAL_VOLUME resp. CTL_GLOBAL_SWITCH based on their suffix before the exception check has a chance to check for a match.
In the first patch of this series fixing this regression is a side-effect of the changes made there.
Since you don't want to take this series, I'll write a new patch fixing just the regression.
Regards,
Hans