diff mbox

[v4,1/6] drm/i915/guc: Make the GuC fw loading helper functions general

Message ID 1470298617-25848-2-git-send-email-peter.antoine@intel.com (mailing list archive)
State New, archived
Headers show

Commit Message

Peter Antoine Aug. 4, 2016, 8:16 a.m. UTC
Rename some of the GuC fw loading code to make them more general. We
will utilise them for HuC loading as well.
     s/intel_guc_fw/intel_uc_fw/g
     s/GUC_FIRMWARE/UC_FIRMWARE/g

Struct intel_guc_fw is renamed to intel_uc_fw. Prefix of tts members,
such as 'guc' or 'guc_fw' either is renamed to 'uc' or removed for
same purpose.

v2: rebased on top of nightly.
    reapplied the search/replace as upstream code as changed.
v3: rebased again on drm-nightly.
v4: removed G from messages in shared fw fetch function.

Signed-off-by: Alex Dai <yu.dai@intel.com>
Signed-off-by: Peter Antoine <peter.antoine@intel.com>
Reviewed-by: Dave Gordon <david.s.gordon@intel.com>
---
 drivers/gpu/drm/i915/i915_debugfs.c        |  12 +--
 drivers/gpu/drm/i915/i915_guc_submission.c |   4 +-
 drivers/gpu/drm/i915/intel_guc.h           |  39 +++----
 drivers/gpu/drm/i915/intel_guc_loader.c    | 160 ++++++++++++++---------------
 4 files changed, 108 insertions(+), 107 deletions(-)

Comments

Dave Gordon Aug. 11, 2016, 10:56 a.m. UTC | #1
On 04/08/16 09:16, Peter Antoine wrote:
> Rename some of the GuC fw loading code to make them more general. We
> will utilise them for HuC loading as well.
>      s/intel_guc_fw/intel_uc_fw/g
>      s/GUC_FIRMWARE/UC_FIRMWARE/g
>
> Struct intel_guc_fw is renamed to intel_uc_fw. Prefix of tts members,
> such as 'guc' or 'guc_fw' either is renamed to 'uc' or removed for
> same purpose.
>
> v2: rebased on top of nightly.
>     reapplied the search/replace as upstream code as changed.
> v3: rebased again on drm-nightly.
> v4: removed G from messages in shared fw fetch function.
>
> Signed-off-by: Alex Dai <yu.dai@intel.com>
> Signed-off-by: Peter Antoine <peter.antoine@intel.com>
> Reviewed-by: Dave Gordon <david.s.gordon@intel.com>

R-b can carry over again, but this will need (ANOTHER!) rebase as Chris 
has nuked one of the functions called below.

> ---
>  drivers/gpu/drm/i915/i915_debugfs.c        |  12 +--
>  drivers/gpu/drm/i915/i915_guc_submission.c |   4 +-
>  drivers/gpu/drm/i915/intel_guc.h           |  39 +++----
>  drivers/gpu/drm/i915/intel_guc_loader.c    | 160 ++++++++++++++---------------
>  4 files changed, 108 insertions(+), 107 deletions(-)
>
> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
> index 531ca02..5df7bd3 100644
> --- a/drivers/gpu/drm/i915/i915_debugfs.c
> +++ b/drivers/gpu/drm/i915/i915_debugfs.c
> @@ -2490,7 +2490,7 @@ static int i915_guc_load_status_info(struct seq_file *m, void *data)
>  {
>  	struct drm_info_node *node = m->private;
>  	struct drm_i915_private *dev_priv = to_i915(node->minor->dev);
> -	struct intel_guc_fw *guc_fw = &dev_priv->guc.guc_fw;
> +	struct intel_uc_fw *guc_fw = &dev_priv->guc.guc_fw;
>  	u32 tmp, i;
>
>  	if (!HAS_GUC_UCODE(dev_priv))
> @@ -2498,15 +2498,15 @@ static int i915_guc_load_status_info(struct seq_file *m, void *data)
>
>  	seq_printf(m, "GuC firmware status:\n");
>  	seq_printf(m, "\tpath: %s\n",
> -		guc_fw->guc_fw_path);
> +		guc_fw->uc_fw_path);
>  	seq_printf(m, "\tfetch: %s\n",
> -		intel_guc_fw_status_repr(guc_fw->guc_fw_fetch_status));
> +		intel_uc_fw_status_repr(guc_fw->fetch_status));
>  	seq_printf(m, "\tload: %s\n",
> -		intel_guc_fw_status_repr(guc_fw->guc_fw_load_status));
> +		intel_uc_fw_status_repr(guc_fw->load_status));
>  	seq_printf(m, "\tversion wanted: %d.%d\n",
> -		guc_fw->guc_fw_major_wanted, guc_fw->guc_fw_minor_wanted);
> +		guc_fw->major_ver_wanted, guc_fw->minor_ver_wanted);
>  	seq_printf(m, "\tversion found: %d.%d\n",
> -		guc_fw->guc_fw_major_found, guc_fw->guc_fw_minor_found);
> +		guc_fw->major_ver_found, guc_fw->minor_ver_found);
>  	seq_printf(m, "\theader: offset is %d; size = %d\n",
>  		guc_fw->header_offset, guc_fw->header_size);
>  	seq_printf(m, "\tuCode: offset is %d; size = %d\n",
> diff --git a/drivers/gpu/drm/i915/i915_guc_submission.c b/drivers/gpu/drm/i915/i915_guc_submission.c
> index 01c1c16..f665a87 100644
> --- a/drivers/gpu/drm/i915/i915_guc_submission.c
> +++ b/drivers/gpu/drm/i915/i915_guc_submission.c
> @@ -1044,7 +1044,7 @@ int intel_guc_suspend(struct drm_device *dev)
>  	struct i915_gem_context *ctx;
>  	u32 data[3];
>
> -	if (guc->guc_fw.guc_fw_load_status != GUC_FIRMWARE_SUCCESS)
> +	if (guc->guc_fw.load_status != UC_FIRMWARE_SUCCESS)
>  		return 0;
>
>  	ctx = dev_priv->kernel_context;
> @@ -1070,7 +1070,7 @@ int intel_guc_resume(struct drm_device *dev)
>  	struct i915_gem_context *ctx;
>  	u32 data[3];
>
> -	if (guc->guc_fw.guc_fw_load_status != GUC_FIRMWARE_SUCCESS)
> +	if (guc->guc_fw.load_status != UC_FIRMWARE_SUCCESS)
>  		return 0;
>
>  	ctx = dev_priv->kernel_context;
> diff --git a/drivers/gpu/drm/i915/intel_guc.h b/drivers/gpu/drm/i915/intel_guc.h
> index 3e3e743..02adcfc 100644
> --- a/drivers/gpu/drm/i915/intel_guc.h
> +++ b/drivers/gpu/drm/i915/intel_guc.h
> @@ -90,29 +90,29 @@ struct i915_guc_client {
>  	uint64_t submissions[I915_NUM_ENGINES];
>  };
>
> -enum intel_guc_fw_status {
> -	GUC_FIRMWARE_FAIL = -1,
> -	GUC_FIRMWARE_NONE = 0,
> -	GUC_FIRMWARE_PENDING,
> -	GUC_FIRMWARE_SUCCESS
> +enum intel_uc_fw_status {
> +	UC_FIRMWARE_FAIL = -1,
> +	UC_FIRMWARE_NONE = 0,
> +	UC_FIRMWARE_PENDING,
> +	UC_FIRMWARE_SUCCESS
>  };
>
>  /*
>   * This structure encapsulates all the data needed during the process
>   * of fetching, caching, and loading the firmware image into the GuC.
>   */
> -struct intel_guc_fw {
> -	struct drm_device *		guc_dev;
> -	const char *			guc_fw_path;
> -	size_t				guc_fw_size;
> -	struct drm_i915_gem_object *	guc_fw_obj;
> -	enum intel_guc_fw_status	guc_fw_fetch_status;
> -	enum intel_guc_fw_status	guc_fw_load_status;
> -
> -	uint16_t			guc_fw_major_wanted;
> -	uint16_t			guc_fw_minor_wanted;
> -	uint16_t			guc_fw_major_found;
> -	uint16_t			guc_fw_minor_found;
> +struct intel_uc_fw {
> +	struct drm_device *uc_dev;
> +	const char *uc_fw_path;
> +	size_t uc_fw_size;
> +	struct drm_i915_gem_object *uc_fw_obj;
> +	enum intel_uc_fw_status fetch_status;
> +	enum intel_uc_fw_status load_status;
> +
> +	uint16_t major_ver_wanted;
> +	uint16_t minor_ver_wanted;
> +	uint16_t major_ver_found;
> +	uint16_t minor_ver_found;
>
>  	uint32_t header_size;
>  	uint32_t header_offset;
> @@ -123,7 +123,7 @@ struct intel_guc_fw {
>  };
>
>  struct intel_guc {
> -	struct intel_guc_fw guc_fw;
> +	struct intel_uc_fw guc_fw;
>  	uint32_t log_flags;
>  	struct drm_i915_gem_object *log_obj;
>
> @@ -152,9 +152,10 @@ struct intel_guc {
>  extern void intel_guc_init(struct drm_device *dev);
>  extern int intel_guc_setup(struct drm_device *dev);
>  extern void intel_guc_fini(struct drm_device *dev);
> -extern const char *intel_guc_fw_status_repr(enum intel_guc_fw_status status);
> +extern const char *intel_uc_fw_status_repr(enum intel_uc_fw_status status);
>  extern int intel_guc_suspend(struct drm_device *dev);
>  extern int intel_guc_resume(struct drm_device *dev);
> +void intel_uc_fw_fetch(struct drm_device *dev, struct intel_uc_fw *uc_fw);
>
>  /* i915_guc_submission.c */
>  int i915_guc_submission_init(struct drm_i915_private *dev_priv);
> diff --git a/drivers/gpu/drm/i915/intel_guc_loader.c b/drivers/gpu/drm/i915/intel_guc_loader.c
> index b883efd..25c9a91 100644
> --- a/drivers/gpu/drm/i915/intel_guc_loader.c
> +++ b/drivers/gpu/drm/i915/intel_guc_loader.c
> @@ -69,16 +69,16 @@ MODULE_FIRMWARE(I915_BXT_GUC_UCODE);
>  MODULE_FIRMWARE(I915_KBL_GUC_UCODE);
>
>  /* User-friendly representation of an enum */
> -const char *intel_guc_fw_status_repr(enum intel_guc_fw_status status)
> +const char *intel_uc_fw_status_repr(enum intel_uc_fw_status status)
>  {
>  	switch (status) {
> -	case GUC_FIRMWARE_FAIL:
> +	case UC_FIRMWARE_FAIL:
>  		return "FAIL";
> -	case GUC_FIRMWARE_NONE:
> +	case UC_FIRMWARE_NONE:
>  		return "NONE";
> -	case GUC_FIRMWARE_PENDING:
> +	case UC_FIRMWARE_PENDING:
>  		return "PENDING";
> -	case GUC_FIRMWARE_SUCCESS:
> +	case UC_FIRMWARE_SUCCESS:
>  		return "SUCCESS";
>  	default:
>  		return "UNKNOWN!";
> @@ -240,8 +240,8 @@ static inline bool guc_ucode_response(struct drm_i915_private *dev_priv,
>   */
>  static int guc_ucode_xfer_dma(struct drm_i915_private *dev_priv)
>  {
> -	struct intel_guc_fw *guc_fw = &dev_priv->guc.guc_fw;
> -	struct drm_i915_gem_object *fw_obj = guc_fw->guc_fw_obj;
> +	struct intel_uc_fw *guc_fw = &dev_priv->guc.guc_fw;
> +	struct drm_i915_gem_object *fw_obj = guc_fw->uc_fw_obj;
>  	unsigned long offset;
>  	struct sg_table *sg = fw_obj->pages;
>  	u32 status, rsa[UOS_RSA_SCRATCH_MAX_COUNT];
> @@ -313,17 +313,17 @@ static u32 guc_wopcm_size(struct drm_i915_private *dev_priv)
>   */
>  static int guc_ucode_xfer(struct drm_i915_private *dev_priv)
>  {
> -	struct intel_guc_fw *guc_fw = &dev_priv->guc.guc_fw;
> +	struct intel_uc_fw *guc_fw = &dev_priv->guc.guc_fw;
>  	struct drm_device *dev = &dev_priv->drm;
>  	int ret;
>
> -	ret = i915_gem_object_set_to_gtt_domain(guc_fw->guc_fw_obj, false);
> +	ret = i915_gem_object_set_to_gtt_domain(guc_fw->uc_fw_obj, false);
>  	if (ret) {
>  		DRM_DEBUG_DRIVER("set-domain failed %d\n", ret);
>  		return ret;
>  	}
>
> -	ret = i915_gem_obj_ggtt_pin(guc_fw->guc_fw_obj, 0, 0);
> +	ret = i915_gem_obj_ggtt_pin(guc_fw->uc_fw_obj, 0, 0);

This now has to be:

+	ret = i915_gem_object_ggtt_pin(guc_fw->uc_fw_obj, NULL, 0, 0, 0);

Otherwise OK

.Dave.

>  	if (ret) {
>  		DRM_DEBUG_DRIVER("pin failed %d\n", ret);
>  		return ret;
> @@ -377,7 +377,7 @@ static int guc_ucode_xfer(struct drm_i915_private *dev_priv)
>  	 * We keep the object pages for reuse during resume. But we can unpin it
>  	 * now that DMA has completed, so it doesn't continue to take up space.
>  	 */
> -	i915_gem_object_ggtt_unpin(guc_fw->guc_fw_obj);
> +	i915_gem_object_ggtt_unpin(guc_fw->uc_fw_obj);
>
>  	return ret;
>  }
> @@ -416,14 +416,14 @@ static int i915_reset_guc(struct drm_i915_private *dev_priv)
>  int intel_guc_setup(struct drm_device *dev)
>  {
>  	struct drm_i915_private *dev_priv = to_i915(dev);
> -	struct intel_guc_fw *guc_fw = &dev_priv->guc.guc_fw;
> -	const char *fw_path = guc_fw->guc_fw_path;
> +	struct intel_uc_fw *guc_fw = &dev_priv->guc.guc_fw;
> +	const char *fw_path = guc_fw->uc_fw_path;
>  	int retries, ret, err;
>
>  	DRM_DEBUG_DRIVER("GuC fw status: path %s, fetch %s, load %s\n",
>  		fw_path,
> -		intel_guc_fw_status_repr(guc_fw->guc_fw_fetch_status),
> -		intel_guc_fw_status_repr(guc_fw->guc_fw_load_status));
> +		intel_uc_fw_status_repr(guc_fw->fetch_status),
> +		intel_uc_fw_status_repr(guc_fw->load_status));
>
>  	/* Loading forbidden, or no firmware to load? */
>  	if (!i915.enable_guc_loading) {
> @@ -441,21 +441,21 @@ int intel_guc_setup(struct drm_device *dev)
>  	}
>
>  	/* Fetch failed, or already fetched but failed to load? */
> -	if (guc_fw->guc_fw_fetch_status != GUC_FIRMWARE_SUCCESS) {
> +	if (guc_fw->fetch_status != UC_FIRMWARE_SUCCESS) {
>  		err = -EIO;
>  		goto fail;
> -	} else if (guc_fw->guc_fw_load_status == GUC_FIRMWARE_FAIL) {
> +	} else if (guc_fw->load_status == UC_FIRMWARE_FAIL) {
>  		err = -ENOEXEC;
>  		goto fail;
>  	}
>
>  	direct_interrupts_to_host(dev_priv);
>
> -	guc_fw->guc_fw_load_status = GUC_FIRMWARE_PENDING;
> +	guc_fw->load_status = UC_FIRMWARE_PENDING;
>
>  	DRM_DEBUG_DRIVER("GuC fw status: fetch %s, load %s\n",
> -		intel_guc_fw_status_repr(guc_fw->guc_fw_fetch_status),
> -		intel_guc_fw_status_repr(guc_fw->guc_fw_load_status));
> +		intel_uc_fw_status_repr(guc_fw->fetch_status),
> +		intel_uc_fw_status_repr(guc_fw->load_status));
>
>  	err = i915_guc_submission_init(dev_priv);
>  	if (err)
> @@ -489,11 +489,11 @@ int intel_guc_setup(struct drm_device *dev)
>  			 "retry %d more time(s)\n", err, retries);
>  	}
>
> -	guc_fw->guc_fw_load_status = GUC_FIRMWARE_SUCCESS;
> +	guc_fw->load_status = UC_FIRMWARE_SUCCESS;
>
>  	DRM_DEBUG_DRIVER("GuC fw status: fetch %s, load %s\n",
> -		intel_guc_fw_status_repr(guc_fw->guc_fw_fetch_status),
> -		intel_guc_fw_status_repr(guc_fw->guc_fw_load_status));
> +		intel_uc_fw_status_repr(guc_fw->fetch_status),
> +		intel_uc_fw_status_repr(guc_fw->load_status));
>
>  	if (i915.enable_guc_submission) {
>  		err = i915_guc_submission_enable(dev_priv);
> @@ -505,8 +505,8 @@ int intel_guc_setup(struct drm_device *dev)
>  	return 0;
>
>  fail:
> -	if (guc_fw->guc_fw_load_status == GUC_FIRMWARE_PENDING)
> -		guc_fw->guc_fw_load_status = GUC_FIRMWARE_FAIL;
> +	if (guc_fw->load_status == UC_FIRMWARE_PENDING)
> +		guc_fw->load_status = UC_FIRMWARE_FAIL;
>
>  	direct_interrupts_to_host(dev_priv);
>  	i915_guc_submission_disable(dev_priv);
> @@ -551,7 +551,7 @@ fail:
>  	return ret;
>  }
>
> -static void guc_fw_fetch(struct drm_device *dev, struct intel_guc_fw *guc_fw)
> +void intel_uc_fw_fetch(struct drm_device *dev, struct intel_uc_fw *uc_fw)
>  {
>  	struct drm_i915_gem_object *obj;
>  	const struct firmware *fw;
> @@ -559,17 +559,17 @@ static void guc_fw_fetch(struct drm_device *dev, struct intel_guc_fw *guc_fw)
>  	size_t size;
>  	int err;
>
> -	DRM_DEBUG_DRIVER("before requesting firmware: GuC fw fetch status %s\n",
> -		intel_guc_fw_status_repr(guc_fw->guc_fw_fetch_status));
> +	DRM_DEBUG_DRIVER("before requesting firmware: uC fw fetch status %s\n",
> +		intel_uc_fw_status_repr(uc_fw->fetch_status));
>
> -	err = request_firmware(&fw, guc_fw->guc_fw_path, &dev->pdev->dev);
> +	err = request_firmware(&fw, uc_fw->uc_fw_path, &dev->pdev->dev);
>  	if (err)
>  		goto fail;
>  	if (!fw)
>  		goto fail;
>
> -	DRM_DEBUG_DRIVER("fetch GuC fw from %s succeeded, fw %p\n",
> -		guc_fw->guc_fw_path, fw);
> +	DRM_DEBUG_DRIVER("fetch uC fw from %s succeeded, fw %p\n",
> +		uc_fw->uc_fw_path, fw);
>
>  	/* Check the size of the blob before examining buffer contents */
>  	if (fw->size < sizeof(struct guc_css_header)) {
> @@ -580,36 +580,36 @@ static void guc_fw_fetch(struct drm_device *dev, struct intel_guc_fw *guc_fw)
>  	css = (struct guc_css_header *)fw->data;
>
>  	/* Firmware bits always start from header */
> -	guc_fw->header_offset = 0;
> -	guc_fw->header_size = (css->header_size_dw - css->modulus_size_dw -
> +	uc_fw->header_offset = 0;
> +	uc_fw->header_size = (css->header_size_dw - css->modulus_size_dw -
>  		css->key_size_dw - css->exponent_size_dw) * sizeof(u32);
>
> -	if (guc_fw->header_size != sizeof(struct guc_css_header)) {
> +	if (uc_fw->header_size != sizeof(struct guc_css_header)) {
>  		DRM_ERROR("CSS header definition mismatch\n");
>  		goto fail;
>  	}
>
>  	/* then, uCode */
> -	guc_fw->ucode_offset = guc_fw->header_offset + guc_fw->header_size;
> -	guc_fw->ucode_size = (css->size_dw - css->header_size_dw) * sizeof(u32);
> +	uc_fw->ucode_offset = uc_fw->header_offset + uc_fw->header_size;
> +	uc_fw->ucode_size = (css->size_dw - css->header_size_dw) * sizeof(u32);
>
>  	/* now RSA */
>  	if (css->key_size_dw != UOS_RSA_SCRATCH_MAX_COUNT) {
>  		DRM_ERROR("RSA key size is bad\n");
>  		goto fail;
>  	}
> -	guc_fw->rsa_offset = guc_fw->ucode_offset + guc_fw->ucode_size;
> -	guc_fw->rsa_size = css->key_size_dw * sizeof(u32);
> +	uc_fw->rsa_offset = uc_fw->ucode_offset + uc_fw->ucode_size;
> +	uc_fw->rsa_size = css->key_size_dw * sizeof(u32);
>
>  	/* At least, it should have header, uCode and RSA. Size of all three. */
> -	size = guc_fw->header_size + guc_fw->ucode_size + guc_fw->rsa_size;
> +	size = uc_fw->header_size + uc_fw->ucode_size + uc_fw->rsa_size;
>  	if (fw->size < size) {
>  		DRM_ERROR("Missing firmware components\n");
>  		goto fail;
>  	}
>
>  	/* Header and uCode will be loaded to WOPCM. Size of the two. */
> -	size = guc_fw->header_size + guc_fw->ucode_size;
> +	size = uc_fw->header_size + uc_fw->ucode_size;
>  	if (size > guc_wopcm_size(to_i915(dev))) {
>  		DRM_ERROR("Firmware is too large to fit in WOPCM\n");
>  		goto fail;
> @@ -621,21 +621,21 @@ static void guc_fw_fetch(struct drm_device *dev, struct intel_guc_fw *guc_fw)
>  	 * TWO bytes each (i.e. u16), although all pointers and offsets are defined
>  	 * in terms of bytes (u8).
>  	 */
> -	guc_fw->guc_fw_major_found = css->guc_sw_version >> 16;
> -	guc_fw->guc_fw_minor_found = css->guc_sw_version & 0xFFFF;
> -
> -	if (guc_fw->guc_fw_major_found != guc_fw->guc_fw_major_wanted ||
> -	    guc_fw->guc_fw_minor_found < guc_fw->guc_fw_minor_wanted) {
> -		DRM_ERROR("GuC firmware version %d.%d, required %d.%d\n",
> -			guc_fw->guc_fw_major_found, guc_fw->guc_fw_minor_found,
> -			guc_fw->guc_fw_major_wanted, guc_fw->guc_fw_minor_wanted);
> +	uc_fw->major_ver_found = css->guc_sw_version >> 16;
> +	uc_fw->minor_ver_found = css->guc_sw_version & 0xFFFF;
> +
> +	if (uc_fw->major_ver_found != uc_fw->major_ver_wanted ||
> +	    uc_fw->minor_ver_found < uc_fw->minor_ver_wanted) {
> +		DRM_ERROR("uC firmware version %d.%d, required %d.%d\n",
> +			uc_fw->major_ver_found, uc_fw->minor_ver_found,
> +			uc_fw->major_ver_wanted, uc_fw->minor_ver_wanted);
>  		err = -ENOEXEC;
>  		goto fail;
>  	}
>
>  	DRM_DEBUG_DRIVER("firmware version %d.%d OK (minimum %d.%d)\n",
> -			guc_fw->guc_fw_major_found, guc_fw->guc_fw_minor_found,
> -			guc_fw->guc_fw_major_wanted, guc_fw->guc_fw_minor_wanted);
> +			uc_fw->major_ver_found, uc_fw->minor_ver_found,
> +			uc_fw->major_ver_wanted, uc_fw->minor_ver_wanted);
>
>  	mutex_lock(&dev->struct_mutex);
>  	obj = i915_gem_object_create_from_data(dev, fw->data, fw->size);
> @@ -645,31 +645,31 @@ static void guc_fw_fetch(struct drm_device *dev, struct intel_guc_fw *guc_fw)
>  		goto fail;
>  	}
>
> -	guc_fw->guc_fw_obj = obj;
> -	guc_fw->guc_fw_size = fw->size;
> +	uc_fw->uc_fw_obj = obj;
> +	uc_fw->uc_fw_size = fw->size;
>
> -	DRM_DEBUG_DRIVER("GuC fw fetch status SUCCESS, obj %p\n",
> -			guc_fw->guc_fw_obj);
> +	DRM_DEBUG_DRIVER("uC fw fetch status SUCCESS, obj %p\n",
> +			uc_fw->uc_fw_obj);
>
>  	release_firmware(fw);
> -	guc_fw->guc_fw_fetch_status = GUC_FIRMWARE_SUCCESS;
> +	uc_fw->fetch_status = UC_FIRMWARE_SUCCESS;
>  	return;
>
>  fail:
> -	DRM_DEBUG_DRIVER("GuC fw fetch status FAIL; err %d, fw %p, obj %p\n",
> -		err, fw, guc_fw->guc_fw_obj);
> -	DRM_ERROR("Failed to fetch GuC firmware from %s (error %d)\n",
> -		  guc_fw->guc_fw_path, err);
> +	DRM_DEBUG_DRIVER("uC fw fetch status FAIL; err %d, fw %p, obj %p\n",
> +		err, fw, uc_fw->uc_fw_obj);
> +	DRM_ERROR("Failed to fetch uC firmware from %s (error %d)\n",
> +		  uc_fw->uc_fw_path, err);
>
>  	mutex_lock(&dev->struct_mutex);
> -	obj = guc_fw->guc_fw_obj;
> +	obj = uc_fw->uc_fw_obj;
>  	if (obj)
>  		i915_gem_object_put(obj);
> -	guc_fw->guc_fw_obj = NULL;
> +	uc_fw->uc_fw_obj = NULL;
>  	mutex_unlock(&dev->struct_mutex);
>
>  	release_firmware(fw);		/* OK even if fw is NULL */
> -	guc_fw->guc_fw_fetch_status = GUC_FIRMWARE_FAIL;
> +	uc_fw->fetch_status = UC_FIRMWARE_FAIL;
>  }
>
>  /**
> @@ -684,7 +684,7 @@ fail:
>  void intel_guc_init(struct drm_device *dev)
>  {
>  	struct drm_i915_private *dev_priv = to_i915(dev);
> -	struct intel_guc_fw *guc_fw = &dev_priv->guc.guc_fw;
> +	struct intel_uc_fw *guc_fw = &dev_priv->guc.guc_fw;
>  	const char *fw_path;
>
>  	/* A negative value means "use platform default" */
> @@ -697,24 +697,24 @@ void intel_guc_init(struct drm_device *dev)
>  		fw_path = NULL;
>  	} else if (IS_SKYLAKE(dev)) {
>  		fw_path = I915_SKL_GUC_UCODE;
> -		guc_fw->guc_fw_major_wanted = 6;
> -		guc_fw->guc_fw_minor_wanted = 1;
> +		guc_fw->major_ver_wanted = 6;
> +		guc_fw->minor_ver_wanted = 1;
>  	} else if (IS_BROXTON(dev)) {
>  		fw_path = I915_BXT_GUC_UCODE;
> -		guc_fw->guc_fw_major_wanted = 8;
> -		guc_fw->guc_fw_minor_wanted = 7;
> +		guc_fw->major_ver_wanted = 8;
> +		guc_fw->minor_ver_wanted = 7;
>  	} else if (IS_KABYLAKE(dev)) {
>  		fw_path = I915_KBL_GUC_UCODE;
> -		guc_fw->guc_fw_major_wanted = 9;
> -		guc_fw->guc_fw_minor_wanted = 14;
> +		guc_fw->major_ver_wanted = 9;
> +		guc_fw->minor_ver_wanted = 14;
>  	} else {
>  		fw_path = "";	/* unknown device */
>  	}
>
> -	guc_fw->guc_dev = dev;
> -	guc_fw->guc_fw_path = fw_path;
> -	guc_fw->guc_fw_fetch_status = GUC_FIRMWARE_NONE;
> -	guc_fw->guc_fw_load_status = GUC_FIRMWARE_NONE;
> +	guc_fw->uc_dev = dev;
> +	guc_fw->uc_fw_path = fw_path;
> +	guc_fw->fetch_status = UC_FIRMWARE_NONE;
> +	guc_fw->load_status = UC_FIRMWARE_NONE;
>
>  	/* Early (and silent) return if GuC loading is disabled */
>  	if (!i915.enable_guc_loading)
> @@ -724,9 +724,9 @@ void intel_guc_init(struct drm_device *dev)
>  	if (*fw_path == '\0')
>  		return;
>
> -	guc_fw->guc_fw_fetch_status = GUC_FIRMWARE_PENDING;
> +	guc_fw->fetch_status = UC_FIRMWARE_PENDING;
>  	DRM_DEBUG_DRIVER("GuC firmware pending, path %s\n", fw_path);
> -	guc_fw_fetch(dev, guc_fw);
> +	intel_uc_fw_fetch(dev, guc_fw);
>  	/* status must now be FAIL or SUCCESS */
>  }
>
> @@ -737,17 +737,17 @@ void intel_guc_init(struct drm_device *dev)
>  void intel_guc_fini(struct drm_device *dev)
>  {
>  	struct drm_i915_private *dev_priv = to_i915(dev);
> -	struct intel_guc_fw *guc_fw = &dev_priv->guc.guc_fw;
> +	struct intel_uc_fw *guc_fw = &dev_priv->guc.guc_fw;
>
>  	mutex_lock(&dev->struct_mutex);
>  	direct_interrupts_to_host(dev_priv);
>  	i915_guc_submission_disable(dev_priv);
>  	i915_guc_submission_fini(dev_priv);
>
> -	if (guc_fw->guc_fw_obj)
> -		i915_gem_object_put(guc_fw->guc_fw_obj);
> -	guc_fw->guc_fw_obj = NULL;
> +	if (guc_fw->uc_fw_obj)
> +		i915_gem_object_put(guc_fw->uc_fw_obj);
> +	guc_fw->uc_fw_obj = NULL;
>  	mutex_unlock(&dev->struct_mutex);
>
> -	guc_fw->guc_fw_fetch_status = GUC_FIRMWARE_NONE;
> +	guc_fw->fetch_status = UC_FIRMWARE_NONE;
>  }
>
Peter Antoine Aug. 11, 2016, 10:58 a.m. UTC | #2
Just rebasing now.
Is the other patch ok?

Peter.

-----Original Message-----
From: Gordon, David S 

Sent: Thursday, August 11, 2016 11:56 AM
To: Antoine, Peter <peter.antoine@intel.com>; intel-gfx@lists.freedesktop.org
Cc: Alex Dai <yu.dai@intel.com>
Subject: Re: [PATCH v4 1/6] drm/i915/guc: Make the GuC fw loading helper functions general

On 04/08/16 09:16, Peter Antoine wrote:
> Rename some of the GuC fw loading code to make them more general. We 

> will utilise them for HuC loading as well.

>      s/intel_guc_fw/intel_uc_fw/g

>      s/GUC_FIRMWARE/UC_FIRMWARE/g

>

> Struct intel_guc_fw is renamed to intel_uc_fw. Prefix of tts members, 

> such as 'guc' or 'guc_fw' either is renamed to 'uc' or removed for 

> same purpose.

>

> v2: rebased on top of nightly.

>     reapplied the search/replace as upstream code as changed.

> v3: rebased again on drm-nightly.

> v4: removed G from messages in shared fw fetch function.

>

> Signed-off-by: Alex Dai <yu.dai@intel.com>

> Signed-off-by: Peter Antoine <peter.antoine@intel.com>

> Reviewed-by: Dave Gordon <david.s.gordon@intel.com>


R-b can carry over again, but this will need (ANOTHER!) rebase as Chris has nuked one of the functions called below.

> ---

>  drivers/gpu/drm/i915/i915_debugfs.c        |  12 +--

>  drivers/gpu/drm/i915/i915_guc_submission.c |   4 +-

>  drivers/gpu/drm/i915/intel_guc.h           |  39 +++----

>  drivers/gpu/drm/i915/intel_guc_loader.c    | 160 ++++++++++++++---------------

>  4 files changed, 108 insertions(+), 107 deletions(-)

>

> diff --git a/drivers/gpu/drm/i915/i915_debugfs.c 

> b/drivers/gpu/drm/i915/i915_debugfs.c

> index 531ca02..5df7bd3 100644

> --- a/drivers/gpu/drm/i915/i915_debugfs.c

> +++ b/drivers/gpu/drm/i915/i915_debugfs.c

> @@ -2490,7 +2490,7 @@ static int i915_guc_load_status_info(struct 

> seq_file *m, void *data)  {

>  	struct drm_info_node *node = m->private;

>  	struct drm_i915_private *dev_priv = to_i915(node->minor->dev);

> -	struct intel_guc_fw *guc_fw = &dev_priv->guc.guc_fw;

> +	struct intel_uc_fw *guc_fw = &dev_priv->guc.guc_fw;

>  	u32 tmp, i;

>

>  	if (!HAS_GUC_UCODE(dev_priv))

> @@ -2498,15 +2498,15 @@ static int i915_guc_load_status_info(struct 

> seq_file *m, void *data)

>

>  	seq_printf(m, "GuC firmware status:\n");

>  	seq_printf(m, "\tpath: %s\n",

> -		guc_fw->guc_fw_path);

> +		guc_fw->uc_fw_path);

>  	seq_printf(m, "\tfetch: %s\n",

> -		intel_guc_fw_status_repr(guc_fw->guc_fw_fetch_status));

> +		intel_uc_fw_status_repr(guc_fw->fetch_status));

>  	seq_printf(m, "\tload: %s\n",

> -		intel_guc_fw_status_repr(guc_fw->guc_fw_load_status));

> +		intel_uc_fw_status_repr(guc_fw->load_status));

>  	seq_printf(m, "\tversion wanted: %d.%d\n",

> -		guc_fw->guc_fw_major_wanted, guc_fw->guc_fw_minor_wanted);

> +		guc_fw->major_ver_wanted, guc_fw->minor_ver_wanted);

>  	seq_printf(m, "\tversion found: %d.%d\n",

> -		guc_fw->guc_fw_major_found, guc_fw->guc_fw_minor_found);

> +		guc_fw->major_ver_found, guc_fw->minor_ver_found);

>  	seq_printf(m, "\theader: offset is %d; size = %d\n",

>  		guc_fw->header_offset, guc_fw->header_size);

>  	seq_printf(m, "\tuCode: offset is %d; size = %d\n", diff --git 

> a/drivers/gpu/drm/i915/i915_guc_submission.c 

> b/drivers/gpu/drm/i915/i915_guc_submission.c

> index 01c1c16..f665a87 100644

> --- a/drivers/gpu/drm/i915/i915_guc_submission.c

> +++ b/drivers/gpu/drm/i915/i915_guc_submission.c

> @@ -1044,7 +1044,7 @@ int intel_guc_suspend(struct drm_device *dev)

>  	struct i915_gem_context *ctx;

>  	u32 data[3];

>

> -	if (guc->guc_fw.guc_fw_load_status != GUC_FIRMWARE_SUCCESS)

> +	if (guc->guc_fw.load_status != UC_FIRMWARE_SUCCESS)

>  		return 0;

>

>  	ctx = dev_priv->kernel_context;

> @@ -1070,7 +1070,7 @@ int intel_guc_resume(struct drm_device *dev)

>  	struct i915_gem_context *ctx;

>  	u32 data[3];

>

> -	if (guc->guc_fw.guc_fw_load_status != GUC_FIRMWARE_SUCCESS)

> +	if (guc->guc_fw.load_status != UC_FIRMWARE_SUCCESS)

>  		return 0;

>

>  	ctx = dev_priv->kernel_context;

> diff --git a/drivers/gpu/drm/i915/intel_guc.h 

> b/drivers/gpu/drm/i915/intel_guc.h

> index 3e3e743..02adcfc 100644

> --- a/drivers/gpu/drm/i915/intel_guc.h

> +++ b/drivers/gpu/drm/i915/intel_guc.h

> @@ -90,29 +90,29 @@ struct i915_guc_client {

>  	uint64_t submissions[I915_NUM_ENGINES];  };

>

> -enum intel_guc_fw_status {

> -	GUC_FIRMWARE_FAIL = -1,

> -	GUC_FIRMWARE_NONE = 0,

> -	GUC_FIRMWARE_PENDING,

> -	GUC_FIRMWARE_SUCCESS

> +enum intel_uc_fw_status {

> +	UC_FIRMWARE_FAIL = -1,

> +	UC_FIRMWARE_NONE = 0,

> +	UC_FIRMWARE_PENDING,

> +	UC_FIRMWARE_SUCCESS

>  };

>

>  /*

>   * This structure encapsulates all the data needed during the process

>   * of fetching, caching, and loading the firmware image into the GuC.

>   */

> -struct intel_guc_fw {

> -	struct drm_device *		guc_dev;

> -	const char *			guc_fw_path;

> -	size_t				guc_fw_size;

> -	struct drm_i915_gem_object *	guc_fw_obj;

> -	enum intel_guc_fw_status	guc_fw_fetch_status;

> -	enum intel_guc_fw_status	guc_fw_load_status;

> -

> -	uint16_t			guc_fw_major_wanted;

> -	uint16_t			guc_fw_minor_wanted;

> -	uint16_t			guc_fw_major_found;

> -	uint16_t			guc_fw_minor_found;

> +struct intel_uc_fw {

> +	struct drm_device *uc_dev;

> +	const char *uc_fw_path;

> +	size_t uc_fw_size;

> +	struct drm_i915_gem_object *uc_fw_obj;

> +	enum intel_uc_fw_status fetch_status;

> +	enum intel_uc_fw_status load_status;

> +

> +	uint16_t major_ver_wanted;

> +	uint16_t minor_ver_wanted;

> +	uint16_t major_ver_found;

> +	uint16_t minor_ver_found;

>

>  	uint32_t header_size;

>  	uint32_t header_offset;

> @@ -123,7 +123,7 @@ struct intel_guc_fw {  };

>

>  struct intel_guc {

> -	struct intel_guc_fw guc_fw;

> +	struct intel_uc_fw guc_fw;

>  	uint32_t log_flags;

>  	struct drm_i915_gem_object *log_obj;

>

> @@ -152,9 +152,10 @@ struct intel_guc {  extern void 

> intel_guc_init(struct drm_device *dev);  extern int 

> intel_guc_setup(struct drm_device *dev);  extern void 

> intel_guc_fini(struct drm_device *dev); -extern const char 

> *intel_guc_fw_status_repr(enum intel_guc_fw_status status);

> +extern const char *intel_uc_fw_status_repr(enum intel_uc_fw_status 

> +status);

>  extern int intel_guc_suspend(struct drm_device *dev);  extern int 

> intel_guc_resume(struct drm_device *dev);

> +void intel_uc_fw_fetch(struct drm_device *dev, struct intel_uc_fw 

> +*uc_fw);

>

>  /* i915_guc_submission.c */

>  int i915_guc_submission_init(struct drm_i915_private *dev_priv); diff 

> --git a/drivers/gpu/drm/i915/intel_guc_loader.c 

> b/drivers/gpu/drm/i915/intel_guc_loader.c

> index b883efd..25c9a91 100644

> --- a/drivers/gpu/drm/i915/intel_guc_loader.c

> +++ b/drivers/gpu/drm/i915/intel_guc_loader.c

> @@ -69,16 +69,16 @@ MODULE_FIRMWARE(I915_BXT_GUC_UCODE);

>  MODULE_FIRMWARE(I915_KBL_GUC_UCODE);

>

>  /* User-friendly representation of an enum */ -const char 

> *intel_guc_fw_status_repr(enum intel_guc_fw_status status)

> +const char *intel_uc_fw_status_repr(enum intel_uc_fw_status status)

>  {

>  	switch (status) {

> -	case GUC_FIRMWARE_FAIL:

> +	case UC_FIRMWARE_FAIL:

>  		return "FAIL";

> -	case GUC_FIRMWARE_NONE:

> +	case UC_FIRMWARE_NONE:

>  		return "NONE";

> -	case GUC_FIRMWARE_PENDING:

> +	case UC_FIRMWARE_PENDING:

>  		return "PENDING";

> -	case GUC_FIRMWARE_SUCCESS:

> +	case UC_FIRMWARE_SUCCESS:

>  		return "SUCCESS";

>  	default:

>  		return "UNKNOWN!";

> @@ -240,8 +240,8 @@ static inline bool guc_ucode_response(struct drm_i915_private *dev_priv,

>   */

>  static int guc_ucode_xfer_dma(struct drm_i915_private *dev_priv)  {

> -	struct intel_guc_fw *guc_fw = &dev_priv->guc.guc_fw;

> -	struct drm_i915_gem_object *fw_obj = guc_fw->guc_fw_obj;

> +	struct intel_uc_fw *guc_fw = &dev_priv->guc.guc_fw;

> +	struct drm_i915_gem_object *fw_obj = guc_fw->uc_fw_obj;

>  	unsigned long offset;

>  	struct sg_table *sg = fw_obj->pages;

>  	u32 status, rsa[UOS_RSA_SCRATCH_MAX_COUNT]; @@ -313,17 +313,17 @@ 

> static u32 guc_wopcm_size(struct drm_i915_private *dev_priv)

>   */

>  static int guc_ucode_xfer(struct drm_i915_private *dev_priv)  {

> -	struct intel_guc_fw *guc_fw = &dev_priv->guc.guc_fw;

> +	struct intel_uc_fw *guc_fw = &dev_priv->guc.guc_fw;

>  	struct drm_device *dev = &dev_priv->drm;

>  	int ret;

>

> -	ret = i915_gem_object_set_to_gtt_domain(guc_fw->guc_fw_obj, false);

> +	ret = i915_gem_object_set_to_gtt_domain(guc_fw->uc_fw_obj, false);

>  	if (ret) {

>  		DRM_DEBUG_DRIVER("set-domain failed %d\n", ret);

>  		return ret;

>  	}

>

> -	ret = i915_gem_obj_ggtt_pin(guc_fw->guc_fw_obj, 0, 0);

> +	ret = i915_gem_obj_ggtt_pin(guc_fw->uc_fw_obj, 0, 0);


This now has to be:

+	ret = i915_gem_object_ggtt_pin(guc_fw->uc_fw_obj, NULL, 0, 0, 0);

Otherwise OK

.Dave.

>  	if (ret) {

>  		DRM_DEBUG_DRIVER("pin failed %d\n", ret);

>  		return ret;

> @@ -377,7 +377,7 @@ static int guc_ucode_xfer(struct drm_i915_private *dev_priv)

>  	 * We keep the object pages for reuse during resume. But we can unpin it

>  	 * now that DMA has completed, so it doesn't continue to take up space.

>  	 */

> -	i915_gem_object_ggtt_unpin(guc_fw->guc_fw_obj);

> +	i915_gem_object_ggtt_unpin(guc_fw->uc_fw_obj);

>

>  	return ret;

>  }

> @@ -416,14 +416,14 @@ static int i915_reset_guc(struct 

> drm_i915_private *dev_priv)  int intel_guc_setup(struct drm_device 

> *dev)  {

>  	struct drm_i915_private *dev_priv = to_i915(dev);

> -	struct intel_guc_fw *guc_fw = &dev_priv->guc.guc_fw;

> -	const char *fw_path = guc_fw->guc_fw_path;

> +	struct intel_uc_fw *guc_fw = &dev_priv->guc.guc_fw;

> +	const char *fw_path = guc_fw->uc_fw_path;

>  	int retries, ret, err;

>

>  	DRM_DEBUG_DRIVER("GuC fw status: path %s, fetch %s, load %s\n",

>  		fw_path,

> -		intel_guc_fw_status_repr(guc_fw->guc_fw_fetch_status),

> -		intel_guc_fw_status_repr(guc_fw->guc_fw_load_status));

> +		intel_uc_fw_status_repr(guc_fw->fetch_status),

> +		intel_uc_fw_status_repr(guc_fw->load_status));

>

>  	/* Loading forbidden, or no firmware to load? */

>  	if (!i915.enable_guc_loading) {

> @@ -441,21 +441,21 @@ int intel_guc_setup(struct drm_device *dev)

>  	}

>

>  	/* Fetch failed, or already fetched but failed to load? */

> -	if (guc_fw->guc_fw_fetch_status != GUC_FIRMWARE_SUCCESS) {

> +	if (guc_fw->fetch_status != UC_FIRMWARE_SUCCESS) {

>  		err = -EIO;

>  		goto fail;

> -	} else if (guc_fw->guc_fw_load_status == GUC_FIRMWARE_FAIL) {

> +	} else if (guc_fw->load_status == UC_FIRMWARE_FAIL) {

>  		err = -ENOEXEC;

>  		goto fail;

>  	}

>

>  	direct_interrupts_to_host(dev_priv);

>

> -	guc_fw->guc_fw_load_status = GUC_FIRMWARE_PENDING;

> +	guc_fw->load_status = UC_FIRMWARE_PENDING;

>

>  	DRM_DEBUG_DRIVER("GuC fw status: fetch %s, load %s\n",

> -		intel_guc_fw_status_repr(guc_fw->guc_fw_fetch_status),

> -		intel_guc_fw_status_repr(guc_fw->guc_fw_load_status));

> +		intel_uc_fw_status_repr(guc_fw->fetch_status),

> +		intel_uc_fw_status_repr(guc_fw->load_status));

>

>  	err = i915_guc_submission_init(dev_priv);

>  	if (err)

> @@ -489,11 +489,11 @@ int intel_guc_setup(struct drm_device *dev)

>  			 "retry %d more time(s)\n", err, retries);

>  	}

>

> -	guc_fw->guc_fw_load_status = GUC_FIRMWARE_SUCCESS;

> +	guc_fw->load_status = UC_FIRMWARE_SUCCESS;

>

>  	DRM_DEBUG_DRIVER("GuC fw status: fetch %s, load %s\n",

> -		intel_guc_fw_status_repr(guc_fw->guc_fw_fetch_status),

> -		intel_guc_fw_status_repr(guc_fw->guc_fw_load_status));

> +		intel_uc_fw_status_repr(guc_fw->fetch_status),

> +		intel_uc_fw_status_repr(guc_fw->load_status));

>

>  	if (i915.enable_guc_submission) {

>  		err = i915_guc_submission_enable(dev_priv);

> @@ -505,8 +505,8 @@ int intel_guc_setup(struct drm_device *dev)

>  	return 0;

>

>  fail:

> -	if (guc_fw->guc_fw_load_status == GUC_FIRMWARE_PENDING)

> -		guc_fw->guc_fw_load_status = GUC_FIRMWARE_FAIL;

> +	if (guc_fw->load_status == UC_FIRMWARE_PENDING)

> +		guc_fw->load_status = UC_FIRMWARE_FAIL;

>

>  	direct_interrupts_to_host(dev_priv);

>  	i915_guc_submission_disable(dev_priv);

> @@ -551,7 +551,7 @@ fail:

>  	return ret;

>  }

>

> -static void guc_fw_fetch(struct drm_device *dev, struct intel_guc_fw 

> *guc_fw)

> +void intel_uc_fw_fetch(struct drm_device *dev, struct intel_uc_fw 

> +*uc_fw)

>  {

>  	struct drm_i915_gem_object *obj;

>  	const struct firmware *fw;

> @@ -559,17 +559,17 @@ static void guc_fw_fetch(struct drm_device *dev, struct intel_guc_fw *guc_fw)

>  	size_t size;

>  	int err;

>

> -	DRM_DEBUG_DRIVER("before requesting firmware: GuC fw fetch status %s\n",

> -		intel_guc_fw_status_repr(guc_fw->guc_fw_fetch_status));

> +	DRM_DEBUG_DRIVER("before requesting firmware: uC fw fetch status %s\n",

> +		intel_uc_fw_status_repr(uc_fw->fetch_status));

>

> -	err = request_firmware(&fw, guc_fw->guc_fw_path, &dev->pdev->dev);

> +	err = request_firmware(&fw, uc_fw->uc_fw_path, &dev->pdev->dev);

>  	if (err)

>  		goto fail;

>  	if (!fw)

>  		goto fail;

>

> -	DRM_DEBUG_DRIVER("fetch GuC fw from %s succeeded, fw %p\n",

> -		guc_fw->guc_fw_path, fw);

> +	DRM_DEBUG_DRIVER("fetch uC fw from %s succeeded, fw %p\n",

> +		uc_fw->uc_fw_path, fw);

>

>  	/* Check the size of the blob before examining buffer contents */

>  	if (fw->size < sizeof(struct guc_css_header)) { @@ -580,36 +580,36 

> @@ static void guc_fw_fetch(struct drm_device *dev, struct intel_guc_fw *guc_fw)

>  	css = (struct guc_css_header *)fw->data;

>

>  	/* Firmware bits always start from header */

> -	guc_fw->header_offset = 0;

> -	guc_fw->header_size = (css->header_size_dw - css->modulus_size_dw -

> +	uc_fw->header_offset = 0;

> +	uc_fw->header_size = (css->header_size_dw - css->modulus_size_dw -

>  		css->key_size_dw - css->exponent_size_dw) * sizeof(u32);

>

> -	if (guc_fw->header_size != sizeof(struct guc_css_header)) {

> +	if (uc_fw->header_size != sizeof(struct guc_css_header)) {

>  		DRM_ERROR("CSS header definition mismatch\n");

>  		goto fail;

>  	}

>

>  	/* then, uCode */

> -	guc_fw->ucode_offset = guc_fw->header_offset + guc_fw->header_size;

> -	guc_fw->ucode_size = (css->size_dw - css->header_size_dw) * sizeof(u32);

> +	uc_fw->ucode_offset = uc_fw->header_offset + uc_fw->header_size;

> +	uc_fw->ucode_size = (css->size_dw - css->header_size_dw) * 

> +sizeof(u32);

>

>  	/* now RSA */

>  	if (css->key_size_dw != UOS_RSA_SCRATCH_MAX_COUNT) {

>  		DRM_ERROR("RSA key size is bad\n");

>  		goto fail;

>  	}

> -	guc_fw->rsa_offset = guc_fw->ucode_offset + guc_fw->ucode_size;

> -	guc_fw->rsa_size = css->key_size_dw * sizeof(u32);

> +	uc_fw->rsa_offset = uc_fw->ucode_offset + uc_fw->ucode_size;

> +	uc_fw->rsa_size = css->key_size_dw * sizeof(u32);

>

>  	/* At least, it should have header, uCode and RSA. Size of all three. */

> -	size = guc_fw->header_size + guc_fw->ucode_size + guc_fw->rsa_size;

> +	size = uc_fw->header_size + uc_fw->ucode_size + uc_fw->rsa_size;

>  	if (fw->size < size) {

>  		DRM_ERROR("Missing firmware components\n");

>  		goto fail;

>  	}

>

>  	/* Header and uCode will be loaded to WOPCM. Size of the two. */

> -	size = guc_fw->header_size + guc_fw->ucode_size;

> +	size = uc_fw->header_size + uc_fw->ucode_size;

>  	if (size > guc_wopcm_size(to_i915(dev))) {

>  		DRM_ERROR("Firmware is too large to fit in WOPCM\n");

>  		goto fail;

> @@ -621,21 +621,21 @@ static void guc_fw_fetch(struct drm_device *dev, struct intel_guc_fw *guc_fw)

>  	 * TWO bytes each (i.e. u16), although all pointers and offsets are defined

>  	 * in terms of bytes (u8).

>  	 */

> -	guc_fw->guc_fw_major_found = css->guc_sw_version >> 16;

> -	guc_fw->guc_fw_minor_found = css->guc_sw_version & 0xFFFF;

> -

> -	if (guc_fw->guc_fw_major_found != guc_fw->guc_fw_major_wanted ||

> -	    guc_fw->guc_fw_minor_found < guc_fw->guc_fw_minor_wanted) {

> -		DRM_ERROR("GuC firmware version %d.%d, required %d.%d\n",

> -			guc_fw->guc_fw_major_found, guc_fw->guc_fw_minor_found,

> -			guc_fw->guc_fw_major_wanted, guc_fw->guc_fw_minor_wanted);

> +	uc_fw->major_ver_found = css->guc_sw_version >> 16;

> +	uc_fw->minor_ver_found = css->guc_sw_version & 0xFFFF;

> +

> +	if (uc_fw->major_ver_found != uc_fw->major_ver_wanted ||

> +	    uc_fw->minor_ver_found < uc_fw->minor_ver_wanted) {

> +		DRM_ERROR("uC firmware version %d.%d, required %d.%d\n",

> +			uc_fw->major_ver_found, uc_fw->minor_ver_found,

> +			uc_fw->major_ver_wanted, uc_fw->minor_ver_wanted);

>  		err = -ENOEXEC;

>  		goto fail;

>  	}

>

>  	DRM_DEBUG_DRIVER("firmware version %d.%d OK (minimum %d.%d)\n",

> -			guc_fw->guc_fw_major_found, guc_fw->guc_fw_minor_found,

> -			guc_fw->guc_fw_major_wanted, guc_fw->guc_fw_minor_wanted);

> +			uc_fw->major_ver_found, uc_fw->minor_ver_found,

> +			uc_fw->major_ver_wanted, uc_fw->minor_ver_wanted);

>

>  	mutex_lock(&dev->struct_mutex);

>  	obj = i915_gem_object_create_from_data(dev, fw->data, fw->size); @@ 

> -645,31 +645,31 @@ static void guc_fw_fetch(struct drm_device *dev, struct intel_guc_fw *guc_fw)

>  		goto fail;

>  	}

>

> -	guc_fw->guc_fw_obj = obj;

> -	guc_fw->guc_fw_size = fw->size;

> +	uc_fw->uc_fw_obj = obj;

> +	uc_fw->uc_fw_size = fw->size;

>

> -	DRM_DEBUG_DRIVER("GuC fw fetch status SUCCESS, obj %p\n",

> -			guc_fw->guc_fw_obj);

> +	DRM_DEBUG_DRIVER("uC fw fetch status SUCCESS, obj %p\n",

> +			uc_fw->uc_fw_obj);

>

>  	release_firmware(fw);

> -	guc_fw->guc_fw_fetch_status = GUC_FIRMWARE_SUCCESS;

> +	uc_fw->fetch_status = UC_FIRMWARE_SUCCESS;

>  	return;

>

>  fail:

> -	DRM_DEBUG_DRIVER("GuC fw fetch status FAIL; err %d, fw %p, obj %p\n",

> -		err, fw, guc_fw->guc_fw_obj);

> -	DRM_ERROR("Failed to fetch GuC firmware from %s (error %d)\n",

> -		  guc_fw->guc_fw_path, err);

> +	DRM_DEBUG_DRIVER("uC fw fetch status FAIL; err %d, fw %p, obj %p\n",

> +		err, fw, uc_fw->uc_fw_obj);

> +	DRM_ERROR("Failed to fetch uC firmware from %s (error %d)\n",

> +		  uc_fw->uc_fw_path, err);

>

>  	mutex_lock(&dev->struct_mutex);

> -	obj = guc_fw->guc_fw_obj;

> +	obj = uc_fw->uc_fw_obj;

>  	if (obj)

>  		i915_gem_object_put(obj);

> -	guc_fw->guc_fw_obj = NULL;

> +	uc_fw->uc_fw_obj = NULL;

>  	mutex_unlock(&dev->struct_mutex);

>

>  	release_firmware(fw);		/* OK even if fw is NULL */

> -	guc_fw->guc_fw_fetch_status = GUC_FIRMWARE_FAIL;

> +	uc_fw->fetch_status = UC_FIRMWARE_FAIL;

>  }

>

>  /**

> @@ -684,7 +684,7 @@ fail:

>  void intel_guc_init(struct drm_device *dev)  {

>  	struct drm_i915_private *dev_priv = to_i915(dev);

> -	struct intel_guc_fw *guc_fw = &dev_priv->guc.guc_fw;

> +	struct intel_uc_fw *guc_fw = &dev_priv->guc.guc_fw;

>  	const char *fw_path;

>

>  	/* A negative value means "use platform default" */ @@ -697,24 

> +697,24 @@ void intel_guc_init(struct drm_device *dev)

>  		fw_path = NULL;

>  	} else if (IS_SKYLAKE(dev)) {

>  		fw_path = I915_SKL_GUC_UCODE;

> -		guc_fw->guc_fw_major_wanted = 6;

> -		guc_fw->guc_fw_minor_wanted = 1;

> +		guc_fw->major_ver_wanted = 6;

> +		guc_fw->minor_ver_wanted = 1;

>  	} else if (IS_BROXTON(dev)) {

>  		fw_path = I915_BXT_GUC_UCODE;

> -		guc_fw->guc_fw_major_wanted = 8;

> -		guc_fw->guc_fw_minor_wanted = 7;

> +		guc_fw->major_ver_wanted = 8;

> +		guc_fw->minor_ver_wanted = 7;

>  	} else if (IS_KABYLAKE(dev)) {

>  		fw_path = I915_KBL_GUC_UCODE;

> -		guc_fw->guc_fw_major_wanted = 9;

> -		guc_fw->guc_fw_minor_wanted = 14;

> +		guc_fw->major_ver_wanted = 9;

> +		guc_fw->minor_ver_wanted = 14;

>  	} else {

>  		fw_path = "";	/* unknown device */

>  	}

>

> -	guc_fw->guc_dev = dev;

> -	guc_fw->guc_fw_path = fw_path;

> -	guc_fw->guc_fw_fetch_status = GUC_FIRMWARE_NONE;

> -	guc_fw->guc_fw_load_status = GUC_FIRMWARE_NONE;

> +	guc_fw->uc_dev = dev;

> +	guc_fw->uc_fw_path = fw_path;

> +	guc_fw->fetch_status = UC_FIRMWARE_NONE;

> +	guc_fw->load_status = UC_FIRMWARE_NONE;

>

>  	/* Early (and silent) return if GuC loading is disabled */

>  	if (!i915.enable_guc_loading)

> @@ -724,9 +724,9 @@ void intel_guc_init(struct drm_device *dev)

>  	if (*fw_path == '\0')

>  		return;

>

> -	guc_fw->guc_fw_fetch_status = GUC_FIRMWARE_PENDING;

> +	guc_fw->fetch_status = UC_FIRMWARE_PENDING;

>  	DRM_DEBUG_DRIVER("GuC firmware pending, path %s\n", fw_path);

> -	guc_fw_fetch(dev, guc_fw);

> +	intel_uc_fw_fetch(dev, guc_fw);

>  	/* status must now be FAIL or SUCCESS */  }

>

> @@ -737,17 +737,17 @@ void intel_guc_init(struct drm_device *dev)  

> void intel_guc_fini(struct drm_device *dev)  {

>  	struct drm_i915_private *dev_priv = to_i915(dev);

> -	struct intel_guc_fw *guc_fw = &dev_priv->guc.guc_fw;

> +	struct intel_uc_fw *guc_fw = &dev_priv->guc.guc_fw;

>

>  	mutex_lock(&dev->struct_mutex);

>  	direct_interrupts_to_host(dev_priv);

>  	i915_guc_submission_disable(dev_priv);

>  	i915_guc_submission_fini(dev_priv);

>

> -	if (guc_fw->guc_fw_obj)

> -		i915_gem_object_put(guc_fw->guc_fw_obj);

> -	guc_fw->guc_fw_obj = NULL;

> +	if (guc_fw->uc_fw_obj)

> +		i915_gem_object_put(guc_fw->uc_fw_obj);

> +	guc_fw->uc_fw_obj = NULL;

>  	mutex_unlock(&dev->struct_mutex);

>

> -	guc_fw->guc_fw_fetch_status = GUC_FIRMWARE_NONE;

> +	guc_fw->fetch_status = UC_FIRMWARE_NONE;

>  }

>
diff mbox

Patch

diff --git a/drivers/gpu/drm/i915/i915_debugfs.c b/drivers/gpu/drm/i915/i915_debugfs.c
index 531ca02..5df7bd3 100644
--- a/drivers/gpu/drm/i915/i915_debugfs.c
+++ b/drivers/gpu/drm/i915/i915_debugfs.c
@@ -2490,7 +2490,7 @@  static int i915_guc_load_status_info(struct seq_file *m, void *data)
 {
 	struct drm_info_node *node = m->private;
 	struct drm_i915_private *dev_priv = to_i915(node->minor->dev);
-	struct intel_guc_fw *guc_fw = &dev_priv->guc.guc_fw;
+	struct intel_uc_fw *guc_fw = &dev_priv->guc.guc_fw;
 	u32 tmp, i;
 
 	if (!HAS_GUC_UCODE(dev_priv))
@@ -2498,15 +2498,15 @@  static int i915_guc_load_status_info(struct seq_file *m, void *data)
 
 	seq_printf(m, "GuC firmware status:\n");
 	seq_printf(m, "\tpath: %s\n",
-		guc_fw->guc_fw_path);
+		guc_fw->uc_fw_path);
 	seq_printf(m, "\tfetch: %s\n",
-		intel_guc_fw_status_repr(guc_fw->guc_fw_fetch_status));
+		intel_uc_fw_status_repr(guc_fw->fetch_status));
 	seq_printf(m, "\tload: %s\n",
-		intel_guc_fw_status_repr(guc_fw->guc_fw_load_status));
+		intel_uc_fw_status_repr(guc_fw->load_status));
 	seq_printf(m, "\tversion wanted: %d.%d\n",
-		guc_fw->guc_fw_major_wanted, guc_fw->guc_fw_minor_wanted);
+		guc_fw->major_ver_wanted, guc_fw->minor_ver_wanted);
 	seq_printf(m, "\tversion found: %d.%d\n",
-		guc_fw->guc_fw_major_found, guc_fw->guc_fw_minor_found);
+		guc_fw->major_ver_found, guc_fw->minor_ver_found);
 	seq_printf(m, "\theader: offset is %d; size = %d\n",
 		guc_fw->header_offset, guc_fw->header_size);
 	seq_printf(m, "\tuCode: offset is %d; size = %d\n",
diff --git a/drivers/gpu/drm/i915/i915_guc_submission.c b/drivers/gpu/drm/i915/i915_guc_submission.c
index 01c1c16..f665a87 100644
--- a/drivers/gpu/drm/i915/i915_guc_submission.c
+++ b/drivers/gpu/drm/i915/i915_guc_submission.c
@@ -1044,7 +1044,7 @@  int intel_guc_suspend(struct drm_device *dev)
 	struct i915_gem_context *ctx;
 	u32 data[3];
 
-	if (guc->guc_fw.guc_fw_load_status != GUC_FIRMWARE_SUCCESS)
+	if (guc->guc_fw.load_status != UC_FIRMWARE_SUCCESS)
 		return 0;
 
 	ctx = dev_priv->kernel_context;
@@ -1070,7 +1070,7 @@  int intel_guc_resume(struct drm_device *dev)
 	struct i915_gem_context *ctx;
 	u32 data[3];
 
-	if (guc->guc_fw.guc_fw_load_status != GUC_FIRMWARE_SUCCESS)
+	if (guc->guc_fw.load_status != UC_FIRMWARE_SUCCESS)
 		return 0;
 
 	ctx = dev_priv->kernel_context;
diff --git a/drivers/gpu/drm/i915/intel_guc.h b/drivers/gpu/drm/i915/intel_guc.h
index 3e3e743..02adcfc 100644
--- a/drivers/gpu/drm/i915/intel_guc.h
+++ b/drivers/gpu/drm/i915/intel_guc.h
@@ -90,29 +90,29 @@  struct i915_guc_client {
 	uint64_t submissions[I915_NUM_ENGINES];
 };
 
-enum intel_guc_fw_status {
-	GUC_FIRMWARE_FAIL = -1,
-	GUC_FIRMWARE_NONE = 0,
-	GUC_FIRMWARE_PENDING,
-	GUC_FIRMWARE_SUCCESS
+enum intel_uc_fw_status {
+	UC_FIRMWARE_FAIL = -1,
+	UC_FIRMWARE_NONE = 0,
+	UC_FIRMWARE_PENDING,
+	UC_FIRMWARE_SUCCESS
 };
 
 /*
  * This structure encapsulates all the data needed during the process
  * of fetching, caching, and loading the firmware image into the GuC.
  */
-struct intel_guc_fw {
-	struct drm_device *		guc_dev;
-	const char *			guc_fw_path;
-	size_t				guc_fw_size;
-	struct drm_i915_gem_object *	guc_fw_obj;
-	enum intel_guc_fw_status	guc_fw_fetch_status;
-	enum intel_guc_fw_status	guc_fw_load_status;
-
-	uint16_t			guc_fw_major_wanted;
-	uint16_t			guc_fw_minor_wanted;
-	uint16_t			guc_fw_major_found;
-	uint16_t			guc_fw_minor_found;
+struct intel_uc_fw {
+	struct drm_device *uc_dev;
+	const char *uc_fw_path;
+	size_t uc_fw_size;
+	struct drm_i915_gem_object *uc_fw_obj;
+	enum intel_uc_fw_status fetch_status;
+	enum intel_uc_fw_status load_status;
+
+	uint16_t major_ver_wanted;
+	uint16_t minor_ver_wanted;
+	uint16_t major_ver_found;
+	uint16_t minor_ver_found;
 
 	uint32_t header_size;
 	uint32_t header_offset;
@@ -123,7 +123,7 @@  struct intel_guc_fw {
 };
 
 struct intel_guc {
-	struct intel_guc_fw guc_fw;
+	struct intel_uc_fw guc_fw;
 	uint32_t log_flags;
 	struct drm_i915_gem_object *log_obj;
 
@@ -152,9 +152,10 @@  struct intel_guc {
 extern void intel_guc_init(struct drm_device *dev);
 extern int intel_guc_setup(struct drm_device *dev);
 extern void intel_guc_fini(struct drm_device *dev);
-extern const char *intel_guc_fw_status_repr(enum intel_guc_fw_status status);
+extern const char *intel_uc_fw_status_repr(enum intel_uc_fw_status status);
 extern int intel_guc_suspend(struct drm_device *dev);
 extern int intel_guc_resume(struct drm_device *dev);
+void intel_uc_fw_fetch(struct drm_device *dev, struct intel_uc_fw *uc_fw);
 
 /* i915_guc_submission.c */
 int i915_guc_submission_init(struct drm_i915_private *dev_priv);
diff --git a/drivers/gpu/drm/i915/intel_guc_loader.c b/drivers/gpu/drm/i915/intel_guc_loader.c
index b883efd..25c9a91 100644
--- a/drivers/gpu/drm/i915/intel_guc_loader.c
+++ b/drivers/gpu/drm/i915/intel_guc_loader.c
@@ -69,16 +69,16 @@  MODULE_FIRMWARE(I915_BXT_GUC_UCODE);
 MODULE_FIRMWARE(I915_KBL_GUC_UCODE);
 
 /* User-friendly representation of an enum */
-const char *intel_guc_fw_status_repr(enum intel_guc_fw_status status)
+const char *intel_uc_fw_status_repr(enum intel_uc_fw_status status)
 {
 	switch (status) {
-	case GUC_FIRMWARE_FAIL:
+	case UC_FIRMWARE_FAIL:
 		return "FAIL";
-	case GUC_FIRMWARE_NONE:
+	case UC_FIRMWARE_NONE:
 		return "NONE";
-	case GUC_FIRMWARE_PENDING:
+	case UC_FIRMWARE_PENDING:
 		return "PENDING";
-	case GUC_FIRMWARE_SUCCESS:
+	case UC_FIRMWARE_SUCCESS:
 		return "SUCCESS";
 	default:
 		return "UNKNOWN!";
@@ -240,8 +240,8 @@  static inline bool guc_ucode_response(struct drm_i915_private *dev_priv,
  */
 static int guc_ucode_xfer_dma(struct drm_i915_private *dev_priv)
 {
-	struct intel_guc_fw *guc_fw = &dev_priv->guc.guc_fw;
-	struct drm_i915_gem_object *fw_obj = guc_fw->guc_fw_obj;
+	struct intel_uc_fw *guc_fw = &dev_priv->guc.guc_fw;
+	struct drm_i915_gem_object *fw_obj = guc_fw->uc_fw_obj;
 	unsigned long offset;
 	struct sg_table *sg = fw_obj->pages;
 	u32 status, rsa[UOS_RSA_SCRATCH_MAX_COUNT];
@@ -313,17 +313,17 @@  static u32 guc_wopcm_size(struct drm_i915_private *dev_priv)
  */
 static int guc_ucode_xfer(struct drm_i915_private *dev_priv)
 {
-	struct intel_guc_fw *guc_fw = &dev_priv->guc.guc_fw;
+	struct intel_uc_fw *guc_fw = &dev_priv->guc.guc_fw;
 	struct drm_device *dev = &dev_priv->drm;
 	int ret;
 
-	ret = i915_gem_object_set_to_gtt_domain(guc_fw->guc_fw_obj, false);
+	ret = i915_gem_object_set_to_gtt_domain(guc_fw->uc_fw_obj, false);
 	if (ret) {
 		DRM_DEBUG_DRIVER("set-domain failed %d\n", ret);
 		return ret;
 	}
 
-	ret = i915_gem_obj_ggtt_pin(guc_fw->guc_fw_obj, 0, 0);
+	ret = i915_gem_obj_ggtt_pin(guc_fw->uc_fw_obj, 0, 0);
 	if (ret) {
 		DRM_DEBUG_DRIVER("pin failed %d\n", ret);
 		return ret;
@@ -377,7 +377,7 @@  static int guc_ucode_xfer(struct drm_i915_private *dev_priv)
 	 * We keep the object pages for reuse during resume. But we can unpin it
 	 * now that DMA has completed, so it doesn't continue to take up space.
 	 */
-	i915_gem_object_ggtt_unpin(guc_fw->guc_fw_obj);
+	i915_gem_object_ggtt_unpin(guc_fw->uc_fw_obj);
 
 	return ret;
 }
@@ -416,14 +416,14 @@  static int i915_reset_guc(struct drm_i915_private *dev_priv)
 int intel_guc_setup(struct drm_device *dev)
 {
 	struct drm_i915_private *dev_priv = to_i915(dev);
-	struct intel_guc_fw *guc_fw = &dev_priv->guc.guc_fw;
-	const char *fw_path = guc_fw->guc_fw_path;
+	struct intel_uc_fw *guc_fw = &dev_priv->guc.guc_fw;
+	const char *fw_path = guc_fw->uc_fw_path;
 	int retries, ret, err;
 
 	DRM_DEBUG_DRIVER("GuC fw status: path %s, fetch %s, load %s\n",
 		fw_path,
-		intel_guc_fw_status_repr(guc_fw->guc_fw_fetch_status),
-		intel_guc_fw_status_repr(guc_fw->guc_fw_load_status));
+		intel_uc_fw_status_repr(guc_fw->fetch_status),
+		intel_uc_fw_status_repr(guc_fw->load_status));
 
 	/* Loading forbidden, or no firmware to load? */
 	if (!i915.enable_guc_loading) {
@@ -441,21 +441,21 @@  int intel_guc_setup(struct drm_device *dev)
 	}
 
 	/* Fetch failed, or already fetched but failed to load? */
-	if (guc_fw->guc_fw_fetch_status != GUC_FIRMWARE_SUCCESS) {
+	if (guc_fw->fetch_status != UC_FIRMWARE_SUCCESS) {
 		err = -EIO;
 		goto fail;
-	} else if (guc_fw->guc_fw_load_status == GUC_FIRMWARE_FAIL) {
+	} else if (guc_fw->load_status == UC_FIRMWARE_FAIL) {
 		err = -ENOEXEC;
 		goto fail;
 	}
 
 	direct_interrupts_to_host(dev_priv);
 
-	guc_fw->guc_fw_load_status = GUC_FIRMWARE_PENDING;
+	guc_fw->load_status = UC_FIRMWARE_PENDING;
 
 	DRM_DEBUG_DRIVER("GuC fw status: fetch %s, load %s\n",
-		intel_guc_fw_status_repr(guc_fw->guc_fw_fetch_status),
-		intel_guc_fw_status_repr(guc_fw->guc_fw_load_status));
+		intel_uc_fw_status_repr(guc_fw->fetch_status),
+		intel_uc_fw_status_repr(guc_fw->load_status));
 
 	err = i915_guc_submission_init(dev_priv);
 	if (err)
@@ -489,11 +489,11 @@  int intel_guc_setup(struct drm_device *dev)
 			 "retry %d more time(s)\n", err, retries);
 	}
 
-	guc_fw->guc_fw_load_status = GUC_FIRMWARE_SUCCESS;
+	guc_fw->load_status = UC_FIRMWARE_SUCCESS;
 
 	DRM_DEBUG_DRIVER("GuC fw status: fetch %s, load %s\n",
-		intel_guc_fw_status_repr(guc_fw->guc_fw_fetch_status),
-		intel_guc_fw_status_repr(guc_fw->guc_fw_load_status));
+		intel_uc_fw_status_repr(guc_fw->fetch_status),
+		intel_uc_fw_status_repr(guc_fw->load_status));
 
 	if (i915.enable_guc_submission) {
 		err = i915_guc_submission_enable(dev_priv);
@@ -505,8 +505,8 @@  int intel_guc_setup(struct drm_device *dev)
 	return 0;
 
 fail:
-	if (guc_fw->guc_fw_load_status == GUC_FIRMWARE_PENDING)
-		guc_fw->guc_fw_load_status = GUC_FIRMWARE_FAIL;
+	if (guc_fw->load_status == UC_FIRMWARE_PENDING)
+		guc_fw->load_status = UC_FIRMWARE_FAIL;
 
 	direct_interrupts_to_host(dev_priv);
 	i915_guc_submission_disable(dev_priv);
@@ -551,7 +551,7 @@  fail:
 	return ret;
 }
 
-static void guc_fw_fetch(struct drm_device *dev, struct intel_guc_fw *guc_fw)
+void intel_uc_fw_fetch(struct drm_device *dev, struct intel_uc_fw *uc_fw)
 {
 	struct drm_i915_gem_object *obj;
 	const struct firmware *fw;
@@ -559,17 +559,17 @@  static void guc_fw_fetch(struct drm_device *dev, struct intel_guc_fw *guc_fw)
 	size_t size;
 	int err;
 
-	DRM_DEBUG_DRIVER("before requesting firmware: GuC fw fetch status %s\n",
-		intel_guc_fw_status_repr(guc_fw->guc_fw_fetch_status));
+	DRM_DEBUG_DRIVER("before requesting firmware: uC fw fetch status %s\n",
+		intel_uc_fw_status_repr(uc_fw->fetch_status));
 
-	err = request_firmware(&fw, guc_fw->guc_fw_path, &dev->pdev->dev);
+	err = request_firmware(&fw, uc_fw->uc_fw_path, &dev->pdev->dev);
 	if (err)
 		goto fail;
 	if (!fw)
 		goto fail;
 
-	DRM_DEBUG_DRIVER("fetch GuC fw from %s succeeded, fw %p\n",
-		guc_fw->guc_fw_path, fw);
+	DRM_DEBUG_DRIVER("fetch uC fw from %s succeeded, fw %p\n",
+		uc_fw->uc_fw_path, fw);
 
 	/* Check the size of the blob before examining buffer contents */
 	if (fw->size < sizeof(struct guc_css_header)) {
@@ -580,36 +580,36 @@  static void guc_fw_fetch(struct drm_device *dev, struct intel_guc_fw *guc_fw)
 	css = (struct guc_css_header *)fw->data;
 
 	/* Firmware bits always start from header */
-	guc_fw->header_offset = 0;
-	guc_fw->header_size = (css->header_size_dw - css->modulus_size_dw -
+	uc_fw->header_offset = 0;
+	uc_fw->header_size = (css->header_size_dw - css->modulus_size_dw -
 		css->key_size_dw - css->exponent_size_dw) * sizeof(u32);
 
-	if (guc_fw->header_size != sizeof(struct guc_css_header)) {
+	if (uc_fw->header_size != sizeof(struct guc_css_header)) {
 		DRM_ERROR("CSS header definition mismatch\n");
 		goto fail;
 	}
 
 	/* then, uCode */
-	guc_fw->ucode_offset = guc_fw->header_offset + guc_fw->header_size;
-	guc_fw->ucode_size = (css->size_dw - css->header_size_dw) * sizeof(u32);
+	uc_fw->ucode_offset = uc_fw->header_offset + uc_fw->header_size;
+	uc_fw->ucode_size = (css->size_dw - css->header_size_dw) * sizeof(u32);
 
 	/* now RSA */
 	if (css->key_size_dw != UOS_RSA_SCRATCH_MAX_COUNT) {
 		DRM_ERROR("RSA key size is bad\n");
 		goto fail;
 	}
-	guc_fw->rsa_offset = guc_fw->ucode_offset + guc_fw->ucode_size;
-	guc_fw->rsa_size = css->key_size_dw * sizeof(u32);
+	uc_fw->rsa_offset = uc_fw->ucode_offset + uc_fw->ucode_size;
+	uc_fw->rsa_size = css->key_size_dw * sizeof(u32);
 
 	/* At least, it should have header, uCode and RSA. Size of all three. */
-	size = guc_fw->header_size + guc_fw->ucode_size + guc_fw->rsa_size;
+	size = uc_fw->header_size + uc_fw->ucode_size + uc_fw->rsa_size;
 	if (fw->size < size) {
 		DRM_ERROR("Missing firmware components\n");
 		goto fail;
 	}
 
 	/* Header and uCode will be loaded to WOPCM. Size of the two. */
-	size = guc_fw->header_size + guc_fw->ucode_size;
+	size = uc_fw->header_size + uc_fw->ucode_size;
 	if (size > guc_wopcm_size(to_i915(dev))) {
 		DRM_ERROR("Firmware is too large to fit in WOPCM\n");
 		goto fail;
@@ -621,21 +621,21 @@  static void guc_fw_fetch(struct drm_device *dev, struct intel_guc_fw *guc_fw)
 	 * TWO bytes each (i.e. u16), although all pointers and offsets are defined
 	 * in terms of bytes (u8).
 	 */
-	guc_fw->guc_fw_major_found = css->guc_sw_version >> 16;
-	guc_fw->guc_fw_minor_found = css->guc_sw_version & 0xFFFF;
-
-	if (guc_fw->guc_fw_major_found != guc_fw->guc_fw_major_wanted ||
-	    guc_fw->guc_fw_minor_found < guc_fw->guc_fw_minor_wanted) {
-		DRM_ERROR("GuC firmware version %d.%d, required %d.%d\n",
-			guc_fw->guc_fw_major_found, guc_fw->guc_fw_minor_found,
-			guc_fw->guc_fw_major_wanted, guc_fw->guc_fw_minor_wanted);
+	uc_fw->major_ver_found = css->guc_sw_version >> 16;
+	uc_fw->minor_ver_found = css->guc_sw_version & 0xFFFF;
+
+	if (uc_fw->major_ver_found != uc_fw->major_ver_wanted ||
+	    uc_fw->minor_ver_found < uc_fw->minor_ver_wanted) {
+		DRM_ERROR("uC firmware version %d.%d, required %d.%d\n",
+			uc_fw->major_ver_found, uc_fw->minor_ver_found,
+			uc_fw->major_ver_wanted, uc_fw->minor_ver_wanted);
 		err = -ENOEXEC;
 		goto fail;
 	}
 
 	DRM_DEBUG_DRIVER("firmware version %d.%d OK (minimum %d.%d)\n",
-			guc_fw->guc_fw_major_found, guc_fw->guc_fw_minor_found,
-			guc_fw->guc_fw_major_wanted, guc_fw->guc_fw_minor_wanted);
+			uc_fw->major_ver_found, uc_fw->minor_ver_found,
+			uc_fw->major_ver_wanted, uc_fw->minor_ver_wanted);
 
 	mutex_lock(&dev->struct_mutex);
 	obj = i915_gem_object_create_from_data(dev, fw->data, fw->size);
@@ -645,31 +645,31 @@  static void guc_fw_fetch(struct drm_device *dev, struct intel_guc_fw *guc_fw)
 		goto fail;
 	}
 
-	guc_fw->guc_fw_obj = obj;
-	guc_fw->guc_fw_size = fw->size;
+	uc_fw->uc_fw_obj = obj;
+	uc_fw->uc_fw_size = fw->size;
 
-	DRM_DEBUG_DRIVER("GuC fw fetch status SUCCESS, obj %p\n",
-			guc_fw->guc_fw_obj);
+	DRM_DEBUG_DRIVER("uC fw fetch status SUCCESS, obj %p\n",
+			uc_fw->uc_fw_obj);
 
 	release_firmware(fw);
-	guc_fw->guc_fw_fetch_status = GUC_FIRMWARE_SUCCESS;
+	uc_fw->fetch_status = UC_FIRMWARE_SUCCESS;
 	return;
 
 fail:
-	DRM_DEBUG_DRIVER("GuC fw fetch status FAIL; err %d, fw %p, obj %p\n",
-		err, fw, guc_fw->guc_fw_obj);
-	DRM_ERROR("Failed to fetch GuC firmware from %s (error %d)\n",
-		  guc_fw->guc_fw_path, err);
+	DRM_DEBUG_DRIVER("uC fw fetch status FAIL; err %d, fw %p, obj %p\n",
+		err, fw, uc_fw->uc_fw_obj);
+	DRM_ERROR("Failed to fetch uC firmware from %s (error %d)\n",
+		  uc_fw->uc_fw_path, err);
 
 	mutex_lock(&dev->struct_mutex);
-	obj = guc_fw->guc_fw_obj;
+	obj = uc_fw->uc_fw_obj;
 	if (obj)
 		i915_gem_object_put(obj);
-	guc_fw->guc_fw_obj = NULL;
+	uc_fw->uc_fw_obj = NULL;
 	mutex_unlock(&dev->struct_mutex);
 
 	release_firmware(fw);		/* OK even if fw is NULL */
-	guc_fw->guc_fw_fetch_status = GUC_FIRMWARE_FAIL;
+	uc_fw->fetch_status = UC_FIRMWARE_FAIL;
 }
 
 /**
@@ -684,7 +684,7 @@  fail:
 void intel_guc_init(struct drm_device *dev)
 {
 	struct drm_i915_private *dev_priv = to_i915(dev);
-	struct intel_guc_fw *guc_fw = &dev_priv->guc.guc_fw;
+	struct intel_uc_fw *guc_fw = &dev_priv->guc.guc_fw;
 	const char *fw_path;
 
 	/* A negative value means "use platform default" */
@@ -697,24 +697,24 @@  void intel_guc_init(struct drm_device *dev)
 		fw_path = NULL;
 	} else if (IS_SKYLAKE(dev)) {
 		fw_path = I915_SKL_GUC_UCODE;
-		guc_fw->guc_fw_major_wanted = 6;
-		guc_fw->guc_fw_minor_wanted = 1;
+		guc_fw->major_ver_wanted = 6;
+		guc_fw->minor_ver_wanted = 1;
 	} else if (IS_BROXTON(dev)) {
 		fw_path = I915_BXT_GUC_UCODE;
-		guc_fw->guc_fw_major_wanted = 8;
-		guc_fw->guc_fw_minor_wanted = 7;
+		guc_fw->major_ver_wanted = 8;
+		guc_fw->minor_ver_wanted = 7;
 	} else if (IS_KABYLAKE(dev)) {
 		fw_path = I915_KBL_GUC_UCODE;
-		guc_fw->guc_fw_major_wanted = 9;
-		guc_fw->guc_fw_minor_wanted = 14;
+		guc_fw->major_ver_wanted = 9;
+		guc_fw->minor_ver_wanted = 14;
 	} else {
 		fw_path = "";	/* unknown device */
 	}
 
-	guc_fw->guc_dev = dev;
-	guc_fw->guc_fw_path = fw_path;
-	guc_fw->guc_fw_fetch_status = GUC_FIRMWARE_NONE;
-	guc_fw->guc_fw_load_status = GUC_FIRMWARE_NONE;
+	guc_fw->uc_dev = dev;
+	guc_fw->uc_fw_path = fw_path;
+	guc_fw->fetch_status = UC_FIRMWARE_NONE;
+	guc_fw->load_status = UC_FIRMWARE_NONE;
 
 	/* Early (and silent) return if GuC loading is disabled */
 	if (!i915.enable_guc_loading)
@@ -724,9 +724,9 @@  void intel_guc_init(struct drm_device *dev)
 	if (*fw_path == '\0')
 		return;
 
-	guc_fw->guc_fw_fetch_status = GUC_FIRMWARE_PENDING;
+	guc_fw->fetch_status = UC_FIRMWARE_PENDING;
 	DRM_DEBUG_DRIVER("GuC firmware pending, path %s\n", fw_path);
-	guc_fw_fetch(dev, guc_fw);
+	intel_uc_fw_fetch(dev, guc_fw);
 	/* status must now be FAIL or SUCCESS */
 }
 
@@ -737,17 +737,17 @@  void intel_guc_init(struct drm_device *dev)
 void intel_guc_fini(struct drm_device *dev)
 {
 	struct drm_i915_private *dev_priv = to_i915(dev);
-	struct intel_guc_fw *guc_fw = &dev_priv->guc.guc_fw;
+	struct intel_uc_fw *guc_fw = &dev_priv->guc.guc_fw;
 
 	mutex_lock(&dev->struct_mutex);
 	direct_interrupts_to_host(dev_priv);
 	i915_guc_submission_disable(dev_priv);
 	i915_guc_submission_fini(dev_priv);
 
-	if (guc_fw->guc_fw_obj)
-		i915_gem_object_put(guc_fw->guc_fw_obj);
-	guc_fw->guc_fw_obj = NULL;
+	if (guc_fw->uc_fw_obj)
+		i915_gem_object_put(guc_fw->uc_fw_obj);
+	guc_fw->uc_fw_obj = NULL;
 	mutex_unlock(&dev->struct_mutex);
 
-	guc_fw->guc_fw_fetch_status = GUC_FIRMWARE_NONE;
+	guc_fw->fetch_status = UC_FIRMWARE_NONE;
 }