On 11/17/15 12:28, Daniel Vetter wrote:
On Fri, Nov 13, 2015 at 05:53:13PM +0200, Jyri Sarha wrote:
Since first RFC version:
- Added "drm/atomic: Track drm_plane's active state"-patch
We would need something like this to get rid off OMAPDSS somewhat messy runtime_resume code. How does this look, could this generic solution be refined to be acceptable for mainline, or should we start looking a local solution to omapdrm?
One more comment besides what I've written in the older thread: You're not on latest drm-next, which has gained an active_only paramter to the plane commit helper.
It might be best to discuss this topic on #dri-devel on freenode irc a bit.
Thanks a lot for you comments!
I just rebased my stuff on top the the latest drm-next. I think I need digest your earlier comments about drm_atomic_add_affected_planes etc. a bit before I can take advantage of any interactive communication. I'll try to follow those instruction first and see where it leads me.
Thanks, Jyri
Cheers, Daniel
Jyri Sarha (2): drm/atomic: Track drm_plane's active state drm/atomic: Disable planes on blanked CRTC and enable on unblank
drivers/gpu/drm/drm_atomic_helper.c | 82 +++++++++++++++++++++++++++++-------- drivers/gpu/drm/drm_plane_helper.c | 10 +---- include/drm/drm_atomic_helper.h | 39 +----------------- include/drm/drm_crtc.h | 2 + 4 files changed, 70 insertions(+), 63 deletions(-)
-- 1.9.1