[alsa-devel] [PATCH] ASoC: Intel: Power down links before turning off display audio power

[Takashi Iwai]

On Fri, 22 Jun 2018 18:42:24 +0200,
Pierre-Louis Bossart wrote:
> 
> On 6/22/18 4:25 AM, Sriram Periyasamy wrote:
> > On certain platforms, Display HDMI HDA codec was not going to sleep state
> > after the use when links are powered down after turning off the display
> > power. As per the HW recommendation, links are powered down before turning
> > off the display power to ensure that the codec goes to sleep state.
> 
> The change looks ok, I am still not clear on display power
> management. snd_hdac_display_power is called both from hdac_hdmi.c and
> skl.c in the respective probe/suspend/resume routines, wondering if
> this is redundant or intentional.

Yeah, it looks like that skl.c keeps the display power on by assuming
that it's turned off by the codec driver, but it doesn't happen
actually.  I also find it fishy there...


thanks,

Takashi