Hi Takashi

Is the attachment what you suspect? That merged to our kernel v4.19 already, have any suggestions? Thanks.

-----Original Message-----
From: Takashi Iwai <tiwai@suse.de> 
Sent: Wednesday, January 8, 2020 2:57 AM
To: Nathan Ciobanu <nathan.d.ciobanu@linux.intel.com>
Cc: Kao. Lucien (TPE) <Lucien_Kao@compal.com>; Cheng. AJ (TPE) <AJ_Cheng@compal.com>; intel-gfx@lists.freedesktop.org; alsa-devel@alsa-project.org; Wang. CindyXT (TPE) <CindyXT_Wang@compal.com>; Ye. Nelson (TPE) <Nelson_Ye@compal.com>; Yap. Shane (TPE) <Shane_Yap@compal.com>; Tseng. Evan (TPE) <Evan_Tseng@compal.com>
Subject: Re: [alsa-devel] USB Type-C monitor flashes once when play a video file after unplug and re-plug the monitor

On Tue, 07 Jan 2020 18:24:57 +0100,
Nathan Ciobanu wrote:
> 
> On Mon, Jan 06, 2020 at 08:08:04AM +0000, Lucien_Kao@compal.com wrote:
> > Hi Takashi
> > 
> > We verified on Ubuntu 19.10 with kernel 5.4.0.0-050400-generic (please refer to attachment), the result is positive which symptom doesn't happen anymore once I played music or video sound output through Dell S2718D Type-C monitor. It seems had some fix in latest kernel.
> 
> Takashi, can you point to the patch series you suspect may have fixed this issue? 

The first suspect would be
2756d9143aa517b97961e85412882b8ce31371a6
    ALSA: hda - Fix intermittent CORB/RIRB stall on Intel chips
    

Takashi

> 
> Thanks,
> Nathan
> > 
> > Thanks.
> > 
> > 
> > -----Original Message-----
> > From: Takashi Iwai <tiwai@suse.de>
> > Sent: Friday, January 3, 2020 5:16 PM
> > To: Cheng. AJ (TPE) <AJ_Cheng@compal.com>
> > Cc: intel-gfx@lists.freedesktop.org; alsa-devel@alsa-project.org; 
> > nathan.d.ciobanu@linux.intel.com; Wang. CindyXT (TPE) 
> > <CindyXT_Wang@compal.com>; Ye. Nelson (TPE) <Nelson_Ye@compal.com>; 
> > Yap. Shane (TPE) <Shane_Yap@compal.com>; Kao. Lucien (TPE) 
> > <Lucien_Kao@compal.com>; Tseng. Evan (TPE) <Evan_Tseng@compal.com>
> > Subject: Re: [alsa-devel] USB Type-C monitor flashes once when play 
> > a video file after unplug and re-plug the monitor
> > 
> > On Fri, 03 Jan 2020 02:57:03 +0100,
> > <AJ_Cheng@compal.com> wrote:
> > > 
> > > Hi Sirs,
> > >         Here is chromebook SW team from Compal.
> > > As the mail title, we hit issue that the external monitor will flash once when play video after hot pluging.
> > > We can reproduce not only on chromebook but also ubuntu 16.04.
> > > There has higher failure rate with Dell Solomon dock and Dell S2718D monitor.
> > > 
> > > We found adding the delay in "sound/pci/hda/patch_hdmi.c " can fix 
> > > this issue.(as the attachment) May need your help to review and advice. Thanks.
> > > 
> > > Here is the issue number in gitlab for more detail.
> > > https://gitlab.freedesktop.org/drm/intel/issues/318
> > 
> > Could you check whether it still happens with the latest upstream kernel, at least 5.4.y, if it wasn't tested yet?
> > 
> > I don't want to put a long delay just because of random reason unless it's really mandatory.  I'm wondering whether the recent write-sync change improves the situation, so let's check the recent code.
> > 
> > 
> > thanks,
> > 
> > Takashi
> > 
> > > 
> > > 
> > > 
> > > AJ Cheng
> > > NID/NID1
> > > e-mail: AJ_Cheng@compal.com<mailto:AJ_Cheng@compal.com>
> > > Tel:  +886-2-8797-8599 ext. 17561
> > > Mobile : +886-932827829
> > > COMPAL Electronics, Inc.
> > > 
> > > [2 flash_once.diff <application/octet-stream (base64)>]
> > > 
> > > _______________________________________________
> > > Alsa-devel mailing list
> > > Alsa-devel@alsa-project.org
> > > https://mailman.alsa-project.org/mailman/listinfo/alsa-devel
> 
>