Re: [Intel-gfx] [PATCH] drm/i915/dp: Reset the link params on HPD/connected boot/resume

2017-02-15 Thread Jani Nikula
On Tue, 14 Feb 2017, Manasi Navare wrote: > On Tue, Feb 14, 2017 at 09:08:25PM +0200, Ville Syrjälä wrote: >> On Tue, Feb 07, 2017 at 04:54:11PM -0800, Manasi Navare wrote: >> > The max link parameters should be set/reset only on HPD or >> > connected boot case or on

Re: [Intel-gfx] [PATCH] drm/i915/dp: Reset the link params on HPD/connected boot/resume

2017-02-14 Thread Manasi Navare
On Tue, Feb 14, 2017 at 09:08:25PM +0200, Ville Syrjälä wrote: > On Tue, Feb 07, 2017 at 04:54:11PM -0800, Manasi Navare wrote: > > The max link parameters should be set/reset only on HPD or > > connected boot case or on system resume. > > > > Add a flag reset_link_params to intel_dp to decide

Re: [Intel-gfx] [PATCH] drm/i915/dp: Reset the link params on HPD/connected boot/resume

2017-02-14 Thread Ville Syrjälä
On Tue, Feb 07, 2017 at 04:54:11PM -0800, Manasi Navare wrote: > The max link parameters should be set/reset only on HPD or > connected boot case or on system resume. > > Add a flag reset_link_params to intel_dp to decide when > to reset the max link parameters. This prevents the parameters >

Re: [Intel-gfx] [PATCH] drm/i915/dp: Reset the link params on HPD/connected boot/resume

2017-02-13 Thread Manasi Navare
Hi Ville, I implemented this based on our IRC discussion on resetting the link params only on HPD and not on every long pulse function call. This will avoid them getting reset during the fill_modes call which is exactly what is required duirng the link training fallback tow ork correctly. Could

[Intel-gfx] [PATCH] drm/i915/dp: Reset the link params on HPD/connected boot/resume

2017-02-07 Thread Manasi Navare
The max link parameters should be set/reset only on HPD or connected boot case or on system resume. Add a flag reset_link_params to intel_dp to decide when to reset the max link parameters. This prevents the parameters from getting reset/overwritten through all other connector->funcs->detect()