Re: [PATCH] drm/i915/gmbus: Add Wa_16025573575 for PTL for bit-bashing

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

On Fri, 13 Jun 2025, Gustavo Sousa <gustavo.sousa@intel.com> wrote:
> Quoting Jani Nikula (2025-06-13 06:13:55-03:00)
>>For now, you'd manually check for each enum in __intel_display_wa()
>>implementation, but in the future that code could be generated similar
>>to what xe currently has. I'm just not very enthusiastic about adding C
>>language hostprogs with custom .rules files and parsing.
>
> Hehe. Yeah. I'm wondering if something nice could be achieved with
> header files and cpp magic...

Ugh! :)

If it was universally okay to add Python as a build dependency, I'd have
done this already. The rules format could be yaml, json, toml, or
something supported by Python out of the box (not xml though!). The
whole thing could be 100-200 lines of Python without a hostprogs build
step.

drm/msm has quietly added a Python build dependency a few releases ago,
and I'm kind of still waiting to see how it plays out...

BR,
Jani.


-- 
Jani Nikula, Intel