<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Feb 27, 2013 at 10:46 AM, Paulo Zanoni <span dir="ltr"><<a href="mailto:przanoni@gmail.com" target="_blank">przanoni@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi<br>
<br>
2013/2/25 Rodrigo Vivi <<a href="mailto:rodrigo.vivi@gmail.com">rodrigo.vivi@gmail.com</a>>:<br>
<div class="im">> This is the last cleaning up patch for HSW, letting render standby<br>
> programming sequence like the documented one at HSW PM programing guide.<br>
><br>
> Signed-off-by: Rodrigo Vivi <<a href="mailto:rodrigo.vivi@gmail.com">rodrigo.vivi@gmail.com</a>><br>
> ---<br>
>  drivers/gpu/drm/i915/intel_pm.c | 9 ---------<br>
>  1 file changed, 9 deletions(-)<br>
><br>
> diff --git a/drivers/gpu/drm/i915/intel_pm.c b/drivers/gpu/drm/i915/intel_pm.c<br>
> index d43e011..ef51174 100644<br>
> --- a/drivers/gpu/drm/i915/intel_pm.c<br>
> +++ b/drivers/gpu/drm/i915/intel_pm.c<br>
> @@ -2749,16 +2749,7 @@ static void hsw_enable_rps(struct drm_device *dev)<br>
>                    GEN6_RP_UP_BUSY_AVG |<br>
>                    GEN7_RP_DOWN_IDLE_AVG);<br>
><br>
> -       gen6_set_rps(dev_priv->dev, (gt_perf_status & 0xff00) >> 8);<br>
<br>
</div>Looks like the line above is not even on the SNB docs, and git blame<br>
shows it's a bug fix. I'll let Chris comment on that. Having a nice<br>
comment explaining why that code is there would be good :)<br></blockquote><div><br></div><div style>Any comment Chris? what is this for? is this still needed for HSW?</div><div style> <br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im"><br>
> -<br>
> -       /* requires MSI enabled */<br>
>         I915_WRITE(GEN6_PMIER, GEN7_PM_DEFERRED_EVENTS);<br>
> -       spin_lock_irq(&dev_priv->rps.lock);<br>
> -       WARN_ON(dev_priv->rps.pm_iir != 0);<br>
> -       I915_WRITE(GEN6_PMIMR, 0);<br>
> -       spin_unlock_irq(&dev_priv->rps.lock);<br>
> -       /* enable all PM interrupts */<br>
> -       I915_WRITE(GEN6_PMINTRMSK, 0);<br>
<br>
</div>This is all part of the interrupt enabling: just changing PMIER is not<br>
enough, you have to change the masks and possibly clear the IIR<br>
register.<br></blockquote><div><br></div><div style>Why?</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
><br>
>         gen6_gt_force_wake_put(dev_priv);<br>
>  }<br>
<span class="HOEnZb"><font color="#888888">> --<br>
> 1.8.1.2<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>
<br>
<br>
--<br>
Paulo Zanoni<br>
</font></span></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>