drm/amd/powerplay: fix save dpm level error for smu
authorKevin Wang <kevin1.wang@amd.com>
Fri, 12 Jul 2019 09:05:52 +0000 (17:05 +0800)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 16 Jul 2019 18:08:12 +0000 (13:08 -0500)
the save dpm level should be save previous dpm profile level,
should not modified by get dpm level function.
eg: default auto
1. auto -> standard ==> dpm_level = standard, save_dpm = auto.
2. standard -> auto ==> dpm_level = auto, save_dpm = standard.

Signed-off-by: Kevin Wang <kevin1.wang@amd.com>
Reviewed-by: Evan Quan <evan.quan@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/powerplay/amdgpu_smu.c

index e6f1039..8150cb8 100644 (file)
@@ -1448,17 +1448,16 @@ int smu_handle_task(struct smu_context *smu,
 enum amd_dpm_forced_level smu_get_performance_level(struct smu_context *smu)
 {
        struct smu_dpm_context *smu_dpm_ctx = &(smu->smu_dpm);
+       enum amd_dpm_forced_level level;
 
        if (!smu_dpm_ctx->dpm_context)
                return -EINVAL;
 
        mutex_lock(&(smu->mutex));
-       if (smu_dpm_ctx->dpm_level != smu_dpm_ctx->saved_dpm_level) {
-               smu_dpm_ctx->saved_dpm_level = smu_dpm_ctx->dpm_level;
-       }
+       level = smu_dpm_ctx->dpm_level;
        mutex_unlock(&(smu->mutex));
 
-       return smu_dpm_ctx->dpm_level;
+       return level;
 }
 
 int smu_force_performance_level(struct smu_context *smu, enum amd_dpm_forced_level level)