[alsa-devel] [PATCH] ALSA: hda - Fix missing ELD info when using jackpoll_ms parameter
tiwai at suse.de
Wed Dec 4 09:04:43 CET 2013
At Wed, 04 Dec 2013 15:56:05 +0800,
David Henningsson wrote:
> 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
> > Takashi
> 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.
OK, then I took your patch. It shouldn't break the things, at least,
through a quick glance.
More information about the Alsa-devel