[Intel-gfx] [PATCH] drm/i915: Add I915_PARAM_REVISION

[Daniel Vetter]

On Wed, Mar 04, 2015 at 02:41:16PM +0000, Neil Roberts wrote:
> Adds a parameter which can be used with DRM_I915_GETPARAM to query the
> GPU revision. The intention is to use this in Mesa to implement the
> WaDisableSIMD16On3SrcInstr workaround on Skylake but only for
> revision 2.
> 
> Signed-off-by: Neil Roberts <neil at linux.intel.com>

Just listened to Damien explain this all and pulled it right into dinq.

Cheers, Daniel

> ---
> The corresponding Mesa patches are here:
> 
> http://lists.freedesktop.org/archives/mesa-dev/2015-March/078534.html
> 
> and the libdrm patch is here:
> 
> http://lists.freedesktop.org/archives/dri-devel/2015-March/078645.html
> 
>  drivers/gpu/drm/i915/i915_dma.c | 3 +++
>  include/uapi/drm/i915_drm.h     | 1 +
>  2 files changed, 4 insertions(+)
> 
> diff --git a/drivers/gpu/drm/i915/i915_dma.c b/drivers/gpu/drm/i915/i915_dma.c
> index 0312879..36dd1df 100644
> --- a/drivers/gpu/drm/i915/i915_dma.c
> +++ b/drivers/gpu/drm/i915/i915_dma.c
> @@ -68,6 +68,9 @@ static int i915_getparam(struct drm_device *dev, void *data,
>  	case I915_PARAM_CHIPSET_ID:
>  		value = dev->pdev->device;
>  		break;
> +	case I915_PARAM_REVISION:
> +		value = dev->pdev->revision;
> +		break;
>  	case I915_PARAM_HAS_GEM:
>  		value = 1;
>  		break;
> diff --git a/include/uapi/drm/i915_drm.h b/include/uapi/drm/i915_drm.h
> index 6eed16b..b768f3b 100644
> --- a/include/uapi/drm/i915_drm.h
> +++ b/include/uapi/drm/i915_drm.h
> @@ -347,6 +347,7 @@ typedef struct drm_i915_irq_wait {
>  #define I915_PARAM_HAS_COHERENT_PHYS_GTT 29
>  #define I915_PARAM_MMAP_VERSION          30
>  #define I915_PARAM_HAS_BSD2		 31
> +#define I915_PARAM_REVISION              32
>  
>  typedef struct drm_i915_getparam {
>  	int param;
> -- 
> 1.9.3
> 
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx

-- 
Daniel Vetter
Software Engineer, Intel Corporation
+41 (0) 79 365 57 48 - http://blog.ffwll.ch