On Thu, 2018-09-20 at 13:43 -0700, José Roberto de Souza wrote:
> ALPM is a requirement and we don't need to keep it's cached, what
> were done in commit 97c9de66ca80
> ("drm/i915/psr: Fix ALPM cap check for PSR2") but the alpm was not
> removed from i915_psr.:
You're right.Reviewed-by: Dhinakaran Pandiyan <[email protected]> > > Cc: Dhinakaran Pandiyan <[email protected]> > Signed-off-by: José Roberto de Souza <[email protected]> > --- > drivers/gpu/drm/i915/i915_drv.h | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/i915_drv.h > b/drivers/gpu/drm/i915/i915_drv.h > index 41f1082da122..4ed129cf4d12 100644 > --- a/drivers/gpu/drm/i915/i915_drv.h > +++ b/drivers/gpu/drm/i915/i915_drv.h > @@ -630,7 +630,6 @@ struct i915_psr { We should rename this intel_psr and move it to the same file where struct intel_dp lives. > bool sink_psr2_support; > bool link_standby; > bool colorimetry_support; > - bool alpm; > bool psr2_enabled; And rename this too? The bool is set to enable_psr2 and does not mean PSR2 is enabled. > u8 sink_sync_latency; > ktime_t last_entry_attempt; _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
