drm/radeon/ci_dpm: Remove redundant initialization of variables hi_sidd, lo_sidd
authorColin Ian King <colin.king@canonical.com>
Tue, 7 Sep 2021 11:09:13 +0000 (12:09 +0100)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 7 Sep 2021 17:30:50 +0000 (13:30 -0400)
The variables hi_sidd and lo_sidd are being initialized with a values
that are never read, they are being updated later on. The assignments
are 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/radeon/ci_dpm.c

index f0cfb58..ac006be 100644 (file)
@@ -390,8 +390,7 @@ static int ci_min_max_v_gnbl_pm_lid_from_bapm_vddc(struct radeon_device *rdev)
 static int ci_populate_bapm_vddc_base_leakage_sidd(struct radeon_device *rdev)
 {
        struct ci_power_info *pi = ci_get_pi(rdev);
-       u16 hi_sidd = pi->smc_powertune_table.BapmVddCBaseLeakageHiSidd;
-       u16 lo_sidd = pi->smc_powertune_table.BapmVddCBaseLeakageLoSidd;
+       u16 hi_sidd, lo_sidd;
        struct radeon_cac_tdp_table *cac_tdp_table =
                rdev->pm.dpm.dyn_state.cac_tdp_table;