drm/i915: Fix uninitialized return from mi_set_context
authorTvrtko Ursulin <tvrtko.ursulin@intel.com>
Tue, 14 Feb 2017 15:29:01 +0000 (15:29 +0000)
committerTvrtko Ursulin <tvrtko.ursulin@intel.com>
Tue, 14 Feb 2017 19:34:54 +0000 (19:34 +0000)
For some reason my compiler (and CI as well) failed to spot the
uninitialized ret in mi_set_context.

Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Fixes: 73dec95e6ba3 ("drm/i915: Emit to ringbuffer directly")
Cc: Tvrtko Ursulin <tvrtko.ursulin@intel.com>
Cc: Chris Wilson <chris@chris-wilson.co.uk>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Link: http://patchwork.freedesktop.org/patch/msgid/20170214152901.20361-1-tvrtko.ursulin@linux.intel.com
drivers/gpu/drm/i915/i915_gem_context.c

index 9037356536dcd28b086eea6f77716f12a9f8c056..26245205556396c54a19c60f394db6956765f916 100644 (file)
@@ -604,7 +604,7 @@ mi_set_context(struct drm_i915_gem_request *req, u32 hw_flags)
                i915.semaphores ?
                INTEL_INFO(dev_priv)->num_rings - 1 :
                0;
-       int len, ret;
+       int len;
 
        /* w/a: If Flush TLB Invalidation Mode is enabled, driver must do a TLB
         * invalidation prior to MI_SET_CONTEXT. On GEN6 we don't set the value
@@ -612,7 +612,7 @@ mi_set_context(struct drm_i915_gem_request *req, u32 hw_flags)
         * itlb_before_ctx_switch.
         */
        if (IS_GEN6(dev_priv)) {
-               ret = engine->emit_flush(req, EMIT_INVALIDATE);
+               int ret = engine->emit_flush(req, EMIT_INVALIDATE);
                if (ret)
                        return ret;
        }
@@ -687,7 +687,7 @@ mi_set_context(struct drm_i915_gem_request *req, u32 hw_flags)
 
        intel_ring_advance(req, cs);
 
-       return ret;
+       return 0;
 }
 
 static int remap_l3(struct drm_i915_gem_request *req, int slice)