On Tue, 04 Feb 2020 17:10:45 +0100, Martin Regner wrote:
Applied the new patch. The device is detected correctly by pulseaudio. Thanks for your efforts.
Great, thanks for quick testing. Now I applied and pushed out.
Takashi
On 04.02.20 12:18, Takashi Iwai wrote:
On Tue, 04 Feb 2020 11:27:46 +0100, Nikhil Mahale wrote:
If dyn_pcm_assign is set, different jack objects are being created for pcm and pins.
If dyn_pcm_assign is set, generic_hdmi_build_jack() calls into add_hdmi_jack_kctl() to create and track separate jack object for pcm. Like sync_eld_via_acomp(), hdmi_present_sense_via_verbs() also need to report status change of the pcm jack.
Rename pin_idx_to_jack() to pin_idx_to_pcm_jack(). Update hdmi_present_sense_via_verbs() to report plug state of pcm jack object. Unlike sync_eld_via_acomp(), for !acomp drivers the pcm jack's plug state must be consistent with plug state of pin's jack.
Thanks, the new patch looks better.
Fixes: 5398e94fb753 ("ALSA: hda - Add DP-MST support for NVIDIA codecs") Signed-off-by: Nikhil Mahale nmahale@nvidia.com
We need Cc to stable here. I'll add it when applying.
Also, it deserves Reported-by from Martin. Martin, could you retest with this patch? I'll queue the patch once after confirmation.
Just one minor nitpick:
if (spec->dyn_pcm_assign && pcm_jack && !do_repoll) {int state = 0;if (!!(jack->pin_sense & AC_PINSENSE_PRESENCE))state = SND_JACK_AVOUT;The "!!" is superfluous. I'll drop it.
Takashi