[Intel-gfx] [PATCH 2/5] drm/i915/dp: Restrict link retrain workaround to external monitors

2018-09-18 Thread Dhinakaran Pandiyan
commit '3cf71bc9904d ("drm/i915: Re-apply "Perform link quality check, unconditionally during long pulse"")' applies a work around for monitors that don't signal link loss. Limit this only for external displays as eDP features like PSR when active will have the link turned off and the driver ends u

Re: [Intel-gfx] [PATCH 2/5] drm/i915/dp: Restrict link retrain workaround to external monitors

2018-09-18 Thread Lyude Paul
Did the patch this references get pushed? I saw it fly by and I thought we had decided not to pull it in On Tue, 2018-09-18 at 00:20 -0700, Dhinakaran Pandiyan wrote: > commit '3cf71bc9904d ("drm/i915: Re-apply "Perform link quality check, > unconditionally during long pulse"")' applies a work aro

Re: [Intel-gfx] [PATCH 2/5] drm/i915/dp: Restrict link retrain workaround to external monitors

2018-09-18 Thread Rodrigo Vivi
On Tue, Sep 18, 2018 at 11:33:24AM -0400, Lyude Paul wrote: > Did the patch this references get pushed? I saw it fly by and I thought we had > decided not to pull it in ops, I created the confusion. Sorry. What I decided that week was to not pull to drm-intel-fixes until Ville or Jani acking. But

Re: [Intel-gfx] [PATCH 2/5] drm/i915/dp: Restrict link retrain workaround to external monitors

2018-09-19 Thread Pandiyan, Dhinakaran
On Tue, 2018-09-18 at 08:50 -0700, Rodrigo Vivi wrote: > On Tue, Sep 18, 2018 at 11:33:24AM -0400, Lyude Paul wrote: > > Did the patch this references get pushed? I saw it fly by and I > > thought we had > > decided not to pull it in > > ops, I created the confusion. Sorry. > > What I decided tha

Re: [Intel-gfx] [PATCH 2/5] drm/i915/dp: Restrict link retrain workaround to external monitors

2018-09-19 Thread Lyude Paul
SGTM On Wed, 2018-09-19 at 17:58 +, Pandiyan, Dhinakaran wrote: > On Tue, 2018-09-18 at 08:50 -0700, Rodrigo Vivi wrote: > > On Tue, Sep 18, 2018 at 11:33:24AM -0400, Lyude Paul wrote: > > > Did the patch this references get pushed? I saw it fly by and I > > > thought we had > > > decided not