drm/amd/display: remove redundant initialization of variable result
authorColin Ian King <colin.king@canonical.com>
Wed, 1 Jul 2020 14:59:05 +0000 (15:59 +0100)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 2 Jul 2020 16:02:58 +0000 (12:02 -0400)
The variable result is being initialized with a value that is never read
and it is being updated later with a new value.  The initialization is
redundant and can be removed.

Addresses-Coverity: ("Unused value")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/core/dc_link_hwss.c

index 1b3474a..d0a23b7 100644 (file)
@@ -151,7 +151,8 @@ bool edp_receiver_ready_T9(struct dc_link *link)
        unsigned int tries = 0;
        unsigned char sinkstatus = 0;
        unsigned char edpRev = 0;
-       enum dc_status result = DC_OK;
+       enum dc_status result;
+
        result = core_link_read_dpcd(link, DP_EDP_DPCD_REV, &edpRev, sizeof(edpRev));
 
      /* start from eDP version 1.2, SINK_STAUS indicate the sink is ready.*/
@@ -177,7 +178,7 @@ bool edp_receiver_ready_T7(struct dc_link *link)
 {
        unsigned char sinkstatus = 0;
        unsigned char edpRev = 0;
-       enum dc_status result = DC_OK;
+       enum dc_status result;
 
        /* use absolute time stamp to constrain max T7*/
        unsigned long long enter_timestamp = 0;