Re: [PATCH] ASoC: Intel: Power down links before turning off display audio power

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux