kms/nv50: always call drm_dp_dpcd_write after drm_dp_mst_hpd_irq

Submitted by Karol Herbst on Feb. 15, 2018, 1:18 p.m.

Details

Message ID 20180215131817.19290-1-kherbst@redhat.com
State New
Headers show
Series "kms/nv50: always call drm_dp_dpcd_write after drm_dp_mst_hpd_irq" ( rev: 1 ) in Nouveau

Not browsing as part of any series.

Commit Message

Karol Herbst Feb. 15, 2018, 1:18 p.m.
When using the dock for the Lenovo P50 laptop it could happen quite often that
on redocking the laptop no attached displays are found. Same situation when
displays are connected to the dock.

There seems to be no unhandled bits in the esi though.

With this patch detecting the displays doesn't seem to fail anymore on above
mentioned setup.

Maybe there is a better fix for this issue.

Signed-off-by: Karol Herbst <kherbst@redhat.com>
---
 drm/nouveau/nv50_display.c | 3 ---
 1 file changed, 3 deletions(-)

Patch hide | download patch | download mbox

diff --git a/drm/nouveau/nv50_display.c b/drm/nouveau/nv50_display.c
index dd8d4352..9e360ffa 100644
--- a/drm/nouveau/nv50_display.c
+++ b/drm/nouveau/nv50_display.c
@@ -3337,9 +3337,6 @@  nv50_mstm_service(struct nv50_mstm *mstm)
 		}
 
 		drm_dp_mst_hpd_irq(&mstm->mgr, esi, &handled);
-		if (!handled)
-			break;
-
 		drm_dp_dpcd_write(aux, DP_SINK_COUNT_ESI + 1, &esi[1], 3);
 	}
 }