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@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> 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 _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
