On 02/10/2015 05:28 AM, [email protected] 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 <[email protected]>
_______________________________________________
Intel-gfx mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to