[Intel-gfx] [PATCH 29/39] drm/i915: Use b->irq_enable() as predicate for mock engine

Chris Wilson chris at chris-wilson.co.uk
Fri Jan 4 12:13:32 UTC 2019


Quoting Tvrtko Ursulin (2019-01-02 15:21:21)
> > diff --git a/drivers/gpu/drm/i915/intel_engine_cs.c b/drivers/gpu/drm/i915/intel_engine_cs.c
> > index 36177546f68b..7b80a087cc32 100644
> > --- a/drivers/gpu/drm/i915/intel_engine_cs.c
> > +++ b/drivers/gpu/drm/i915/intel_engine_cs.c
> > @@ -917,6 +917,9 @@ static bool ring_is_idle(struct intel_engine_cs *engine)
> >       intel_wakeref_t wakeref;
> >       bool idle = true;
> >   
> > +     if (I915_SELFTEST_ONLY(!engine->mmio_base))
> > +             return true;
> 
> Seems to make sense, but shouldn't the patch title be "Use 
> engine->mmio_base as predicate for mock engine hw access"?

But we use b->irq_enable for the important bit.
-Chris


More information about the Intel-gfx mailing list