[Intel-gfx] [PATCH 13/13] drm/i915: Always call intel_update_sprite_watermarks() when disabling a plane

Daniel Vetter daniel at ffwll.ch
Thu Aug 8 11:51:03 CEST 2013


On Tue, Aug 06, 2013 at 09:06:16PM +0100, Chris Wilson wrote:
> On Tue, Aug 06, 2013 at 10:24:12PM +0300, ville.syrjala at linux.intel.com wrote:
> > From: Ville Syrjälä <ville.syrjala at linux.intel.com>
> > 
> > ILK and VLV codepaths didn't update sprite watermarks when disabling a
> > sprite. Make them do that.
> > 
> > Signed-off-by: Ville Syrjälä <ville.syrjala at linux.intel.com>
> 
> I am totally loving the naming confusion between intel_plane and
> intel_sprite in intel_display.c. There when we refer to a plane, we may
> mean a slice of the CRTC pipeline or what we refer elsewhere as a sprite.
> 
> e.g. intel_disable_plane and intel_plane_disable.
> 
> Having got past that to realise this is intel_sprite.c not
> intel_display.c,
> Reviewed->by: Chris Wilson <chris at chris-wilson.co.uk>

All merged to dinq, thanks for patches&review.
-Daniel
-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch



More information about the Intel-gfx mailing list