On Fri, Sep 05, 2014 at 05:04:45PM -0300, Gustavo Padovan wrote:
From: Gustavo Padovan gustavo.padovan@collabora.co.uk
This is the beginning of the work to prepare i915 for the upcoming atomic modesetting API. Here we split the plane update fucntions in the check and commit states.
v2: use struct intel_plane_state to keep states between check and commit stages.
v3: take Ville's comments: - rename pstate to state - get rid of non-drm_rect coordinates in intel_plane_state - keep 'clip' const
v4: take more Ville's comments:
- populates orig_dst and orig_src too
- use orig_dst coordinates to program the cursor plane
Gustavo Padovan (4): drm/i915: create struct intel_plane_state drm/i915: split intel_update_plane into check() and commit() drm/i915: split intel_cursor_plane_update() into check() and commit() drm/i915: split intel_primary_plane_setplane() into check() and commit()
It's looking pretty nice. I didn't spot any more problems, so for the series: Reviewed-by: Ville Syrjälä ville.syrjala@linux.intel.com
drivers/gpu/drm/i915/intel_display.c | 240 +++++++++++++++++++++-------------- drivers/gpu/drm/i915/intel_drv.h | 12 ++ drivers/gpu/drm/i915/intel_sprite.c | 233 ++++++++++++++++++++-------------- 3 files changed, 298 insertions(+), 187 deletions(-)
-- 1.9.3
Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx