RE: [PATCH 1/3] drm/xe/display: drop redundant intel_hpd_poll_fini() call

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

On Tue, 04 Aug 2026, "Shankar, Uma" <uma.shankar@intel.com> wrote:
>> -----Original Message-----
>> From: Intel-xe <intel-xe-bounces@lists.freedesktop.org> On Behalf Of Jani Nikula
>> Sent: Tuesday, July 28, 2026 8:20 PM
>> To: intel-xe@lists.freedesktop.org
>> Cc: Nikula, Jani <jani.nikula@intel.com>
>> Subject: [PATCH 1/3] drm/xe/display: drop redundant intel_hpd_poll_fini() call
>> 
>> intel_hpd_poll_fini() is already being called via
>> xe_display_fini_early() -> intel_display_driver_remove_noirq(). Remove the
>> redundant call.
>
> Looks Good to me.
> Reviewed-by: Uma Shankar <uma.shankar@intel.com>

Thanks for the reviews, series pushed to din.

BR,
Jani.

>
>> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
>> ---
>>  drivers/gpu/drm/xe/display/xe_display.c | 1 -
>>  1 file changed, 1 deletion(-)
>> 
>> diff --git a/drivers/gpu/drm/xe/display/xe_display.c
>> b/drivers/gpu/drm/xe/display/xe_display.c
>> index 8da4c457a4a0..57088e24aba2 100644
>> --- a/drivers/gpu/drm/xe/display/xe_display.c
>> +++ b/drivers/gpu/drm/xe/display/xe_display.c
>> @@ -129,7 +129,6 @@ static void xe_display_fini(void *arg)
>>  	struct xe_device *xe = arg;
>>  	struct intel_display *display = xe->display;
>> 
>> -	intel_hpd_poll_fini(display);
>>  	intel_hdcp_component_fini(display);
>>  	intel_audio_deinit(display);
>>  	intel_display_driver_remove(display);
>> --
>> 2.47.3
>

-- 
Jani Nikula, Intel