Skip to content

DRAFT [for alsa-lib 1.2.15]: HDA: Handle better Bass speaker including 4 channel mode #533

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

perexg
Copy link
Member

@perexg perexg commented Mar 25, 2025

It's a follow up of #410 .

@perexg perexg force-pushed the topic/pr410-bass-speakers branch from 7d62b84 to c6c7e61 Compare March 25, 2025 15:31
@perexg perexg force-pushed the topic/pr410-bass-speakers branch from c6c7e61 to 75af816 Compare March 25, 2025 15:43
@perexg perexg changed the title HDA: Handle better Bass speaker including 4 channel mode DRAFT: HDA: Handle better Bass speaker including 4 channel mode Mar 25, 2025
@perexg perexg force-pushed the topic/pr410-bass-speakers branch from 75af816 to 12848ce Compare March 26, 2025 14:06
Create 'Speaker 4ch' virtual mixer controls for this purpose.

'Syntax 8' is required (alsa-lib 1.2.14).

Link: alsa-project#410
Signed-off-by: Jaroslav Kysela <perex@perex.cz>
@perexg perexg force-pushed the topic/pr410-bass-speakers branch from 12848ce to 29f140f Compare March 26, 2025 14:11
@perexg perexg changed the title DRAFT: HDA: Handle better Bass speaker including 4 channel mode DRAFT [for alsa-lib 1.2.15]: HDA: Handle better Bass speaker including 4 channel mode Apr 25, 2025
Copy link

@tuxedoxt tuxedoxt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

First of all thanks for the initiative.

I'm having some issues testing the functionality though. I've taken some time to look at it again now.

Here are my observations so far

  • Using a distribution that already comes with alsa v1.2.14 I didn't manage to load the config due to the syntax 8 limitation (error also seen on alsaucm reload)
  • Manually building/installing alsa-lib (current master branch ee5a58f48e141aeb28df182247c3f789f4160d02) makes alsaucm reload "work" (with error only on dump, see typo)
  • Fixing the typo also produces a dump but I do not get a "Play HiFi quality Music" config in system audio config dialog. I get the "Analog Stereo Duplex" legacy non-UCM config.

alsaucm-dump.txt

If.bass-vol {
Condition {
Type String
Heystack "${var:HasBassSpeaker}"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here's a typo Heystack => Haystack

It did not give me an error on alsaucm reload but did on alsaucm -c 2 dump text

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.

2 participants