RE: [PATCH 2/3] drm/xe/display: drop redundant intel_hdcp_component_fini() call

> -----Original Message-----
> From: Intel-xe <[email protected]> On Behalf Of Jani Nikula
> Sent: Tuesday, July 28, 2026 8:20 PM
> To: [email protected]
> Cc: Nikula, Jani <[email protected]>
> Subject: [PATCH 2/3] drm/xe/display: drop redundant intel_hdcp_component_fini()
> call
> 
> intel_hdcp_component_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 <[email protected]>

> Signed-off-by: Jani Nikula <[email protected]>
> ---
>  drivers/gpu/drm/xe/display/xe_display.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/xe/display/xe_display.c
> b/drivers/gpu/drm/xe/display/xe_display.c
> index 57088e24aba2..2fc12592627c 100644
> --- a/drivers/gpu/drm/xe/display/xe_display.c
> +++ b/drivers/gpu/drm/xe/display/xe_display.c
> @@ -29,7 +29,6 @@
>  #include "intel_dmc_wl.h"
>  #include "intel_dp.h"
>  #include "intel_fbdev.h"
> -#include "intel_hdcp.h"
>  #include "intel_hotplug.h"
>  #include "intel_opregion.h"
>  #include "skl_watermark.h"
> @@ -129,7 +128,6 @@ static void xe_display_fini(void *arg)
>  	struct xe_device *xe = arg;
>  	struct intel_display *display = xe->display;
> 
> -	intel_hdcp_component_fini(display);
>  	intel_audio_deinit(display);
>  	intel_display_driver_remove(display);
>  }
> --
> 2.47.3