mbox series

[SRU,OEM-5.6,0/1] The DP/HDMI audio via USB-C to DP dongle or Dell Zeus adapter can't work after suspend

Message ID 20200828025304.7390-1-hui.wang@canonical.com
Headers show
Series The DP/HDMI audio via USB-C to DP dongle or Dell Zeus adapter can't work after suspend | expand

Message

Hui Wang Aug. 28, 2020, 2:53 a.m. UTC 
BugLink: https://bugs.launchpad.net/bugs/1893290

Since this patch is CCed to stable kernel, I just sent this patch
to oem-5.6 kernel, our oem project is waiting for this patch to
be landed to oem-5.6 kernel first.

[Impact]
On the Dell TGL laptop, if connecting a monitor via type-c to DP/HDMI
adapter, we will have an audio issue, that is playing the music and
suspend the system, after resuming back, the audio can't output from
DP/HDMI monitor anymore.

[Fix]
Cherry-pick a fix from upstream, this patch is verifying the needed
converter and pin to be in the D0 state.

[Test Case]
Connect a monitor with USB-C to DP/HDMI adapter, open the youtu.com
and playing sth, suspend the system and resume, check if the sound
could play the music, repeat 20 times, all work very well.

[Regression Risk]
If this patch could introduce regression, it should be the power
consumption regression on Intel HDMI audio, since it makes the
converter and pin to be D0 state earlier than before. But this
impact is very small. one converter and one pin consume very little
power.

Kai Vehmanen (1):
  ALSA: hda/hdmi: always check pin power status in i915 pin fixup

 sound/pci/hda/patch_hdmi.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Timo Aaltonen Sept. 1, 2020, 10:39 a.m. UTC | #1 
On 28.8.2020 5.53, Hui Wang wrote:
> BugLink: https://bugs.launchpad.net/bugs/1893290
> 
> Since this patch is CCed to stable kernel, I just sent this patch
> to oem-5.6 kernel, our oem project is waiting for this patch to
> be landed to oem-5.6 kernel first.
> 
> [Impact]
> On the Dell TGL laptop, if connecting a monitor via type-c to DP/HDMI
> adapter, we will have an audio issue, that is playing the music and
> suspend the system, after resuming back, the audio can't output from
> DP/HDMI monitor anymore.
> 
> [Fix]
> Cherry-pick a fix from upstream, this patch is verifying the needed
> converter and pin to be in the D0 state.
> 
> [Test Case]
> Connect a monitor with USB-C to DP/HDMI adapter, open the youtu.com
> and playing sth, suspend the system and resume, check if the sound
> could play the music, repeat 20 times, all work very well.
> 
> [Regression Risk]
> If this patch could introduce regression, it should be the power
> consumption regression on Intel HDMI audio, since it makes the
> converter and pin to be D0 state earlier than before. But this
> impact is very small. one converter and one pin consume very little
> power.
> 
> Kai Vehmanen (1):
>   ALSA: hda/hdmi: always check pin power status in i915 pin fixup
> 
>  sound/pci/hda/patch_hdmi.c | 1 +
>  1 file changed, 1 insertion(+)
> 

applied to oem-5.6, thanks