Re: [PATCH 1/7] drm/i915/skl: Store and print the DMC firmware version we load

[Imre Deak <imre.deak@intel.com>]

On ti, 2015-10-27 at 14:46 +0200, Mika Kuoppala wrote:
> From: Damien Lespiau <damien.lespiau@intel.com>
> 
> That can be handy later on to tell which DMC firmware version the user
> has, by just looking at the dmesg.
> 
> v2: use DRM_DEBUG_DRIVER (Chris)
> v3: use DRM_INFO (Marc Herbert)
> 
> Cc: Marc Herbert <marc.herbert@intel.com>
> Signed-off-by: Damien Lespiau <damien.lespiau@intel.com> (v1)
> Signed-off-by: Mika Kuoppala <mika.kuoppala@intel.com>

Reviewed-by: Imre Deak <imre.deak@intel.com>

> ---
>  drivers/gpu/drm/i915/i915_drv.h  | 5 +++++
>  drivers/gpu/drm/i915/intel_csr.c | 9 ++++++++-
>  2 files changed, 13 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
> index b408ebf..0bee438 100644
> --- a/drivers/gpu/drm/i915/i915_drv.h
> +++ b/drivers/gpu/drm/i915/i915_drv.h
> @@ -734,6 +734,10 @@ struct intel_uncore {
>  #define for_each_fw_domain(domain__, dev_priv__, i__) \
>  	for_each_fw_domain_mask(domain__, FORCEWAKE_ALL, dev_priv__, i__)
>  
> +#define CSR_VERSION(major, minor)	((major) << 16 | (minor))
> +#define CSR_VERSION_MAJOR(version)	((version) >> 16)
> +#define CSR_VERSION_MINOR(version)	((version) & 0xffff)
> +
>  enum csr_state {
>  	FW_UNINITIALIZED = 0,
>  	FW_LOADED,
> @@ -744,6 +748,7 @@ struct intel_csr {
>  	const char *fw_path;
>  	uint32_t *dmc_payload;
>  	uint32_t dmc_fw_size;
> +	uint32_t version;
>  	uint32_t mmio_count;
>  	uint32_t mmioaddr[8];
>  	uint32_t mmiodata[8];
> diff --git a/drivers/gpu/drm/i915/intel_csr.c b/drivers/gpu/drm/i915/intel_csr.c
> index 9e530a7..e620e85 100644
> --- a/drivers/gpu/drm/i915/intel_csr.c
> +++ b/drivers/gpu/drm/i915/intel_csr.c
> @@ -321,6 +321,9 @@ static void finish_csr_load(const struct firmware *fw, void *context)
>  			(css_header->header_len * 4));
>  		goto out;
>  	}
> +
> +	csr->version = css_header->version;
> +
>  	readcount += sizeof(struct intel_css_header);
>  
>  	/* Extract Package Header information*/
> @@ -402,7 +405,11 @@ static void finish_csr_load(const struct firmware *fw, void *context)
>  	intel_csr_load_program(dev);
>  	fw_loaded = true;
>  
> -	DRM_DEBUG_KMS("Finished loading %s\n", dev_priv->csr.fw_path);
> +	DRM_INFO("Finished loading %s (v%u.%u)\n",
> +		 dev_priv->csr.fw_path,
> +		 CSR_VERSION_MAJOR(csr->version),
> +		 CSR_VERSION_MINOR(csr->version));
> +
>  out:
>  	if (fw_loaded)
>  		intel_runtime_pm_put(dev_priv);


_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx