Re: [PATCH 0/2] Introduce helper for display workarounds and add Wa_16025573575

["Nautiyal, Ankit K" <ankit.k.nautiyal@intel.com>]

On 7/11/2025 9:48 AM, Ankit Nautiyal wrote:
> This series introduces a generic infrastructure for querying display
> workarounds. The goal is to simplify WA checks, avoid open-coded conditions,
> and make it easier to extend support for future workarounds.
>
> Patch 1 introduces the base infrastructure using an enum and a central
> helper function. It also migrates Wa_16023588340 to use this new interface.
>
> Patch 2 adds support for Wa_16025573575, which applies to PTL platforms
> and requires preserving additional GPIO bits in GMBUS.
>
> The series is in response to the suggestions to unify workaround handling
> and allowing future automation or generation of WA logic in [1].
>
> [1] https://lore.kernel.org/intel-gfx/7f079861f91861e9e895240cd3272f6e29deab7e@intel.com/


Thanks for the feedback, and suggestions. Thanks Jani and Gustavo for 
the reviews.

Patches were pushed to drm-intel-next.

Regards,

Ankit

>
> Rev2:
> -Remove MISSING_CASE, use drm_WARN.
> -Simplify macro for display_wa.
>
> Rev3:
> -Print missing wa_number.
> -Add more documentation for the WA.
> -Extend the WA for WCL.
>
> Ankit Nautiyal (2):
>    drm/i915/display_wa: Add helpers to check wa
>    drm/i915/gmbus: Add Wa_16025573575 for PTL/WCL for bit-bashing
>
>   .../gpu/drm/i915/display/intel_display_wa.c   | 27 +++++++++++++++
>   .../gpu/drm/i915/display/intel_display_wa.h   | 10 ++++++
>   drivers/gpu/drm/i915/display/intel_fbc.c      |  2 +-
>   drivers/gpu/drm/i915/display/intel_gmbus.c    | 34 +++++++++++++++++--
>   4 files changed, 70 insertions(+), 3 deletions(-)
>