RE: [PATCH v2 1/3] drm/i915/display: pass struct intel_display to PCH macros

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

On Mon, 21 Apr 2025, "Borah, Chaitanya Kumar" <chaitanya.kumar.borah@intel.com> wrote:
>> -----Original Message-----
>> From: Intel-gfx <intel-gfx-bounces@lists.freedesktop.org> On Behalf Of Jani
>> Nikula
>> Sent: Thursday, April 17, 2025 2:41 PM
>> To: intel-gfx@lists.freedesktop.org; intel-xe@lists.freedesktop.org
>> Cc: Nikula, Jani <jani.nikula@intel.com>
>> Subject: [PATCH v2 1/3] drm/i915/display: pass struct intel_display to PCH
>> macros
>> 
>> Now that INTEL_PCH_TYPE() and HAS_PCH_*() macros are under display, and
>> accept a struct intel_display pointer, use that instead of struct
>> drm_i915_private pointer in display code.
>> 
>> This is done naively by running:
>> 
>> $ sed -i 's/\(INTEL_PCH_TYPE\|HAS_PCH_[A-Z0-9_-]*\)([^)]*)/\1(display)/g' \
>>   $(find drivers/gpu/drm/i915/display -name "*.c")
>> 
>> and fixing the fallout, i.e. removing unused local i915 variables and
>> adding display variables where needed.
>> 
>> v2: Rebase
>
> LGTM
>
>
> Reviewed-by: Chaitanya Kumar Borah <chaitanya.kumar.borah@intel.com>

Thanks for the reviews, series pushed to drm-intel-next.

BR,
Jani.

-- 
Jani Nikula, Intel