drm/amd/display: add event type check before restart the authentication
authorXiaodong Yan <Xiaodong.Yan@amd.com>
Fri, 6 Dec 2019 10:43:53 +0000 (18:43 +0800)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 7 Jan 2020 17:13:42 +0000 (12:13 -0500)
[Why]
Some combined docks will always trigger CP_IRQ but there's nothing the driver
needs to take care of, but the CP_IRQ breaks the original hdcp state and
triggers the driver to restart the authentication.

[How]
Add the event type check before restart the authentication or resend the stream
management

Signed-off-by: Xiaodong Yan <Xiaodong.Yan@amd.com>
Reviewed-by: Wenjing Liu <Wenjing.Liu@amd.com>
Acked-by: Harry Wentland <harry.wentland@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/modules/hdcp/hdcp2_transition.c

index da19073..8cae3e3 100644 (file)
@@ -630,7 +630,10 @@ enum mod_hdcp_status mod_hdcp_hdcp2_dp_transition(struct mod_hdcp *hdcp,
                        break;
                } else if (input->prepare_stream_manage != PASS ||
                                input->stream_manage_write != PASS) {
-                       fail_and_restart_in_ms(0, &status, output);
+                       if (event_ctx->event == MOD_HDCP_EVENT_CALLBACK)
+                               fail_and_restart_in_ms(0, &status, output);
+                       else
+                               increment_stay_counter(hdcp);
                        break;
                }
                callback_in_ms(100, output);
@@ -655,10 +658,12 @@ enum mod_hdcp_status mod_hdcp_hdcp2_dp_transition(struct mod_hdcp *hdcp,
                         */
                        if (hdcp->auth.count.stream_management_retry_count > 10) {
                                fail_and_restart_in_ms(0, &status, output);
-                       } else {
+                       } else if (event_ctx->event == MOD_HDCP_EVENT_CALLBACK) {
                                hdcp->auth.count.stream_management_retry_count++;
                                callback_in_ms(0, output);
                                set_state_id(hdcp, output, D2_A9_SEND_STREAM_MANAGEMENT);
+                       } else {
+                               increment_stay_counter(hdcp);
                        }
                        break;
                }