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