drm/i915: Fix W=1 warning for csr_load_work_fn()
authorChris Wilson <chris@chris-wilson.co.uk>
Wed, 18 Jan 2017 12:18:08 +0000 (12:18 +0000)
committerChris Wilson <chris@chris-wilson.co.uk>
Mon, 23 Jan 2017 09:15:29 +0000 (09:15 +0000)
drivers/gpu/drm/i915/intel_csr.c: In function ‘csr_load_work_fn’:
drivers/gpu/drm/i915/intel_csr.c:399:6: error: variable ‘ret’ set but not used [-Werror=unused-but-set-variable]

Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Link: http://patchwork.freedesktop.org/patch/msgid/20170118121808.27869-2-chris@chris-wilson.co.uk
Reviewed-by: Mika Kuoppala <mika.kuoppala@intel.com>
drivers/gpu/drm/i915/intel_csr.c

index 0085bc745f6aa5256cf21c1ae76dd2cbce80e387..9dcc434d3b744574ab21e6984dc6cc97a5e206b6 100644 (file)
@@ -396,13 +396,11 @@ static void csr_load_work_fn(struct work_struct *work)
        struct drm_i915_private *dev_priv;
        struct intel_csr *csr;
        const struct firmware *fw = NULL;
-       int ret;
 
        dev_priv = container_of(work, typeof(*dev_priv), csr.work);
        csr = &dev_priv->csr;
 
-       ret = request_firmware(&fw, dev_priv->csr.fw_path,
-                              &dev_priv->drm.pdev->dev);
+       request_firmware(&fw, dev_priv->csr.fw_path, &dev_priv->drm.pdev->dev);
        if (fw)
                dev_priv->csr.dmc_payload = parse_csr_fw(dev_priv, fw);