<div><br><div class="gmail_quote"><div dir="auto">On Wed, Sep 27, 2017 at 5:14 AM David Weinehall <<a href="mailto:david.weinehall@linux.intel.com">david.weinehall@linux.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 Tue, Aug 08, 2017 at 12:50:51PM -0700, Rodrigo Vivi wrote:<br>
> a long time ago I had agreed with Daniel that we would only add new<br>
> platforms after it was enabled by default on previous platforms.<br>
> a big reason for that is that we was willing to reduce the platforms<br>
> to validate and do better validation one by one before enabling.<br>
><br>
> However now I believe it would be beneficial to have that supported<br>
> added so we can get more brave people using in different platforms so<br>
> we could capture more corner cases before we enable it by default.<br>
> Also we can still enable by default one platform at time if needed.<br>
><br>
> So:<br>
><br>
> Acked-by: Rodrigo Vivi <<a href="mailto:rodrigo.vivi@intel.com" target="_blank">rodrigo.vivi@intel.com</a>><br>
><br>
> I also checked the spec to see if there was anything else new or<br>
> different for these platforms and didn't find anything so:<br>
><br>
> Reviewed-by: Rodrigo Vivi <<a href="mailto:rodrigo.vivi@intel.com" target="_blank">rodrigo.vivi@intel.com</a>><br>
><br>
> But let's wait a bit to merge to give Daniel or others a time to nack ;)<br>
<br>
An update: while testing revealed that our BXT-P RVP doesn't work with<br>
PSR, the GLK definitely does. CI would like to do PSR testing on GLK,<br>
which obviously isn't possible if PSR is reported as unsupported on GLK.<br>
<br>
Based on BSpec alone the PSR failure on BXT-P shouldn't be a<br>
Broxton/Apollo Lake issue, but rather an issue with the RVP board<br>
(or the panel), so I'd say that this patch still makes sense.</blockquote><div dir="auto"><br></div><div dir="auto">It would be very important if we could narrow down the issue on BXT. Panel?! Bios?! Missing Workaround? Different user space? <br></div><div dir="auto">One of the biggest problem with PSR is that when it works well in all machines we have and we enable it we end up finding someone in the community with a machine that does not work well. We have an opportunity to investigate and understand very well what are the issues on this BXT. We shouldn't loose track of it.</div><div dir="auto">And maybe adding that to CI we will be forced to record the bug! ;)</div><div dir="auto"><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
After all it only changes gen9lp to report that they *can* support PSR<br>
(thus allowing for testing of PSR on such platforms), it doesn't enable<br>
it by default.<br>
<br>
So I'd like to nudge once more that this patch be merged.</blockquote><div dir="auto"><br></div><div dir="auto">I agree. Let's add it. Also good to enable on CNL as well. If the panel that you have there on CNL that is on CI doesn't support it you are about to recurve some panels that does support PSR2. </div><div dir="auto"><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
Daniel, any objections?</blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
<br>
<br>
Kind regards, David Weinehall<br>
</blockquote></div></div><div dir="ltr">-- <br></div><div class="gmail_signature" data-smartmail="gmail_signature"><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>