Skip to content

Commit 62f38b4

Browse files
committed
drm/amdgpu/smu13: always apply the powersave optimization
It can avoid margin issues in some very demanding applications. Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/3618 Link: https://gitlab.freedesktop.org/drm/amd/-/issues/3131 Fixes: c50fe28 ("drm/amdgpu/swsmu: always force a state reprogram on init") Reviewed-by: Kenneth Feng <kenneth.feng@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
1 parent f9f3e82 commit 62f38b4

File tree

1 file changed

+10
-12
lines changed

1 file changed

+10
-12
lines changed

drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_0_ppt.c

Lines changed: 10 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -2542,18 +2542,16 @@ static int smu_v13_0_0_set_power_profile_mode(struct smu_context *smu,
25422542
workload_mask = 1 << workload_type;
25432543

25442544
/* Add optimizations for SMU13.0.0/10. Reuse the power saving profile */
2545-
if (smu->power_profile_mode == PP_SMC_POWER_PROFILE_COMPUTE) {
2546-
if ((amdgpu_ip_version(smu->adev, MP1_HWIP, 0) == IP_VERSION(13, 0, 0) &&
2547-
((smu->adev->pm.fw_version == 0x004e6601) ||
2548-
(smu->adev->pm.fw_version >= 0x004e7300))) ||
2549-
(amdgpu_ip_version(smu->adev, MP1_HWIP, 0) == IP_VERSION(13, 0, 10) &&
2550-
smu->adev->pm.fw_version >= 0x00504500)) {
2551-
workload_type = smu_cmn_to_asic_specific_index(smu,
2552-
CMN2ASIC_MAPPING_WORKLOAD,
2553-
PP_SMC_POWER_PROFILE_POWERSAVING);
2554-
if (workload_type >= 0)
2555-
workload_mask |= 1 << workload_type;
2556-
}
2545+
if ((amdgpu_ip_version(smu->adev, MP1_HWIP, 0) == IP_VERSION(13, 0, 0) &&
2546+
((smu->adev->pm.fw_version == 0x004e6601) ||
2547+
(smu->adev->pm.fw_version >= 0x004e7300))) ||
2548+
(amdgpu_ip_version(smu->adev, MP1_HWIP, 0) == IP_VERSION(13, 0, 10) &&
2549+
smu->adev->pm.fw_version >= 0x00504500)) {
2550+
workload_type = smu_cmn_to_asic_specific_index(smu,
2551+
CMN2ASIC_MAPPING_WORKLOAD,
2552+
PP_SMC_POWER_PROFILE_POWERSAVING);
2553+
if (workload_type >= 0)
2554+
workload_mask |= 1 << workload_type;
25572555
}
25582556

25592557
ret = smu_cmn_send_smc_msg_with_param(smu,

0 commit comments

Comments
 (0)