On Fri, Sep 05, 2014 at 05:04:45PM -0300, Gustavo Padovan wrote: > From: Gustavo Padovan <gustavo.pado...@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.syrj...@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 -- Ville Syrjälä Intel OTC _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx
