Re: [PATCH 0/5] Sink CRC stabilization

["Vivi, Rodrigo" <rodrigo.vivi@intel.com>]

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@gmail.com>
> > wrote:
> > 
> > > On Thu, Nov 12, 2015 at 10:56 AM Vivi, Rodrigo <
> > > rodrigo.vivi@intel.com>
> > > wrote:
> > > 
> > > > On Thu, 2015-11-12 at 12:17 +0200, Jani Nikula wrote:
> > > > > On Wed, 11 Nov 2015, Rodrigo Vivi <rodrigo.vivi@intel.com> 
> > > > > 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@lists.freedesktop.org
> > > > http://lists.freedesktop.org/mailman/listinfo/intel-gfx
> > > > 
> > > 
> 
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx