Re: [PATCH 3/3] ASoC: hdac_hdmi : Ensuring proper setting of output widget power state
On 2/14/2018 9:50 PM, Mark Brown wrote: On Tue, Jan 23, 2018 at 11:00:53PM +0530, abhijeet.ku...@intel.com wrote: From: Abhijeet KumarIn usecases like hot plug-unplug DP panel or modeset during a playback, sometimes we observe no audio after codec resets. During no audio condition, we have noticed that the power state of the pin or the connector is D3. Optimizing the way we set the power mitigates the issue. With this changes the verb is sent to set the power state and waits until actual state reaches target state. Thus ensuring power state is set. This doesn't apply after the recent refactoring to move to components - can you please rebase on current code and resend? I pulled Takashi's branch already, it's just this patch needs handling. Please find the re-based patch here. https://patchwork.kernel.org/patch/10220549/ Warm Regards, Abhijeet
Re: [PATCH 3/3] ASoC: hdac_hdmi : Ensuring proper setting of output widget power state
On 2/14/2018 9:50 PM, Mark Brown wrote: On Tue, Jan 23, 2018 at 11:00:53PM +0530, abhijeet.ku...@intel.com wrote: From: Abhijeet Kumar In usecases like hot plug-unplug DP panel or modeset during a playback, sometimes we observe no audio after codec resets. During no audio condition, we have noticed that the power state of the pin or the connector is D3. Optimizing the way we set the power mitigates the issue. With this changes the verb is sent to set the power state and waits until actual state reaches target state. Thus ensuring power state is set. This doesn't apply after the recent refactoring to move to components - can you please rebase on current code and resend? I pulled Takashi's branch already, it's just this patch needs handling. Please find the re-based patch here. https://patchwork.kernel.org/patch/10220549/ Warm Regards, Abhijeet
Re: [PATCH 3/3] ASoC: hdac_hdmi : Ensuring proper setting of output widget power state
On Tue, Jan 23, 2018 at 11:00:53PM +0530, abhijeet.ku...@intel.com wrote: > From: Abhijeet Kumar> > In usecases like hot plug-unplug DP panel or modeset during a playback, > sometimes we observe no audio after codec resets. During no audio > condition, we have noticed that the power state of the pin or the > connector is D3. Optimizing the way we set the power mitigates the > issue. With this changes the verb is sent to set the power state and > waits until actual state reaches target state. Thus ensuring power state > is set. This doesn't apply after the recent refactoring to move to components - can you please rebase on current code and resend? I pulled Takashi's branch already, it's just this patch needs handling. signature.asc Description: PGP signature
Re: [PATCH 3/3] ASoC: hdac_hdmi : Ensuring proper setting of output widget power state
On Tue, Jan 23, 2018 at 11:00:53PM +0530, abhijeet.ku...@intel.com wrote: > From: Abhijeet Kumar > > In usecases like hot plug-unplug DP panel or modeset during a playback, > sometimes we observe no audio after codec resets. During no audio > condition, we have noticed that the power state of the pin or the > connector is D3. Optimizing the way we set the power mitigates the > issue. With this changes the verb is sent to set the power state and > waits until actual state reaches target state. Thus ensuring power state > is set. This doesn't apply after the recent refactoring to move to components - can you please rebase on current code and resend? I pulled Takashi's branch already, it's just this patch needs handling. signature.asc Description: PGP signature
Re: [PATCH 3/3] ASoC: hdac_hdmi : Ensuring proper setting of output widget power state
On Tue, 23 Jan 2018 18:30:53 +0100,wrote: > > From: Abhijeet Kumar > > In usecases like hot plug-unplug DP panel or modeset during a playback, > sometimes we observe no audio after codec resets. During no audio > condition, we have noticed that the power state of the pin or the > connector is D3. Optimizing the way we set the power mitigates the > issue. With this changes the verb is sent to set the power state and > waits until actual state reaches target state. Thus ensuring power state > is set. > > Signed-off-by: Abhijeet Kumar I applied the patches 1 and 2 to topic/hda-sync-power branch. It's based on 4.16-rc1. Mark, could you pull that into your intel branch and apply this fix? thanks, Takashi > --- > sound/soc/codecs/hdac_hdmi.c | 16 +--- > 1 file changed, 13 insertions(+), 3 deletions(-) > > diff --git a/sound/soc/codecs/hdac_hdmi.c b/sound/soc/codecs/hdac_hdmi.c > index f3b4f4dfae6a..4dc9b9b71db9 100644 > --- a/sound/soc/codecs/hdac_hdmi.c > +++ b/sound/soc/codecs/hdac_hdmi.c > @@ -716,10 +716,20 @@ static struct hdac_hdmi_pcm *hdac_hdmi_get_pcm(struct > hdac_ext_device *edev, > static void hdac_hdmi_set_power_state(struct hdac_ext_device *edev, >hda_nid_t nid, unsigned int pwr_state) > { > + int count; > + unsigned int state; > if (get_wcaps(>hdac, nid) & AC_WCAP_POWER) { > - if (!snd_hdac_check_power_state(>hdac, nid, pwr_state)) > - snd_hdac_codec_write(>hdac, nid, 0, > - AC_VERB_SET_POWER_STATE, pwr_state); > + if (!snd_hdac_check_power_state(>hdac, nid, pwr_state)) { > + for (count = 0; count < 10; count++) { > + snd_hdac_codec_read(>hdac, nid, 0, > + AC_VERB_SET_POWER_STATE, > + pwr_state); > + state = snd_hdac_sync_power_state(>hdac, > + nid, pwr_state); > + if (!(state & AC_PWRST_ERROR)) > + break; > + } > + } > } > } > > -- > 1.9.1 > >
Re: [PATCH 3/3] ASoC: hdac_hdmi : Ensuring proper setting of output widget power state
On Tue, 23 Jan 2018 18:30:53 +0100, wrote: > > From: Abhijeet Kumar > > In usecases like hot plug-unplug DP panel or modeset during a playback, > sometimes we observe no audio after codec resets. During no audio > condition, we have noticed that the power state of the pin or the > connector is D3. Optimizing the way we set the power mitigates the > issue. With this changes the verb is sent to set the power state and > waits until actual state reaches target state. Thus ensuring power state > is set. > > Signed-off-by: Abhijeet Kumar I applied the patches 1 and 2 to topic/hda-sync-power branch. It's based on 4.16-rc1. Mark, could you pull that into your intel branch and apply this fix? thanks, Takashi > --- > sound/soc/codecs/hdac_hdmi.c | 16 +--- > 1 file changed, 13 insertions(+), 3 deletions(-) > > diff --git a/sound/soc/codecs/hdac_hdmi.c b/sound/soc/codecs/hdac_hdmi.c > index f3b4f4dfae6a..4dc9b9b71db9 100644 > --- a/sound/soc/codecs/hdac_hdmi.c > +++ b/sound/soc/codecs/hdac_hdmi.c > @@ -716,10 +716,20 @@ static struct hdac_hdmi_pcm *hdac_hdmi_get_pcm(struct > hdac_ext_device *edev, > static void hdac_hdmi_set_power_state(struct hdac_ext_device *edev, >hda_nid_t nid, unsigned int pwr_state) > { > + int count; > + unsigned int state; > if (get_wcaps(>hdac, nid) & AC_WCAP_POWER) { > - if (!snd_hdac_check_power_state(>hdac, nid, pwr_state)) > - snd_hdac_codec_write(>hdac, nid, 0, > - AC_VERB_SET_POWER_STATE, pwr_state); > + if (!snd_hdac_check_power_state(>hdac, nid, pwr_state)) { > + for (count = 0; count < 10; count++) { > + snd_hdac_codec_read(>hdac, nid, 0, > + AC_VERB_SET_POWER_STATE, > + pwr_state); > + state = snd_hdac_sync_power_state(>hdac, > + nid, pwr_state); > + if (!(state & AC_PWRST_ERROR)) > + break; > + } > + } > } > } > > -- > 1.9.1 > >