Yes Jesse, this is a warning fix. -----Original Message----- From: Jesse Barnes [mailto:jbarnes@xxxxxxxxxxxxxxxx] Sent: Monday, November 25, 2013 10:09 PM To: S, Deepak Cc: intel-gfx@xxxxxxxxxxxxxxxxxxxxx Subject: Re: [PATCH 3/3] drm/i915: Enabling DebugFS for valleyview forcewake counts On Sat, 23 Nov 2013 14:55:44 +0530 deepak.s@xxxxxxxxx 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 <jbarnes@xxxxxxxxxxxxxxxx> -- Jesse Barnes, Intel Open Source Technology Center _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
