On ti, 2015-10-27 at 14:46 +0200, Mika Kuoppala wrote:
> From: Damien Lespiau <damien.lesp...@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.herb...@intel.com>
> Signed-off-by: Damien Lespiau <damien.lesp...@intel.com> (v1)
> Signed-off-by: Mika Kuoppala <mika.kuopp...@intel.com>

Reviewed-by: Imre Deak <imre.d...@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

Reply via email to