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

Chris Wilson chris at chris-wilson.co.uk
Tue Aug 6 22:06:16 CEST 2013


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>
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre



More information about the Intel-gfx mailing list