From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: [PATCH] ALSA: hda/hdmi - Don't report Jack event if no need to do that Date: Tue, 30 Apr 2019 09:35:03 +0200 Message-ID: References: <20190430065711.14977-1-hui.wang@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: Received: from mx1.suse.de (mx2.suse.de [195.135.220.15]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by alsa1.perex.cz (Postfix) with ESMTPS id 6CD86F80C0D for ; Tue, 30 Apr 2019 09:35:03 +0200 (CEST) In-Reply-To: <20190430065711.14977-1-hui.wang@canonical.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: "Alsa-devel" To: Hui Wang Cc: alsa-devel@alsa-project.org List-Id: alsa-devel@alsa-project.org On Tue, 30 Apr 2019 08:57:11 +0200, Hui Wang wrote: > > On the machines with AMD GPU or Nvidia GPU, we often meet this issues: > after s3, there are 4 HDMI/DP audio devices in the gnome-sound-setting > even there is no any monitors plugged. > > When this problem happens, we check the /proc/asound/cardX/eld#N.M, we > will find the monitor_present=1, eld_valid=0. > > The root cause is somehow the pin_sense reports the monitor is present > and eld is valid when there is no monitor plugged. > > The current driver will read the eld data if the pin_sense reports the > eld is valid, because of no monitor is plugged, there is no valid eld > data, then the eld->valid is set to 0. > > If we don't let driver report Jack event when monitor_present=1 while > eld_valid=0, there will be no this issue. > > After this change, the driver only reports Jack event with one of the > below 2 conditons: > eld->monitor_present=1 and eld->eld_valid=1 (a valid monitor detect) > eld->monitor_present=0 (a monitor is unplugged) > > Signed-off-by: Hui Wang Well, if the eld_valid=1 is mandatory, basically we can use it as the condition of jack=1, like the patch below. The return value from hdmi_present_sense() indicates only whether we may sync jack state or not, and it's not about the jack state itself. thanks, Takashi --- --- a/sound/pci/hda/patch_hdmi.c +++ b/sound/pci/hda/patch_hdmi.c @@ -1625,7 +1625,7 @@ static void sync_eld_via_acomp(struct hda_codec *codec, if (jack == NULL) goto unlock; snd_jack_report(jack, - eld->monitor_present ? SND_JACK_AVOUT : 0); + (eld->monitor_present && eld->eld_valid) ? SND_JACK_AVOUT : 0); unlock: mutex_unlock(&per_pin->lock); }