disp: msm: dp: fix locking in hotplug processing

Return immediately after releasing the lock if the connect event
has already been processed.

Change-Id: I80552b4c569d436ec85a945f4ddc3d389e3af43f
Signed-off-by: Tatenda Chipeperekwa <tatendac@codeaurora.org>
This commit is contained in:
Tatenda Chipeperekwa
2019-09-09 13:13:58 -07:00
parent 0ce6ca8992
commit c11dcbf65e
2 changed files with 2 additions and 3 deletions

View File

@@ -851,8 +851,7 @@ static int dp_display_process_hpd_high(struct dp_display_private *dp)
if (dp_display_state_is(DP_STATE_CONNECTED)) { if (dp_display_state_is(DP_STATE_CONNECTED)) {
DP_DEBUG("dp already connected, skipping hpd high\n"); DP_DEBUG("dp already connected, skipping hpd high\n");
mutex_unlock(&dp->session_lock); mutex_unlock(&dp->session_lock);
rc = -EISCONN; return -EISCONN;
goto end;
} }
dp_display_state_add(DP_STATE_CONNECTED); dp_display_state_add(DP_STATE_CONNECTED);

View File

@@ -1122,7 +1122,7 @@ static int dp_link_process_link_status_update(struct dp_link_private *link)
return -EINVAL; return -EINVAL;
DP_DEBUG("channel_eq_done = %d, clock_recovery_done = %d\n", DP_DEBUG("channel_eq_done = %d, clock_recovery_done = %d\n",
drm_dp_clock_recovery_ok(link->link_status, drm_dp_channel_eq_ok(link->link_status,
link->dp_link.link_params.lane_count), link->dp_link.link_params.lane_count),
drm_dp_clock_recovery_ok(link->link_status, drm_dp_clock_recovery_ok(link->link_status,
link->dp_link.link_params.lane_count)); link->dp_link.link_params.lane_count));