Re: Surround speaker connection on Acer 8951G

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

 



On Mon, 02 Sep 2019 23:39:12 +0200,
Sergey 'Jin' Bostandzhyan wrote:
> 
> > One thing to be improved would be to drop the surround jack control.
> > Adjust the pin config to different value with the fixed pin
> > connection, so that the auto-parser won't create the "Surround Jack"
> > control.  This isn't needed by PA or else, otherwise it may be
> > confusing.
> 
> Hmm, if I understand you correctly, then you are referring to bits 31:30
> Port Connectivity? 
> 
> It does not seem to work that way... I tried all combinations and I either
> lose jack detect support or I lose the 5.1 profile in Pulse.
> 
> With these settings snd_hda_jack_detect_state() never returns HDA_JACK_PRESENT:
> 0x91130012 [Fixed] Speaker at Int Rear
> 0xd1130012 [Both] Speaker at Int Rear
> 
> I can plug or unplug, I get called, but I always receive HDA_JACK_PHANTOM
> 
> snd_hda_jack_detect_state() works fine with "no physical connection to port":
> 0x51130012 [N/A] Speaker at Int Rear
> 
> But with the above pin setting I "lose" the 5.1 profile in Pulse...
> 
> Which leaves me with with what I had before:
> 0x11130012 [Jack] Speaker at Int Rear
> 
> Am I missing something or did you mean some other setting? Should I be
> using a different function instead of snd_hda_jack_detect_state() to
> check my jack state in the callback?

OK, this would be really tricky to work around it.  It's merely a jack
control that won't be referred by PA, so we can live with it, unless
you see any obvious side effect.

So, when the patch is ready for submission, feel free to send it.


thanks,

Takashi
_______________________________________________
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