On Mon, Sep 08, 2014 at 04:59:42PM +0300, Ville Syrjälä wrote: > On Fri, Sep 05, 2014 at 05:04:45PM -0300, Gustavo Padovan wrote: > > From: Gustavo Padovan <gustavo.padovan@xxxxxxxxxxxxxxx> > > > > 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@xxxxxxxxxxxxxxx> Queued for -next, thanks for the patch. -Daniel > > > > > 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@xxxxxxxxxxxxxxxxxxxxx > > http://lists.freedesktop.org/mailman/listinfo/intel-gfx > > -- > Ville Syrjälä > Intel OTC > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation +41 (0) 79 365 57 48 - http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
