Yes Jesse, this is a warning fix. 

-----Original Message-----
From: Jesse Barnes [mailto:jbar...@virtuousgeek.org] 
Sent: Monday, November 25, 2013 10:09 PM
To: S, Deepak
Cc: intel-gfx@lists.freedesktop.org
Subject: Re: [Intel-gfx] [PATCH 3/3] drm/i915: Enabling DebugFS for valleyview 
forcewake counts

On Sat, 23 Nov 2013 14:55:44 +0530
deepa...@intel.com wrote:
> @@ -2349,7 +2357,7 @@ static int pipe_crc_set_source(struct drm_device 
> *dev, enum pipe pipe,  {
>       struct drm_i915_private *dev_priv = dev->dev_private;
>       struct intel_pipe_crc *pipe_crc = &dev_priv->pipe_crc[pipe];
> -     u32 val;
> +     u32 val = 0;
>       int ret;
>  
>       if (pipe_crc->source == source)

Spurious warning fix?  Otherwise looks fine.

Reviewed-by: Jesse Barnes <jbar...@virtuousgeek.org>

--
Jesse Barnes, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to