<div dir="ltr">Hi Daniel,<div><br></div><div>All 4 patches in this series are reviewed and ready to merge,</div><div>could you please merge them?</div><div><br></div><div>Thanks,</div><div>Rodrigo.</div><br><div class="gmail_quote"><div dir="ltr">On Fri, Nov 13, 2015 at 10:42 AM Vivi, Rodrigo <<a href="mailto:rodrigo.vivi@intel.com">rodrigo.vivi@intel.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Fri, 2015-11-13 at 17:08 +0200, Ville Syrjälä wrote:<br>
> On Wed, Nov 11, 2015 at 11:37:06AM -0800, Rodrigo Vivi wrote:<br>
> > Let's split critical PSR fixes from the series that contains other<br>
> > reworks, stabilization and improvements.<br>
> ><br>
> > The second patch in this series isn't considered critical in terms<br>
> > of functionality, but it depends on the first one and it can be<br>
> > consider<br>
> > a fix for PSR residency on VLV/CHV.<br>
><br>
> FYI I recently glanced at the psr code and a few things that left me<br>
> scratching my head:<br>
<br>
Thanks for spotting this.<br>
<br>
> - hsw_psr_enable_sink() frobs at the AUX registers without holding<br>
> the hw_mutex.<br>
>   On SKL+ it seems to use the normal AUX registers here. Before it<br>
> used<br>
>   the special PSR registers, so that may have been OK, but the SKL+<br>
>   thing seems rather questionable.<br>
<br>
Yes, I agree. I'll take a look.<br>
<br>
> - intel_psr_enable() calls intel_psr_activate() on SKL+ but not on<br>
>   HSW/BDW. I'm thinking there should be a comment there to make it<br>
> clear<br>
>   why, if it's even correct.<br>
<br>
Indeed. Also Durga when reviewing mentioned it would be good to make<br>
only worker calling _activate(). So I will follow-up with a patch to<br>
fix this.<br>
<br>
<br>
><br>
_______________________________________________<br>
Intel-gfx mailing list<br>
<a href="mailto:Intel-gfx@lists.freedesktop.org" target="_blank">Intel-gfx@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/intel-gfx" rel="noreferrer" target="_blank">http://lists.freedesktop.org/mailman/listinfo/intel-gfx</a><br>
</blockquote></div></div>