Re: [PATCH 1/5] drm/i915: Add IS_BDW_GT3 macro.

Jesse Barnes
2014-09-29T19:58:12Z

On Fri, 19 Sep 2014 20:16:26 -0400
Rodrigo Vivi <rodrigo.vivi@intel.com> wrote:

> It will be usefull to specify w/a that affects only BDW GT3.
>
> Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
> ---
> drivers/gpu/drm/i915/i915_drv.h | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/gpu/drm/i915/i915_drv.h b/drivers/gpu/drm/i915/i915_drv.h
> index 5fce16c..a00214e 100644
> --- a/drivers/gpu/drm/i915/i915_drv.h
> +++ b/drivers/gpu/drm/i915/i915_drv.h
> @@ -2099,6 +2099,8 @@ struct drm_i915_cmd_table {
> ((INTEL_DEVID(dev) & 0xf) == 0x2 || \
> (INTEL_DEVID(dev) & 0xf) == 0x6 || \
> (INTEL_DEVID(dev) & 0xf) == 0xe))
> +#define IS_BDW_GT3(dev) (IS_BROADWELL(dev) && \
> + (INTEL_DEVID(dev) & 0x00F0) == 0x0020)
> #define IS_HSW_ULT(dev) (IS_HASWELL(dev) && \
> (INTEL_DEVID(dev) & 0xFF00) == 0x0A00)
> #define IS_ULT(dev) (IS_HSW_ULT(dev) || IS_BDW_ULT(dev))

Looks correct based on the configs I'm staring at...

Reviewed-by: Jesse Barnes <jbarnes@virtuousgeek.org>

--
Jesse Barnes, Intel Open Source Technology Center