Mario Limonciello wrote:
Hi Takashi:
Takashi Iwai wrote:
If "Surround" doesn't do anything, why should it be there? And, does "Headphone" change both headphones even with 4 or 6 ch outputs properly? Doesn't "Headhpone" change any other output volumes?
I found what the surround mixer is being used for. If "Mic as Output" is turned on, "Surround" acts as the mixer to control that jack.
Correct.
The the reason the headphone mixer isn't created without my patch is the "mic as output" and "line in as output" switches.
The Headphone mixer changes both of the regular headphone jack's volumes. Changing the "Headphone" volume doesn't affect the speaker outputs at all, only the two headphone jacks.
The patch introduces another exception, and now nobody would understand how the channels and outputs and mixers are assigned together.
Two points:
- I'm not sure this is as much of a problem as you raise. I don't
believe many people even know how the channels and mixers are assigned in the first place. I see plenty of support requests for "why doesn't this second headphone jack work with headphones". I've never seen one of "I can't hook up my 7.1 surround system to my consumer grade laptop and have all of my speakers work". These machines come with DisplayPort and HDMI ports which people will generally be using for hooking up to their television sets. I seem to think it's not very common to use the analogue jacks to independently hook up pairs of speakers assigned to say Front or Surround. 2) You are still able to hook up a separate set of speakers to the mic output jack and since those are assigned surround have things work as expected.
Hm, I thought I did. Which patch exactly?
The patch title was [PATCH] hda: Add STA_DELL_S14 quirk. Here's the starting part of it:
Add STAC_DELL_S14 quirk for new laptop series. Removed un-needed pins in pin_nids for stac92hd83xxx. Also reorganized connection selection code for the respective ports per quirk define.
sound/pci/hda/patch_sigmatel.c imap://mario_limonciello@outlook.dell.com:143/fetch%3EUID%3E/INBOX%3E13699#sound/pci/hda/patch_sigmatel.c | 52 +++++++++++++++++++++++++++-------------
1 files changed, 35 insertions(+), 17 deletions(-)
You responded to the mail saying it was applied, but I seem to not be able to find it in current ALSA git.
Regards