Re: [PATCH] drm/i915: Add private api for power well usage -- alignment between graphic team and audio team

[Daniel Vetter <daniel@ffwll.ch>]

On Sat, Apr 27, 2013 at 09:20:39AM +0000, Wang, Xingchao wrote:
> Let me throw a basic proposal on Audio driver side,  please give your comments freely.
> 
> it contains the power well control usage points:
> #1: audio request power well at boot up.
> I915 may shut down power well after bootup initialization, as there's no monitor connected outside or only eDP on pipe A.
> #2: audio request power on resume
> After exit from D3 mode, audio driver quest power on. This may happen at normal resume or runtime resume.
> #3: audio release power well control at suspend
> Audio driver will let i915 know it doensot need power well anymore as it's going to suspend. This may happened at normal suspend or runtime suspend point.
> #4: audio release power well when module unload
> Audio release power well at remove callback to let i915 know.

I miss the power well grab/dropping at runtime from the audio side. If the
audio driver forces the power well to be on the entire time it's loaded,
that's not good, since the power well stuff is very much for runtime PM.
We _must_ be able to switch off the power well whenever possible.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch