On Wed, Mar 05, 2014 at 01:05:45PM +0200, ville.syrj...@linux.intel.com wrote:
> From: Ville Syrjälä <ville.syrj...@linux.intel.com>
> 
> Misplaced parens cause us to totally clobber the CHICKEN_PIPESL_1
> registers with 0xffffffff. Move the parens to the correct place
> to avoid this.
> 
> In particular this caused bit 30 of said registers to be set, which
> caused the sprite CSC to produce incorrect results.
> 
> Cc: sta...@vger.kernel.org
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=72220
> Signed-off-by: Ville Syrjälä <ville.syrj...@linux.intel.com>

Reviewed-by: Damien Lespiau <damien.lesp...@intel.com>

-- 
Damien

> ---
>  drivers/gpu/drm/i915/intel_pm.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c
> index 6436b70..245d3ae 100644
> --- a/drivers/gpu/drm/i915/intel_pm.c
> +++ b/drivers/gpu/drm/i915/intel_pm.c
> @@ -4832,8 +4832,8 @@ static void gen8_init_clock_gating(struct drm_device 
> *dev)
>       /* WaPsrDPRSUnmaskVBlankInSRD:bdw */
>       for_each_pipe(i) {
>               I915_WRITE(CHICKEN_PIPESL_1(i),
> -                        I915_READ(CHICKEN_PIPESL_1(i) |
> -                                  DPRS_MASK_VBLANK_SRD));
> +                        I915_READ(CHICKEN_PIPESL_1(i)) |
> +                        DPRS_MASK_VBLANK_SRD);
>       }
>  
>       /* Use Force Non-Coherent whenever executing a 3D context. This is a
> -- 
> 1.8.3.2
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to