Skip to content

Commit 764a90e

Browse files
Timur Kristófalexdeucher
authored andcommitted
drm/amd/pm: Workaround SI powertune issue on Radeon 430 (v2)
Radeon 430 and 520 are OEM GPUs from 2016~2017 They have the same device id: 0x6611 and revision: 0x87 On the Radeon 430, powertune is buggy and throttles the GPU, never allowing it to reach its maximum SCLK. Work around this bug by raising the TDP limits we program to the SMC from 24W (specified by the VBIOS on Radeon 430) to 32W. Disabling powertune entirely is not a viable workaround, because it causes the Radeon 520 to heat up above 100 C, which I prefer to avoid. Additionally, revise the maximum SCLK limit. Considering the above issue, these GPUs never reached a high SCLK on Linux, and the workarounds were added before the GPUs were released, so the workaround likely didn't target these specifically. Use 780 MHz (the maximum SCLK according to the VBIOS on the Radeon 430). Note that the Radeon 520 VBIOS has a higher maximum SCLK: 905 MHz, but in practice it doesn't seem to perform better with the higher clock, only heats up more. v2: Move the workaround to si_populate_smc_tdp_limits. Fixes: 841686d ("drm/amdgpu: add SI DPM support (v4)") Reviewed-by: Alex Deucher <alexander.deucher@amd.com> Signed-off-by: Timur Kristóf <timur.kristof@gmail.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com> (cherry picked from commit 966d70f)
1 parent d507742 commit 764a90e

1 file changed

Lines changed: 12 additions & 1 deletion

File tree

  • drivers/gpu/drm/amd/pm/legacy-dpm

drivers/gpu/drm/amd/pm/legacy-dpm/si_dpm.c

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2281,6 +2281,12 @@ static int si_populate_smc_tdp_limits(struct amdgpu_device *adev,
22812281
if (ret)
22822282
return ret;
22832283

2284+
if (adev->pdev->device == 0x6611 && adev->pdev->revision == 0x87) {
2285+
/* Workaround buggy powertune on Radeon 430 and 520. */
2286+
tdp_limit = 32;
2287+
near_tdp_limit = 28;
2288+
}
2289+
22842290
smc_table->dpm2Params.TDPLimit =
22852291
cpu_to_be32(si_scale_power_for_smc(tdp_limit, scaling_factor) * 1000);
22862292
smc_table->dpm2Params.NearTDPLimit =
@@ -3463,10 +3469,15 @@ static void si_apply_state_adjust_rules(struct amdgpu_device *adev,
34633469
(adev->pdev->revision == 0x80) ||
34643470
(adev->pdev->revision == 0x81) ||
34653471
(adev->pdev->revision == 0x83) ||
3466-
(adev->pdev->revision == 0x87) ||
3472+
(adev->pdev->revision == 0x87 &&
3473+
adev->pdev->device != 0x6611) ||
34673474
(adev->pdev->device == 0x6604) ||
34683475
(adev->pdev->device == 0x6605)) {
34693476
max_sclk = 75000;
3477+
} else if (adev->pdev->revision == 0x87 &&
3478+
adev->pdev->device == 0x6611) {
3479+
/* Radeon 430 and 520 */
3480+
max_sclk = 78000;
34703481
}
34713482
}
34723483

0 commit comments

Comments
 (0)