diff options
| author | Dhinakaran Pandiyan <[email protected]> | 2018-07-18 17:19:43 +0000 |
|---|---|---|
| committer | Dhinakaran Pandiyan <[email protected]> | 2018-07-26 07:32:44 +0000 |
| commit | 65172699a8bd9956705f71fb8b66b1068a1bb5cd (patch) | |
| tree | ee326da62a1b0248d983646f1a96fc00247323de /drivers/gpu/drm/i915/intel_dp_mst.c | |
| parent | drm/i915/mst: Do not retrain new links (diff) | |
| download | kernel-65172699a8bd9956705f71fb8b66b1068a1bb5cd.tar.gz kernel-65172699a8bd9956705f71fb8b66b1068a1bb5cd.zip | |
drm/i915/mst: Continue state updates even if AUX writes fail.
We are too late in the enabling sequence to back out cleanly, not updating
state tracking variables, like intel_dp->active_mst_links in this
instance, results in incorrect behaviour further along.
v2: Fixed int v/s bool comparison
Cc: Ville Syrjälä <[email protected]>
Cc: Rodrigo Vivi <[email protected]>
Cc: Nathan Ciobanu <[email protected]>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=107281
Signed-off-by: Dhinakaran Pandiyan <[email protected]>
Reviewed-by: Nathan Ciobanu <[email protected]>
Tested-by: Nathan Ciobanu <[email protected]>
Reviewed-by: Rodrigo Vivi <[email protected]>
Link: https://patchwork.freedesktop.org/patch/msgid/[email protected]
Diffstat (limited to 'drivers/gpu/drm/i915/intel_dp_mst.c')
| -rw-r--r-- | drivers/gpu/drm/i915/intel_dp_mst.c | 5 |
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/gpu/drm/i915/intel_dp_mst.c b/drivers/gpu/drm/i915/intel_dp_mst.c index 18c65f8e4fe8..352e5216cc65 100644 --- a/drivers/gpu/drm/i915/intel_dp_mst.c +++ b/drivers/gpu/drm/i915/intel_dp_mst.c @@ -241,11 +241,8 @@ static void intel_mst_pre_enable_dp(struct intel_encoder *encoder, connector->port, pipe_config->pbn, pipe_config->dp_m_n.tu); - if (ret == false) { + if (!ret) DRM_ERROR("failed to allocate vcpi\n"); - return; - } - intel_dp->active_mst_links++; temp = I915_READ(DP_TP_STATUS(port)); |