From mboxrd@z Thu Jan  1 00:00:00 1970
From: Takashi Iwai <tiwai@suse.de>
Subject: Re: HDMI Repoll after pm?
Date: Thu, 10 Dec 2015 14:38:41 +0100
Message-ID: <s5hfuzamnj2.wl-tiwai@suse.de>
References: <56694A4F.8070104@canonical.com> <s5htwnq7hpf.wl-tiwai@suse.de>
 <5669774F.3060709@canonical.com> <s5hk2ommoh6.wl-tiwai@suse.de>
 <56697DF0.20502@canonical.com>
Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka")
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Return-path: <alsa-devel-bounces@alsa-project.org>
Received: from mx2.suse.de (mx2.suse.de [195.135.220.15])
 by alsa0.perex.cz (Postfix) with ESMTP id C3280265325
 for <alsa-devel@alsa-project.org>; Thu, 10 Dec 2015 14:38:41 +0100 (CET)
In-Reply-To: <56697DF0.20502@canonical.com>
List-Unsubscribe: <http://mailman.alsa-project.org/mailman/options/alsa-devel>,
 <mailto:alsa-devel-request@alsa-project.org?subject=unsubscribe>
List-Archive: <http://mailman.alsa-project.org/pipermail/alsa-devel/>
List-Post: <mailto:alsa-devel@alsa-project.org>
List-Help: <mailto:alsa-devel-request@alsa-project.org?subject=help>
List-Subscribe: <http://mailman.alsa-project.org/mailman/listinfo/alsa-devel>,
 <mailto:alsa-devel-request@alsa-project.org?subject=subscribe>
Errors-To: alsa-devel-bounces@alsa-project.org
Sender: alsa-devel-bounces@alsa-project.org
To: David Henningsson <david.henningsson@canonical.com>
Cc: alsa-devel@alsa-project.org
List-Id: alsa-devel@alsa-project.org

On Thu, 10 Dec 2015 14:28:16 +0100,
David Henningsson wrote:
> 
> 
> 
> On 2015-12-10 14:18, Takashi Iwai wrote:
> > On Thu, 10 Dec 2015 13:59:59 +0100,
> > David Henningsson wrote:
> >>
> >>
> >>
> >> On 2015-12-10 10:53, Takashi Iwai wrote:
> >>> On Thu, 10 Dec 2015 10:47:59 +0100,
> >>> David Henningsson wrote:
> >>>>
> >>>> Hi Takashi,
> >>>>
> >>>> A recent change in sound.git seems to have introduced some repoll after
> >>>> pm. Is this an intentional change? I don't think it's wrong per se (it
> >>>> stops after 6 repolls), but it breaks my hda-emu tester when "list" does
> >>>> not return the expected input.
> >>>
> >>> Hm, there is no such an intended change.
> >>
> >> After testing some more, I've found that reverting e90247f9 (Split ELD
> >> update code from hdmi_present_sense) fixes the issue, so it seems like
> >> this commit indeed contains a functional change...
> >
> > Thanks for chasing it.
> 
> It's almost unreal...my tester framework finally caught a bug (that you 
> didn't catch before I saw it!). I cannot believe it, it only took, like, 
> three years or so :-)
> 
> > Does the following patch work better?
> 
> It certainly does. Thanks!
> 
> Tested-by: David Henningsson <david.henningsson@canonical.com>

Thanks!  I merged the fix now.


Takashi