]> Pileus Git - ~andy/linux/commitdiff
drm/i915: make sparse happy for the new vlv mmio read function
authorDaniel Vetter <daniel.vetter@ffwll.ch>
Wed, 27 Nov 2013 19:52:23 +0000 (20:52 +0100)
committerDaniel Vetter <daniel.vetter@ffwll.ch>
Thu, 28 Nov 2013 07:31:48 +0000 (08:31 +0100)
It doesn't like that we assign 0 to a pointer, it wants the real NULL.

On closer look that initialization is actually bogus, and the compiler
can easily see that we never use it unitialized. So let's just drop
this.

Cc: Deepak S <deepak.s@intel.com>
Cc: Jesse Barnes <jbarnes@virtuousgeek.org>
Reported-by: kbuild test robot <fengguang.wu@intel.com>
Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch>
drivers/gpu/drm/i915/intel_uncore.c

index a7c6de7d6240ceb5335ef67bcff9c5b1aa685986..d511e00095ac8e63f09b8a2e6456be711bc20f7b 100644 (file)
@@ -479,7 +479,7 @@ gen6_read##x(struct drm_i915_private *dev_priv, off_t reg, bool trace) { \
 static u##x \
 vlv_read##x(struct drm_i915_private *dev_priv, off_t reg, bool trace) { \
        unsigned fwengine = 0; \
-       unsigned *fwcount = 0; \
+       unsigned *fwcount; \
        REG_READ_HEADER(x); \
        if (FORCEWAKE_VLV_RENDER_RANGE_OFFSET(reg)) {   \
                fwengine = FORCEWAKE_RENDER;            \