2013-12-04 15:42, Takashi Iwai skrev:
At Wed, 04 Dec 2013 15:24:28 +0800, David Henningsson wrote:
2013-12-04 15:17, Takashi Iwai skrev:
At Wed, 4 Dec 2013 10:19:41 +0800, David Henningsson wrote:
In the case of using jackpoll_ms instead of unsol events, the jack was correctly detected, but ELD info was not refreshed on plug-in.
And without ELD info, no proper restriction of pcm, which can in turn break sound output on some devices.
Signed-off-by: David Henningsson david.henningsson@canonical.com
sound/pci/hda/patch_hdmi.c | 27 +++++++++++++++------------ 1 file changed, 15 insertions(+), 12 deletions(-)
@Takashi; feel free to add stable if you think it's worth it. (After all using the jackpoll_ms parameter is not something we usually do.) It's still a bug fix so sending to Linus tree should be okay, I believe.
Hrm, I vaguely remember that I gave a very similar patch for testing, but it failed by some reason...
That indeed sounds vague. :-) At this point, is there anything required on my part to help getting this bug fixed? We run into it when we're backporting kernel features, i e, using graphics driver from one kernel version and hda driver from another kernel version... (and in this weird kernel mixup we haven't figured out why we don't get an interrupt, so as a workaround I'm using jackpoll_ms until we've figured that out.)
It's in a thread in June, about [PATCH 1/2] ALSA: hdmi - poll eld at resume time
Takashi
If you mean this patch:
http://mailman.alsa-project.org/pipermail/alsa-devel/2013-June/063336.html
...it would probably resolve this issue, but instead cause problems when jackpoll is not used, because snd_hda_jack_report_sync does not call the jack callback. Essentially, you need to call hdmi_jack_detect_cb from hdmi_intrinsic_event and your patch should work too.
I think - this analysis is all done by quickly looking at your patch, not by actual testing.