drm/amdgpu/powerplay: remove redundant assignment to variable baco_state
authorColin Ian King <colin.king@canonical.com>
Thu, 22 Aug 2019 13:09:48 +0000 (14:09 +0100)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 22 Aug 2019 22:25:21 +0000 (17:25 -0500)
Variable baco_state is initialized to a value that is never read and it is
re-assigned later. 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/powerplay/smu_v11_0.c

index 9750c4194808029341a101e71a64c048081fa8b0..e163ec6f4cb84a5449d88e3b0516baa99ac2425d 100644 (file)
@@ -1664,7 +1664,7 @@ static bool smu_v11_0_baco_is_support(struct smu_context *smu)
 static enum smu_baco_state smu_v11_0_baco_get_state(struct smu_context *smu)
 {
        struct smu_baco_context *smu_baco = &smu->smu_baco;
-       enum smu_baco_state baco_state = SMU_BACO_STATE_EXIT;
+       enum smu_baco_state baco_state;
 
        mutex_lock(&smu_baco->mutex);
        baco_state = smu_baco->state;