On Wed, 2015-11-18 at 15:55 +0100, Daniel Vetter wrote: > On Mon, Nov 16, 2015 at 03:59:15PM +0000, Rodrigo Vivi wrote: > > Hi Daniel, > > > > could you please ignore patch 5 in this series and merge the 4 > > first > > patches. > > I merged an earlier instance of the first 4 patches. Has anything > changed? no changes. thanks for merging and sorry for the doubled noise! > Please double-check the right versions landed in dinq. just did. thanks! > > Thanks, Daniel > > > > > The aux mutex is really unreliable and doesn't help on all SKLs so > > better > > to just ignore that. > > > > Thanks, > > Rodrigo. > > > > On Thu, Nov 12, 2015 at 11:03 AM Rodrigo Vivi < > > rodrigo.vivi@xxxxxxxxx> > > wrote: > > > > > On Thu, Nov 12, 2015 at 10:56 AM Vivi, Rodrigo < > > > rodrigo.vivi@xxxxxxxxx> > > > wrote: > > > > > > > On Thu, 2015-11-12 at 12:17 +0200, Jani Nikula wrote: > > > > > On Wed, 11 Nov 2015, Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> > > > > > wrote: > > > > > > Let's start spliting that big series that enables PSR with > > > > > > this > > > > > > sink crc > > > > > > stabilization. > > > > > > > > > > > > Also I'm adding Wayne's mutex that stabilizes sink CRC on > > > > > > Skylake. > > > > > > > > > > > > All patches already reviewed and ready to merge. > > > > > > > > > > Disagreed on the hw mutex patch. > > > > > > > > I wasn't considered it as nacked. But it seems that we have > > > > more to > > > > discuss around it. > > > > > > > > Although I don't see a reason of blocking a patch that follow > > > > spec, it > > > > is reviewed, and it is used to fix a bug, unblock validation > > > > and only > > > > used for the particular case that is just used on automated > > > > validation > > > > and not broadly to all aux communications. > > > > > > > > > > Oh, now I saw you had comments there on the patch. So please > > > ignore this > > > complain and accept my apologies. > > > > > > > > > > > > > > > > > > > > BR, > > > > > Jani. > > > > > > > > > > > > > > > > > Thank you very much Paulo for the review and Thank you > > > > > > Wayne for > > > > > > the > > > > > > SKL aux mutex. > > > > > > > > > > > > Thanks, > > > > > > Rodrigo. > > > > > > > > > > > > Boyer, Wayne (1): > > > > > > drm/i915/skl: implement DP Aux Mutex framework > > > > > > > > > > > > Rodrigo Vivi (4): > > > > > > drm/i915: Allow 1 vblank to let Sink CRC calculation to > > > > > > start or > > > > > > stop. > > > > > > drm/i915: Make Sink crc calculation waiting for counter > > > > > > to reset. > > > > > > drm/i915: Stop tracking last calculated Sink CRC. > > > > > > drm/i915: Rely on TEST_SINK_START instead of tracking > > > > > > Sink CRC > > > > > > state > > > > > > on dev_priv. > > > > > > > > > > > > drivers/gpu/drm/i915/i915_drv.h | 1 + > > > > > > drivers/gpu/drm/i915/i915_reg.h | 5 ++ > > > > > > drivers/gpu/drm/i915/intel_dp.c | 126 > > > > > > +++++++++++++++++++++++++++ > > > > > > ------------ > > > > > > drivers/gpu/drm/i915/intel_drv.h | 7 --- > > > > > > 4 files changed, 93 insertions(+), 46 deletions(-) > > > > > > > > > _______________________________________________ > > > > Intel-gfx mailing list > > > > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > > > > http://lists.freedesktop.org/mailman/listinfo/intel-gfx > > > > > > > > _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
