Re: [PATCH] hda: Add logic to force HP mixer creation

[Mario Limonciello <mario_limonciello@dell.com>]

[-- Attachment #1.1: Type: text/plain, Size: 2348 bytes --]

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.

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:
1) 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
-- 
Mario Limonciello
*Dell | Linux Engineering*
mario_limonciello@dell.com

[-- Attachment #1.2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 260 bytes --]

[-- Attachment #2: Type: text/plain, Size: 160 bytes --]

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel