drm/i915/hdcp: Fix config_stream_type() ret value
authorAnshuman Gupta <anshuman.gupta@intel.com>
Tue, 3 Mar 2020 07:38:38 +0000 (13:08 +0530)
committerRamalingam C <ramalingam.c@intel.com>
Wed, 4 Mar 2020 01:28:37 +0000 (06:58 +0530)
DP shim's config_stream_type considered to be succeeded when
return value of intel_dp_hdcp2_write_msg() equals to size of
message to be written, it makes config_stream_type to return
a zero success value in order to succeed the HDCP auth.

v2:
- config_stream_type() returns 0 on success. [Ram]

CC: Ramalingam C <ramalingam.c@intel.com>
Signed-off-by: Anshuman Gupta <anshuman.gupta@intel.com>
Reviewed-by: Ramalingam C <ramalingam.c@intel.com>
Signed-off-by: Ramalingam C <ramalingam.c@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20200303073838.25871-1-anshuman.gupta@intel.com
drivers/gpu/drm/i915/display/intel_dp.c

index c31f523..7bf2b68 100644 (file)
@@ -6448,6 +6448,7 @@ static
 int intel_dp_hdcp2_config_stream_type(struct intel_digital_port *intel_dig_port,
                                      bool is_repeater, u8 content_type)
 {
+       int ret;
        struct hdcp2_dp_errata_stream_type stream_type_msg;
 
        if (is_repeater)
@@ -6463,8 +6464,11 @@ int intel_dp_hdcp2_config_stream_type(struct intel_digital_port *intel_dig_port,
        stream_type_msg.msg_id = HDCP_2_2_ERRATA_DP_STREAM_TYPE;
        stream_type_msg.stream_type = content_type;
 
-       return intel_dp_hdcp2_write_msg(intel_dig_port, &stream_type_msg,
+       ret =  intel_dp_hdcp2_write_msg(intel_dig_port, &stream_type_msg,
                                        sizeof(stream_type_msg));
+
+       return ret < 0 ? ret : 0;
+
 }
 
 static