[Intel-gfx] [PATCH 00/12] drm/i915: Some cleanups and fixes to the sprite code v2
chris at chris-wilson.co.uk
Fri Oct 4 12:28:16 CEST 2013
On Tue, Oct 01, 2013 at 06:02:09PM +0300, ville.syrjala at linux.intel.com wrote:
> Chris asked for some renames and assertions during v1. While adding those I
> noticed that what I did in the original patch 02 didn't match quite so well
> with the assertions. So I modified patch 02 a bit, and that caused quite a bit
> of bit of rebase issues for most of the other patches, so I figured it's better
> to repost the whole thing.
> Changes from v1:
> - Move the primary disable/enable calls inside intel_crtc->active checks
> in intel_update_plane/intel_disable_plane. That also ate up patch 03 from
> the original series.
> - Add primary_disabled WARNs
> - Rename primary plane funcs
> - Flush primary plane changes from sprite code
> - Add a POSTING_READ() to intel_flush_primary_plane. This shouldn't really
> be necessary now that I think about it some more. So we might want to drop
> that change...
Looks good, very good, to me.
Even with throwing up over FBC,
Reviewed-by: Chris Wilson <chris at chris-wilson.co.uk>
08/12: drm/i915: Enable/disable IPS when primary is
For which the code looks ok, but only merits an
Acked-by: Chris Wilson <chris at chris-wilson.co.uk>
Chris Wilson, Intel Open Source Technology Centre
More information about the Intel-gfx