On Wed, 27 Mar 2013 17:49:13 +0200 ville.syrjala at linux.intel.com wrote: > From: Ville Syrj?l? <ville.syrjala at linux.intel.com> > > When disabling a sprite, wait for the sprite to stop fetching data > from memory before unpinning the fb. > > Signed-off-by: Ville Syrj?l? <ville.syrjala at linux.intel.com> > --- > drivers/gpu/drm/i915/intel_sprite.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/gpu/drm/i915/intel_sprite.c b/drivers/gpu/drm/i915/intel_sprite.c > index 414d325..27df5b8 100644 > --- a/drivers/gpu/drm/i915/intel_sprite.c > +++ b/drivers/gpu/drm/i915/intel_sprite.c > @@ -582,6 +582,8 @@ intel_disable_plane(struct drm_plane *plane) > if (!intel_plane->obj) > goto out; > > + intel_wait_for_vblank(dev, intel_plane->pipe); > + > mutex_lock(&dev->struct_mutex); > intel_unpin_fb_obj(intel_plane->obj); > intel_plane->obj = NULL; Reviewed-by: Jesse Barnes <jbarnes at virtuousgeek.org> -- Jesse Barnes, Intel Open Source Technology Center