Hmm. What do you think of the attached patch - would it work as well? It removes the part that returns early for all three, letting it fall through to the part that returns "Front" etc.
+++ b/sound/pci/hda/hda_generic.c @@ -1100,11 +1100,9 @@ static const char *get_line_out_pfx(struct hda_codec *codec, int ch, if (!ch && cfg->speaker_outs && cfg->hp_outs) { bool hp_lo_shared = !path_has_mixer(codec,
spec->hp_paths[0], ctl_type);
bool spk_lo_shared = !path_has_mixer(codec,
spec->speaker_paths[0], ctl_type);
if (hp_lo_shared && spk_lo_shared)return spec->vmaster_mute.hook ? "PCM"
: "Master";
if (hp_lo_shared)
if (hp_lo_shared && !spk_lo_shared) return "Headphone+LO";
if (spk_lo_shared)
if (spk_lo_shared && !hp_lo_shared) return "Speaker+LO"; } }After a quick check, this seems better. Master is a real master now and
there
is a Front control.
With the patch I'm getting a real Master (for all
Front/Surround/CLFE) and a
separate Front Volume Control in addition to Surround, Center and LFE
which >> is exactly how it should be.
Hmm. What do you think of the attached patch - would it work as well?
It removes the part that returns early for all three, letting it fall through to the part that returns "Front" etc.
http://cgit.freedesktop.org/pulseaudio/pulseaudio/commit/src/modules/alsa/mi... 96369919e5100865e2469e42fb8f4b8e38e41aef
Do you mean front playback volume is set to maximum when headphone is plugged?
How do Pulseaudio calculate dB min of headphone path and lineout path?
Do it just put dB min of front playback volume in headphone path only when there is no headphone playback volume?
BTW codec alc888 in buglink can use amp out in node 0x26 as Headphone Playback volume
However, due to the speaker having no path with my forced config, I cannot say whether this will work if the parser could find the multi-io config by itself (and until then there probably shouldn't be patches I suggested).
And,
by the way, as a result of the no-path I'm getting a Speaker+LO control.
Here a little additional debug output during the get_line_out_pfx() calls. The "path_has_mixer" output refers to a call which output is in the
following
line, i. e. the call for getting spk_lo_shared:
[ 0.648407] sound hdaudioC1D0: get_line_out_pfx, AUTO_PIN_LINE_OUT:
hp_lo_shared=1
[ 0.648407] sound hdaudioC1D0: path_has_mixer: no path! [ 0.648408] sound hdaudioC1D0: get_line_out_pfx, AUTO_PIN_LINE_OUT:
spk_lo_shared=1
[ 0.648409] sound hdaudioC1D0: get_line_out_pfx, AUTO_PIN_LINE_OUT:
hp_lo_shared=0
[ 0.648410] sound hdaudioC1D0: path_has_mixer: no path! [ 0.648410] sound hdaudioC1D0: get_line_out_pfx, AUTO_PIN_LINE_OUT:
spk_lo_shared=1
Ingo