On Tue, 24 Apr 2012 09:55:08 +0200, Daniel Vetter <daniel.vet...@ffwll.ch> 
wrote:
> Also ditch the cargo-culted dev_priv checks - either we have a
> giant hole in our setup code or this is useless. Plainly bogus
> to check for it in either case.
> 
> v2: Chris Wilson noticed that I've missed one bogus dev_priv check.
> 
> v3: The check in the overlay code is redundant (Chris)
> 
> Signed-off-by: Daniel Vetter <daniel.vet...@ffwll.ch>
Never mind, intel_sprite.c goes through an idr to find its planes and
not a simple pointer hanging off dev_priv. Speaking of which we never
free the memory allocated for the planes...

Reviewed-by: Chris Wilson <ch...@chris-wilson.co.uk>
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to