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

Hogander, Jouni jouni.hogander at intel.com
Tue Sep 19 11:15:56 UTC 2023


On Mon, 2023-09-18 at 10:29 +0300, Hogander, Jouni wrote:
> 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

Thank you Vinod for the patch. This is now merged.

BR,

Jouni Högander

> 
> 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