From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jesse Barnes Subject: Re: [PATCH 3/3] drm/i915: Enabling DebugFS for valleyview forcewake counts Date: Mon, 25 Nov 2013 08:38:35 -0800 Message-ID: <20131125083835.5ccda77f@jbarnes-desktop> References: <1385198744-9333-1-git-send-email-deepak.s@intel.com> <1385198744-9333-4-git-send-email-deepak.s@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from alt-proxy12.mail.unifiedlayer.com (alt-proxy12.mail.unifiedlayer.com [67.20.64.47]) by gabe.freedesktop.org (Postfix) with SMTP id 1F1D8FA3F2 for ; Mon, 25 Nov 2013 08:43:59 -0800 (PST) In-Reply-To: <1385198744-9333-4-git-send-email-deepak.s@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces@lists.freedesktop.org Errors-To: intel-gfx-bounces@lists.freedesktop.org To: deepak.s@intel.com Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Sat, 23 Nov 2013 14:55:44 +0530 deepak.s@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 -- Jesse Barnes, Intel Open Source Technology Center