Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ucm2: sof-soundwire: Correct FixedBootSequence for dmic info #485

Conversation

ujfalusi
Copy link
Contributor

Replace the curly brackets with square brackets.
The curly ones break the FixedBootSequence parsing by resetting the sequence.
If the /var/lib/alsa/card0.conf.d is not present then UCM will fail because all sequences before the dmic is lost, including the card-init:

alsactl init

alsa-lib main.c:619:(execute_cfgsave) unable to open file '/var/lib/alsa/card0.conf.d/42-sof-hdmi.conf': No such file or directory alsa-lib main.c:2456:(set_fixedboot_user) Unable to execute force boot sequence

The full sequence supposed to be:
card-init
ctl-remap
cs42l43
dmic
hdmi

Fixes: 9fa70ad ("sof-hda-dsp,sof-soundwire: add CaptureMicInfoFile fields for dmics")

Replace the curly brackets with square brackets.
The curly ones break the FixedBootSequence parsing by resetting the
sequence.
If the /var/lib/alsa/card0.conf.d is not present then UCM will fail because
all sequences before the dmic is lost, including the card-init:

# alsactl init
alsa-lib main.c:619:(execute_cfgsave) unable to open file '/var/lib/alsa/card0.conf.d/42-sof-hdmi.conf': No such file or directory
alsa-lib main.c:2456:(set_fixedboot_user) Unable to execute force boot sequence

The full sequence supposed to be:
card-init
ctl-remap
cs42l43
dmic
hdmi

Fixes: 9fa70ad ("sof-hda-dsp,sof-soundwire: add CaptureMicInfoFile fields for dmics")
Signed-off-by: Peter Ujfalusi <[email protected]>
@ujfalusi
Copy link
Contributor Author

@perexg, on fresh installs with alsa-ucm-conf 1.2.13 will fail to initialize UCM as the /var/lib/alsa/card0.conf.d is not present. On updated systems the FixedBootSequence will work, but somehow oddly:

alsactl init

alsa-lib parser.c:1237:(parse_sequence) error: sequence command 'echo dmic >> /tmp/ucm.debug' is ignored

cat /tmp/ucm.debug
hdmi
ctl-remap
cs42l43

The expected sequence should be:
card-init
ctl-remap
cs42l43
dmic
hdmi

@perexg perexg closed this in 0032013 Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant