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: Michal Wajdeczko <michal.wajdeczko@intel.com> Cc: Daniele Ceraolo Spurio <daniele.ceraolospurio@intel.com> Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk> Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk> Link: https://patchwork.freedesktop.org/patch/msgid/20190725210314.21188-1-michal.wajdeczko@intel.com
This commit is contained in:
parent
79c7a28e1f
commit
97dee74bb3
@ -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,
|
|||||||
*
|
*
|
||||||
* 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,
|
|||||||
|
|
||||||
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,
|
|||||||
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)
|
|||||||
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);
|
||||||
|
Loading…
Reference in New Issue
Block a user