Quoting Mika Kuoppala (2018-08-03 14:59:58)
> Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> writes:
> > diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h
> > index e0f5999fff07..4b656f31fde9 100644
> > --- a/drivers/gpu/drm/i915/i915_reg.h
> > +++ b/drivers/gpu/drm/i915/i915_reg.h
> > @@ -8582,8 +8582,10 @@ enum {
> > #define GEN6_PM_RP_DOWN_THRESHOLD (1 << 4)
> > #define GEN6_PM_RP_UP_EI_EXPIRED (1 << 2)
> > #define GEN6_PM_RP_DOWN_EI_EXPIRED (1 << 1)
> > -#define GEN6_PM_RPS_EVENTS (GEN6_PM_RP_UP_THRESHOLD | \
> > - GEN6_PM_RP_DOWN_THRESHOLD | \
> > +#define GEN6_PM_RPS_EVENTS (GEN6_PM_RP_UP_EI_EXPIRED | \
> > + GEN6_PM_RP_UP_THRESHOLD | \
> > + GEN6_PM_RP_DOWN_EI_EXPIRED | \
> > + GEN6_PM_RP_DOWN_THRESHOLD | \
> > GEN6_PM_RP_DOWN_TIMEOUT)
>
> GEN6_PM_RPS_MASK ?
I think we are still missing a few, so events as a superset of
pm_rps_events seems like a reasonable minimum change :)
Bonus, add the ones we are missing.
-Chris
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
