drm/i915: remove redundant assignment to variable test_result
authorColin Ian King <colin.king@canonical.com>
Fri, 17 Apr 2020 16:08:29 +0000 (17:08 +0100)
committerChris Wilson <chris@chris-wilson.co.uk>
Sat, 18 Apr 2020 06:47:22 +0000 (07:47 +0100)
The variable test_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>
Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
Signed-off-by: Chris Wilson <chris@chris-wilson.co.uk>
Link: https://patchwork.freedesktop.org/patch/msgid/20200417160829.112776-1-colin.king@canonical.com
drivers/gpu/drm/i915/display/intel_dp.c

index 857a36f..a71d291 100644 (file)
@@ -5563,7 +5563,7 @@ void intel_dp_process_phy_request(struct intel_dp *intel_dp)
 
 static u8 intel_dp_autotest_phy_pattern(struct intel_dp *intel_dp)
 {
-       u8 test_result = DP_TEST_NAK;
+       u8 test_result;
 
        test_result = intel_dp_prepare_phytest(intel_dp);
        if (test_result != DP_TEST_ACK)