Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754366AbeALFm0 (ORCPT + 1 other); Fri, 12 Jan 2018 00:42:26 -0500 Received: from mga09.intel.com ([134.134.136.24]:43863 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754202AbeALFmZ (ORCPT ); Fri, 12 Jan 2018 00:42:25 -0500 X-Amp-Result: UNSCANNABLE X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.46,347,1511856000"; d="scan'208";a="9024798" Date: Fri, 12 Jan 2018 11:16:38 +0530 From: Vinod Koul To: abhijeet.kumar@intel.com Cc: Liam Girdwood , Mark Brown , Jaroslav Kysela , Takashi Iwai , Jeeja KP , "Subhransu S. Prusty" , Guneshwor Singh , Sandeep Tayal , alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] ASoC: hdac_hdmi: Ensuring proper setting of output widget power state Message-ID: <20180112054638.GL18649@localhost> References: <1515670468-9198-1-git-send-email-abhijeet.kumar@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1515670468-9198-1-git-send-email-abhijeet.kumar@intel.com> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: On Thu, Jan 11, 2018 at 05:04:27PM +0530, abhijeet.kumar@intel.com wrote: > From: Abhijeet Kumar > > When we change the resolution of DP pannel or hot plug-unplug it while > playing an audio clip,sometimes we observe a silent playback(no audio). can you rephrase this please > During no audio condition, we have noticed that the power state of the > pin or the connector is D3. Optimzing the way we set the power could > mitigate the issue.With this changes the verb is sent to set the power space after . > state and response is received. Thus ensuring power state is set. am not sure I fully understood the problem here > > Signed-off-by: Abhijeet Kumar > --- > sound/soc/codecs/hdac_hdmi.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/sound/soc/codecs/hdac_hdmi.c b/sound/soc/codecs/hdac_hdmi.c > index f3b4f4dfae6a..e24caecf0a4f 100644 > --- a/sound/soc/codecs/hdac_hdmi.c > +++ b/sound/soc/codecs/hdac_hdmi.c > @@ -718,7 +718,7 @@ static void hdac_hdmi_set_power_state(struct hdac_ext_device *edev, > { > if (get_wcaps(&edev->hdac, nid) & AC_WCAP_POWER) { > if (!snd_hdac_check_power_state(&edev->hdac, nid, pwr_state)) > - snd_hdac_codec_write(&edev->hdac, nid, 0, > + snd_hdac_codec_read(&edev->hdac, nid, 0, how does read help instead of write? -- ~Vinod