Commit 97dee74b authored by Michal Wajdeczko's avatar Michal Wajdeczko Committed by Chris Wilson

drm/i915/uc: Reorder params in intel_uc_fw_fetch

All intel_uc_fw_* functions are taking uc_fw as first param
except intel_uc_fw_fetch() which is taking i915. Fix that.
Signed-off-by: default avatarMichal Wajdeczko <michal.wajdeczko@intel.com>
Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com>
Reviewed-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: default avatarChris Wilson <chris@chris-wilson.co.uk>
Link: https://patchwork.freedesktop.org/patch/msgid/20190725210314.21188-1-michal.wajdeczko@intel.com
parent 79c7a28e
...@@ -355,10 +355,10 @@ void intel_uc_fetch_firmwares(struct intel_uc *uc) ...@@ -355,10 +355,10 @@ void intel_uc_fetch_firmwares(struct intel_uc *uc)
if (!intel_uc_is_using_guc(uc)) if (!intel_uc_is_using_guc(uc))
return; return;
intel_uc_fw_fetch(i915, &uc->guc.fw); intel_uc_fw_fetch(&uc->guc.fw, i915);
if (intel_uc_is_using_huc(uc)) if (intel_uc_is_using_huc(uc))
intel_uc_fw_fetch(i915, &uc->huc.fw); intel_uc_fw_fetch(&uc->huc.fw, i915);
} }
void intel_uc_cleanup_firmwares(struct intel_uc *uc) void intel_uc_cleanup_firmwares(struct intel_uc *uc)
......
...@@ -190,10 +190,8 @@ void intel_uc_fw_init_early(struct intel_uc_fw *uc_fw, ...@@ -190,10 +190,8 @@ void intel_uc_fw_init_early(struct intel_uc_fw *uc_fw,
* *
* Fetch uC firmware into GEM obj. * Fetch uC firmware into GEM obj.
*/ */
void intel_uc_fw_fetch(struct drm_i915_private *dev_priv, void intel_uc_fw_fetch(struct intel_uc_fw *uc_fw, struct drm_i915_private *i915)
struct intel_uc_fw *uc_fw)
{ {
struct pci_dev *pdev = dev_priv->drm.pdev;
struct drm_i915_gem_object *obj; struct drm_i915_gem_object *obj;
const struct firmware *fw = NULL; const struct firmware *fw = NULL;
struct uc_css_header *css; struct uc_css_header *css;
...@@ -202,7 +200,7 @@ void intel_uc_fw_fetch(struct drm_i915_private *dev_priv, ...@@ -202,7 +200,7 @@ void intel_uc_fw_fetch(struct drm_i915_private *dev_priv,
GEM_BUG_ON(!intel_uc_fw_supported(uc_fw)); GEM_BUG_ON(!intel_uc_fw_supported(uc_fw));
err = request_firmware(&fw, uc_fw->path, &pdev->dev); err = request_firmware(&fw, uc_fw->path, i915->drm.dev);
if (err) if (err)
goto fail; goto fail;
...@@ -295,8 +293,7 @@ void intel_uc_fw_fetch(struct drm_i915_private *dev_priv, ...@@ -295,8 +293,7 @@ void intel_uc_fw_fetch(struct drm_i915_private *dev_priv,
goto fail; goto fail;
} }
obj = i915_gem_object_create_shmem_from_data(dev_priv, obj = i915_gem_object_create_shmem_from_data(i915, fw->data, fw->size);
fw->data, fw->size);
if (IS_ERR(obj)) { if (IS_ERR(obj)) {
err = PTR_ERR(obj); err = PTR_ERR(obj);
DRM_DEBUG_DRIVER("%s fw object_create err=%d\n", DRM_DEBUG_DRIVER("%s fw object_create err=%d\n",
......
...@@ -169,8 +169,8 @@ static inline u32 intel_uc_fw_get_upload_size(struct intel_uc_fw *uc_fw) ...@@ -169,8 +169,8 @@ static inline u32 intel_uc_fw_get_upload_size(struct intel_uc_fw *uc_fw)
void intel_uc_fw_init_early(struct intel_uc_fw *uc_fw, void intel_uc_fw_init_early(struct intel_uc_fw *uc_fw,
enum intel_uc_fw_type type, enum intel_uc_fw_type type,
struct drm_i915_private *i915); struct drm_i915_private *i915);
void intel_uc_fw_fetch(struct drm_i915_private *i915, void intel_uc_fw_fetch(struct intel_uc_fw *uc_fw,
struct intel_uc_fw *uc_fw); struct drm_i915_private *i915);
void intel_uc_fw_cleanup_fetch(struct intel_uc_fw *uc_fw); void intel_uc_fw_cleanup_fetch(struct intel_uc_fw *uc_fw);
int intel_uc_fw_upload(struct intel_uc_fw *uc_fw, struct intel_gt *gt, int intel_uc_fw_upload(struct intel_uc_fw *uc_fw, struct intel_gt *gt,
u32 wopcm_offset, u32 dma_flags); u32 wopcm_offset, u32 dma_flags);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment