[Intel-xe] Revert "FIXME: drm/i915/hpd: skip intel_hpd_poll_fini() for non-display paths"

Hogander, Jouni jouni.hogander at intel.com
Mon Sep 18 07:29:33 UTC 2023


On Thu, 2023-09-14 at 17:54 +0300, Vinod Govindapillai wrote:
> This reverts commit 5b503f7b6aa156d452fae3c92d9060b531aa628c.
> 
> This can be reverted as the intel_hpd_poll_fini is called from
> display paths

I don't see reason for this either ->

Reviewed-by: Jouni Högander <jouni.hogander at intel.com>

> 
> Signed-off-by: Vinod Govindapillai <vinod.govindapillai at intel.com>
> ---
>  drivers/gpu/drm/i915/display/intel_display.c | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/drivers/gpu/drm/i915/display/intel_display.c
> b/drivers/gpu/drm/i915/display/intel_display.c
> index 9df8081f78d9..b2e16cb8d026 100644
> --- a/drivers/gpu/drm/i915/display/intel_display.c
> +++ b/drivers/gpu/drm/i915/display/intel_display.c
> @@ -8027,9 +8027,6 @@ void intel_hpd_poll_fini(struct
> drm_i915_private *i915)
>         struct intel_connector *connector;
>         struct drm_connector_list_iter conn_iter;
>  
> -       if (!HAS_DISPLAY(i915))
> -               return;
> -
>         /* Kill all the work that may have been queued by hpd. */
>         drm_connector_list_iter_begin(&i915->drm, &conn_iter);
>         for_each_intel_connector_iter(connector, &conn_iter) {



More information about the Intel-xe mailing list