> -----Original Message----- > From: Chris Wilson [mailto:chris@xxxxxxxxxxxxxxxxxx] > Sent: Thursday, February 26, 2015 12:51 PM > To: Kahola, Mika > Cc: intel-gfx@xxxxxxxxxxxxxxxxxxxxx > Subject: Re: [PATCH] drm/i915: DP link training optimization > > On Thu, Feb 26, 2015 at 11:26:10AM +0200, Mika Kahola wrote: > > In a case when DP link has been once trained we can reuse the existing > > link training parameters i.e. voltage swing and pre-emphasis levels > > from cache when there is a need to restart link training. In a case of > > eDP we initially try to train the link by using the parameter set that > > we can find from VBT. The fallback on both cases is to reset the link > > training parameters and restart. > > I don't think the fallback from bad VBT values works as on failure they just > get reset. Care to enlighten me? > -Chris > > -- > Chris Wilson, Intel Open Source Technology Centre Thank you for the comment! This was my first patch to gfx development so all comments are welcomed. The idea that I had in mind was to initially start DP link training with the parameters that are stored in VBT. If we are unlucky and the link cannot be established at once by applying those initial parameters the parameters are reset to zero and link training is restarted. I do agree that the fallback in this case is unnecessary if VBT values are reset to zero in case of a failure. Cheers, Mika _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
