Re: [Intel-gfx] [PATCH] drm/i915/dp/mst: Fix MST logic in intel_dp_long_pulse()

2016-04-08 Thread Ander Conselvan De Oliveira
On Thu, 2016-04-07 at 09:20 -0700, Jim Bride wrote: > On Thu, Apr 07, 2016 at 11:15:38AM +0300, Ander Conselvan De Oliveira wrote: > > On Wed, 2016-04-06 at 16:00 -0700, Jim Bride wrote: > > > In commit 7d23e3c3 ("drm/i915: Cleaning up intel_dp_hpd_pulse") some > > > much needed clean-up was done,

Re: [Intel-gfx] [PATCH] drm/i915/dp/mst: Fix MST logic in intel_dp_long_pulse()

2016-04-07 Thread Jim Bride
On Thu, Apr 07, 2016 at 11:15:38AM +0300, Ander Conselvan De Oliveira wrote: > On Wed, 2016-04-06 at 16:00 -0700, Jim Bride wrote: > > In commit 7d23e3c3 ("drm/i915: Cleaning up intel_dp_hpd_pulse") some > > much needed clean-up was done, but unfortunately part of the change > > broke DP MST. The

Re: [Intel-gfx] [PATCH] drm/i915/dp/mst: Fix MST logic in intel_dp_long_pulse()

2016-04-07 Thread Ander Conselvan De Oliveira
On Wed, 2016-04-06 at 16:00 -0700, Jim Bride wrote: > In commit 7d23e3c3 ("drm/i915: Cleaning up intel_dp_hpd_pulse") some > much needed clean-up was done, but unfortunately part of the change > broke DP MST. The real issue was setting the connector state to > disconnected in the MST case, which i

[Intel-gfx] [PATCH] drm/i915/dp/mst: Fix MST logic in intel_dp_long_pulse()

2016-04-06 Thread Jim Bride
In commit 7d23e3c3 ("drm/i915: Cleaning up intel_dp_hpd_pulse") some much needed clean-up was done, but unfortunately part of the change broke DP MST. The real issue was setting the connector state to disconnected in the MST case, which is good, but the code then (after a goto) checks if the conne