From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> Here's a quick series that tries to make our .crtc_{enable,disable}() update the derived plane state correctly. The upside of doing this is that we can actually trust the 'visible' and the clipped src/dst coordinates for all planes. I also took the opportunity to make the plane enable/disable happen atomically when the crtc is enabled/disabled. And now that the derived plane state is correct I added a few patchs to use that information in the wm code, and also to prepare for primary plane windowing. Ville Syrjälä (9): drm/i915: Remove debug prints from primary plane update funcs drm/i915: Reduce clutter by using the local plane pointer drm/i915: Use plane->state->fb instead of plane->fb in intel_plane_restore() drm/i915: Make derived plane state correct after crtc_enable drm/i915: Pass primary plane size to .update_primary_plane() drm/i915: Pass the primary plane position to .update_primary_plane() drm/i915: Update watermarks after the derived plane state is uptodate drm/i915: Use state->visible in wm calculation drm/i915: Don't re-enable an explicitly disabled primary plane due to sprite coverage changes drivers/gpu/drm/i915/i915_drv.h | 7 +- drivers/gpu/drm/i915/intel_display.c | 310 +++++++++++++++-------------------- drivers/gpu/drm/i915/intel_pm.c | 24 ++- drivers/gpu/drm/i915/intel_sprite.c | 8 +- 4 files changed, 163 insertions(+), 186 deletions(-) -- 2.0.5 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx