> > On 04/29/2013 05:02 PM, Jesse Barnes wrote: > > > On Sat, 27 Apr 2013 13:35:29 +0200 > > > Daniel Vetter <daniel at ffwll.ch> wrote: > > > The high level goal here should be for the audio driver to call into > > > i915 with get/put power well around the sequences where it needs the > > > power to be up (reading/writing registers, playing audio), but not > > > across the whole time the driver is loaded, just like you already do > > > with the powersave work functions, e.g. hda_call_codec_suspend. Hi Daniel, We can modify the patch so that the audio driver will not request the power well across the whole time the HD-A driver is loaded, but request/release the power well at runtime. The HD-Audio driver can support runtime PM. When the codec is idle, it can suspend the codec and further suspend the controller, thus the power well is no longer needed. So we can add haswell-specific code to release the power well in HD-A runtime suspend callback azx_suspend(), and request the power well in audio runtime resume callback azx_resume(). Thanks Mengdong
