Re: [PATCH] ALSA: hda: set the 2nd headphone's name to be Headphone2

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Dne 05. 03. 19 v 15:09 Hui Wang napsal(a):
> Recently I saw two Headphones on some HP machines, the 1st Headphone
> comes from the Front Headphone Jack, the 2nd one comes from the Front
> Headset Jack, and the current HDA driver will set the same name to
> them and with different index, just like below:
> 'Headphone Playback Volume'
> 'Headphone Playback Switch'
> 'Headphone Playback Volume' index 1
> 'Headphone Playback Switch' index 1
> 'Front Headphone Jack'
> 'Front Headphone Jack' index 1
> 
> This can't be handled by pulseaudio, and pulseaduio already has a
> path analog-output-headphones-2.conf to handle the 2nd headphone, now
> we set the 2nd headphone's name to "Headphone2", leave the other
> headphones unchanged.
> 
> The change of hda_auto_parser.c will change the name of Jack.
> 
> The change of hda_generic.c will change the name of Volume and Switch.

NAK. I'm afraid, but you're trying to revert the change from 2009:

commit 668b9652be33510a2a42b290dd335d34d38e2068
Author: Takashi Iwai <tiwai@xxxxxxx>
Date:   Fri Mar 6 10:13:24 2009 +0100

  ALSA: hda - Create multiple HP / speaker controls with index

  Create multiple "Headphone" and "Speaker" controls with non-zero index
  numbers instead of "Headphone2", etc.

  Signed-off-by: Takashi Iwai <tiwai@xxxxxxx>

I think that PA should be fixed instead try to fix the drivers. The
indexes are exactly designed to distinguish the multiple similar I/O.

I cannot find any other driver which uses the Headphone2 control name.

					Jaroslav

-- 
Jaroslav Kysela <perex@xxxxxxxx>
Linux Sound Maintainer; ALSA Project; Red Hat, Inc.
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux