<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, Sep 4, 2014 at 12:55 AM, Ville Syrjälä <span dir="ltr"><<a href="mailto:ville.syrjala@linux.intel.com" target="_blank">ville.syrjala@linux.intel.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="">On Wed, Sep 03, 2014 at 10:49:56PM -0400, Rodrigo Vivi wrote:<br>

> With Software tracking we are going to PSR sooner than we should and staying<br>
> with blank screens in many cases.<br>
><br>
> Using 2 identical frames to detect idleness is safier.<br>
<br>
</div>This idle frame detection still depends of FBC right?<br></blockquote><div> </div><div style="font-family:arial,sans-serif;font-size:13px">not sure. and fbc is disabled anyway here on my tests.<br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

I believe if we want to go for full sw tracking on HSW/BDW we need to<br>
use the debug register to force PSR entry/exit.<br></blockquote><div style="font-size:13px;font-family:arial,sans-serif"><br class=""><br></div><div><span style="font-family:arial,sans-serif;font-size:13px">I tried this many times in different ways but never had success</span> </div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div class="im"><br>
><br>
> Discovered and validated with refactored igt/kms_sink_psr_crc.<br>
><br>
> Signed-off-by: Rodrigo Vivi <<a href="mailto:rodrigo.vivi@intel.com">rodrigo.vivi@intel.com</a>><br>
> ---<br>
>  drivers/gpu/drm/i915/intel_dp.c | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c<br>
> index f79473b..a796831 100644<br>
> --- a/drivers/gpu/drm/i915/intel_dp.c<br>
> +++ b/drivers/gpu/drm/i915/intel_dp.c<br>
> @@ -1813,7 +1813,7 @@ static void intel_edp_psr_enable_source(struct intel_dp *intel_dp)<br>
>       struct drm_device *dev = dig_port->base.base.dev;<br>
>       struct drm_i915_private *dev_priv = dev->dev_private;<br>
>       uint32_t max_sleep_time = 0x1f;<br>
> -     uint32_t idle_frames = 1;<br>
> +     uint32_t idle_frames = 2;<br>
>       uint32_t val = 0x0;<br>
>       const uint32_t link_entry_time = EDP_PSR_MIN_LINK_ENTRY_TIME_8_LINES;<br>
>       bool only_standby = false;<br>
> --<br>
> 1.9.3<br>
><br>
> _______________________________________________<br>
> Intel-gfx mailing list<br>
> <a href="mailto:Intel-gfx@lists.freedesktop.org">Intel-gfx@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/intel-gfx" target="_blank">http://lists.freedesktop.org/mailman/listinfo/intel-gfx</a><br>
<br>
</div><span class=""><font color="#888888">--<br>
Ville Syrjälä<br>
Intel OTC<br>
</font></span><div class=""><div class="h5">_______________________________________________<br>
Intel-gfx mailing list<br>
<a href="mailto:Intel-gfx@lists.freedesktop.org">Intel-gfx@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/intel-gfx" target="_blank">http://lists.freedesktop.org/mailman/listinfo/intel-gfx</a><br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Rodrigo Vivi</div><div>Blog: <a href="http://blog.vivi.eng.br" target="_blank">http://blog.vivi.eng.br</a></div><div> </div>
</div></div>