[alsa-devel] [PATCH] ALSA: hda - Fix missing ELD info when using jackpoll_ms parameter
david.henningsson at canonical.com
Wed Dec 4 08:56:05 CET 2013
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 at 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
If you mean this patch:
...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.
David Henningsson, Canonical Ltd.
More information about the Alsa-devel