Please ignore this patch for now. On SDP platforms VBT isn't set so we get a false positive on this value causing frozen screens. On Fri, Nov 14, 2014 at 8:52 AM, Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> wrote: > Let's always skip aux on exit unless specified at VBT we need it. > > Signed-off-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> > --- > drivers/gpu/drm/i915/intel_psr.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/intel_psr.c b/drivers/gpu/drm/i915/intel_psr.c > index e706c9d..4cfe7a4 100644 > --- a/drivers/gpu/drm/i915/intel_psr.c > +++ b/drivers/gpu/drm/i915/intel_psr.c > @@ -180,11 +180,13 @@ static void intel_psr_enable_source(struct intel_dp *intel_dp) > val |= EDP_PSR_LINK_STANDBY; > val |= EDP_PSR_TP2_TP3_TIME_0us; > val |= EDP_PSR_TP1_TIME_0us; > - val |= EDP_PSR_SKIP_AUX_EXIT; > val |= IS_BROADWELL(dev) ? BDW_PSR_SINGLE_FRAME : 0; > } else > val |= EDP_PSR_LINK_DISABLE; > > + if (!dev_priv->vbt.psr.require_aux_wakeup) > + val |= EDP_PSR_SKIP_AUX_EXIT; > + > I915_WRITE(EDP_PSR_CTL(dev), val | > (IS_BROADWELL(dev) ? 0 : link_entry_time) | > max_sleep_time << EDP_PSR_MAX_SLEEP_TIME_SHIFT | > -- > 1.9.3 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Rodrigo Vivi Blog: http://blog.vivi.eng.br _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx