Re: [PATCH 1/2] ALSA: hda/realtek: Add generic quirk for Bass speaker DAC avoidance

[Takashi Iwai]

On Mon, 02 Feb 2026 00:12:14 +0100,
Samuel Dionne-Riel wrote:
> 
> This change is for the follow-up commit, where the match on the specific
> device (Minisforum V3 SE) is added.
> 
> This pattern (re-routing the bass speakers off of a DAC without volume
> control) seems common enough that having a "model" to match against and
> quickly use to verify may be worthwhile.
> 
> The alc285_fixup_thinkpad_x1_gen7 routing was selected, amongst the
> different options, as it should allow tuning the ratio between both
> speaker set.
> 
> The routing was verified using `hda-verb`, and picking either 0x00 or
> 0x01. Either routin made the volume of the bass speakers controllable.
> 
>     hda-verb /dev/snd/hwC1D0 0x17 SET_CONNECT_SEL 0x01
> 
> Signed-off-by: Samuel Dionne-Riel <samuel@xxxxxxxxxxxxxxx>
> 
> ---
> 
> It may be worthwhile to clean-up behind and make the other fixups
> re-routing `0x17` from `0x06` to either `0x02` or `0x03` also use the
> same routing. It's a bit confusing, determining which routing to choose
> when adding one for a device.
> 
>  - alc289_fixup_asus_ga401
>  - alc295_fixup_disable_dac3
> 
> Might need to be checked (and re-used within a chain):
> 
>  - alc287_fixup_yoga9_14iap7_bass_spk_pin
>  - alc245_fixup_hp_spectre_x360_eu0xxx
>  - alc245_fixup_hp_spectre_x360_16_aa0xxx
> 
> I might be missing some, if for example they are handled through pin
> fixups, or other fixup mechanisms.
> 
> I don't have the experience needed to know for sure if the cleanup would
> be appropriate. So please excuse the lack of follow-up on that.
> 
> Signed-off-by: Samuel Dionne-Riel <samuel@xxxxxxxxxxxxxxx>

Indeed it looks like that we can cleanup a lot regarding this DAC
routing fix-ups.  Though, *_thinkpad_x1_gen7() has more workarounds
about the control element renames -- which needs to be verified
whether the very same workaround is applicable to other cases.

IIUC, this kcontrol names (DAC1/DAC2) match with your device
configuration?  Could you give alsa-info.sh output?

In anyway, I don't think it's needed to split to two individual
commits.  You can fold into one patch.


thanks,

Takashi