On 02/10/2015 05:28 AM, ville.syrjala@xxxxxxxxxxxxxxx wrote: > @@ -957,8 +954,7 @@ static void valleyview_update_sprite_wm(struct drm_plane *plane, > int plane_prec; > int sprite_dl; > int prec_mult; > - const int high_precision = IS_CHERRYVIEW(dev) ? > - DRAIN_LATENCY_PRECISION_32 : DRAIN_LATENCY_PRECISION_64; > + const int high_precision = IS_CHERRYVIEW(dev) ? 16 : 64; > > sprite_dl = I915_READ(VLV_DDL(pipe)) & ~(DDL_SPRITE_PRECISION_HIGH(sprite) | > (DRAIN_LATENCY_MASK << DDL_SPRITE_SHIFT(sprite))); > This hunk changes things; did it belong in the earlier patch? With that explained: Reviewed-by: Jesse Barnes <jbarnes@xxxxxxxxxxxxxxxx> _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
