drm/i915/uc: Move fw path check to fetch_uc_fw()
authorMichal Wajdeczko <michal.wajdeczko@intel.com>
Thu, 30 Mar 2017 11:21:15 +0000 (11:21 +0000)
committerJoonas Lahtinen <joonas.lahtinen@linux.intel.com>
Fri, 31 Mar 2017 07:39:40 +0000 (10:39 +0300)
There is no reason to separately check for valid fw path before
we try to fetch it. Let the fetch function take care of this.

Signed-off-by: Michal Wajdeczko <michal.wajdeczko@intel.com>
Cc: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
Cc: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Reviewed-by: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
Signed-off-by: Joonas Lahtinen <joonas.lahtinen@linux.intel.com>
Link: http://patchwork.freedesktop.org/patch/msgid/20170330112115.120240-2-michal.wajdeczko@intel.com
drivers/gpu/drm/i915/intel_uc.c

index 2540bfbfccf77dacc29aa48b9fda0c76b39d59a0..c117424f1f50e1a4bde7d5467fb0a4d31ac9d6e8 100644 (file)
@@ -112,6 +112,9 @@ static void fetch_uc_fw(struct drm_i915_private *dev_priv,
        size_t size;
        int err;
 
+       if (!uc_fw->path)
+               return;
+
        uc_fw->fetch_status = INTEL_UC_FIRMWARE_PENDING;
 
        DRM_DEBUG_DRIVER("before requesting firmware: uC fw fetch status %s\n",
@@ -239,11 +242,8 @@ fail:
 
 void intel_uc_init_fw(struct drm_i915_private *dev_priv)
 {
-       if (dev_priv->huc.fw.path)
-               fetch_uc_fw(dev_priv, &dev_priv->huc.fw);
-
-       if (dev_priv->guc.fw.path)
-               fetch_uc_fw(dev_priv, &dev_priv->guc.fw);
+       fetch_uc_fw(dev_priv, &dev_priv->huc.fw);
+       fetch_uc_fw(dev_priv, &dev_priv->guc.fw);
 }
 
 void intel_uc_fini_fw(struct drm_i915_private *dev_priv)