<div dir="ltr">Hi Daniel,<div><br></div><div>could you please ignore patch 5 in this series and merge the 4 first patches.</div><div><br></div><div>The aux mutex is really unreliable and doesn't help on all SKLs so better to just ignore that.</div><div><br></div><div>Thanks,</div><div>Rodrigo. </div><br><div class="gmail_quote"><div dir="ltr">On Thu, Nov 12, 2015 at 11:03 AM Rodrigo Vivi <<a href="mailto:rodrigo.vivi@gmail.com">rodrigo.vivi@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Thu, Nov 12, 2015 at 10:56 AM Vivi, Rodrigo <<a href="mailto:rodrigo.vivi@intel.com" target="_blank">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 Thu, 2015-11-12 at 12:17 +0200, Jani Nikula wrote:<br>
> On Wed, 11 Nov 2015, Rodrigo Vivi <<a href="mailto:rodrigo.vivi@intel.com" target="_blank">rodrigo.vivi@intel.com</a>> wrote:<br>
> > Let's start spliting that big series that enables PSR with this<br>
> > sink crc<br>
> > stabilization.<br>
> ><br>
> > Also I'm adding Wayne's mutex that stabilizes sink CRC on Skylake.<br>
> ><br>
> > All patches already reviewed and ready to merge.<br>
><br>
> Disagreed on the hw mutex patch.<br>
<br>
I wasn't considered it as nacked. But it seems that we have more to<br>
discuss around it.<br>
<br>
Although I don't see a reason of blocking a patch that follow spec, it<br>
is reviewed, and it is used to fix a bug, unblock validation and only<br>
used for the particular case that is just used on automated validation<br>
and not broadly to all aux communications.<br></blockquote><div><br></div></div></div><div dir="ltr"><div class="gmail_quote"><div>Oh, now I saw you had comments there on the patch. So please ignore this complain and accept my apologies.</div></div></div><div dir="ltr"><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
><br>
> BR,<br>
> Jani.<br>
><br>
> ><br>
> > Thank you very much Paulo for the review and Thank you Wayne for<br>
> > the<br>
> > SKL aux mutex.<br>
> ><br>
> > Thanks,<br>
> > Rodrigo.<br>
> ><br>
> > Boyer, Wayne (1):<br>
> >   drm/i915/skl: implement DP Aux Mutex framework<br>
> ><br>
> > Rodrigo Vivi (4):<br>
> >   drm/i915: Allow 1 vblank to let Sink CRC calculation to start or<br>
> > stop.<br>
> >   drm/i915: Make Sink crc calculation waiting for counter to reset.<br>
> >   drm/i915: Stop tracking last calculated Sink CRC.<br>
> >   drm/i915: Rely on TEST_SINK_START instead of tracking Sink CRC<br>
> > state<br>
> >     on dev_priv.<br>
> ><br>
> >  drivers/gpu/drm/i915/i915_drv.h  |   1 +<br>
> >  drivers/gpu/drm/i915/i915_reg.h  |   5 ++<br>
> >  drivers/gpu/drm/i915/intel_dp.c  | 126 +++++++++++++++++++++++++++<br>
> > ------------<br>
> >  drivers/gpu/drm/i915/intel_drv.h |   7 ---<br>
> >  4 files changed, 93 insertions(+), 46 deletions(-)<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></blockquote></div></div>