Re: [PATCH v2 0/4] Implement Wa_14022698537

[Jani Nikula <jani.nikula@linux.intel.com>]

On Fri, 11 Oct 2024, Raag Jadav <raag.jadav@intel.com> wrote:
> This series implements Wa_14022698537 along with its prerequisites.
> Detailed description in commit message.

The same workaround is also needed for the xe driver, right?

Please include that in the same series, so that we can unify the
solution in a way that suits everyone. Especially the PCI ID macros need
to match between i915_pciids.h and xe_pciids.h, because my plan is to
merge those two files together soonish.

BR,
Jani.

>
> v2: Introduce DG2_WA subplatform for workaround (Jani)
>     Fix Wa_ID and include it in subject (Badal)
>     Rephrase commit message (Jani)
>     Move CPU whitelist to intel_wa_cpu.c
> v1: https://patchwork.freedesktop.org/series/139628/
>
> Raag Jadav (4):
>   drm/i915/pciids: Refactor DG2 PCI IDs into workaround ranges
>   drm/i915/dg2: Introduce DG2_WA subplatform
>   drm/i915/wa: Introduce intel_wa_cpu.c for CPU specific workarounds
>   drm/i915/dg2: Implement Wa_14022698537
>
>  drivers/gpu/drm/i915/Makefile               |  1 +
>  drivers/gpu/drm/i915/gt/intel_wa_cpu.c      | 34 +++++++++++++++++++++
>  drivers/gpu/drm/i915/gt/intel_workarounds.c | 18 +++++++++++
>  drivers/gpu/drm/i915/gt/intel_workarounds.h |  2 ++
>  drivers/gpu/drm/i915/i915_drv.h             |  2 ++
>  drivers/gpu/drm/i915/i915_reg.h             |  1 +
>  drivers/gpu/drm/i915/intel_device_info.c    | 34 +++++++++++++++------
>  drivers/gpu/drm/i915/intel_device_info.h    |  5 ++-
>  include/drm/intel/i915_pciids.h             | 34 +++++++++++++++------
>  9 files changed, 111 insertions(+), 20 deletions(-)
>  create mode 100644 drivers/gpu/drm/i915/gt/intel_wa_cpu.c

-- 
Jani Nikula, Intel