On Tue, Aug 06, 2013 at 10:24:12PM +0300, ville.syrjala@xxxxxxxxxxxxxxx wrote: > From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > > 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@xxxxxxxxxxxxxxx> 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@xxxxxxxxxxxxxxxxxx> -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx