Commit 57a80115 authored by Kenneth Feng's avatar Kenneth Feng Committed by Alex Deucher
Browse files

drm/amd/pm: workaround for compute workload type on some skus 



On smu 13.0.0, the compute workload type cannot be set on all the skus
due to some other problems. This workaround is to make sure compute workload type
can also run on some specific skus.

v2: keep the variable consistent

Signed-off-by: default avatarKenneth Feng <kenneth.feng@amd.com>
Acked-by: default avatarLijo Lazar <lijo.lazar@amd.com>
Reviewed-by: default avatarFeifei Xu <Feifei.Xu@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 389c6b3e

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

+31 −2
Original line number Diff line number Diff line
@@ -2179,10 +2179,39 @@ static int smu_v13_0_0_set_power_profile_mode(struct smu_context *smu, 
		}

	}



	if (smu->power_profile_mode == PP_SMC_POWER_PROFILE_COMPUTE &&

		(((smu->adev->pdev->device == 0x744C) && (smu->adev->pdev->revision == 0xC8)) ||

		((smu->adev->pdev->device == 0x744C) && (smu->adev->pdev->revision == 0xCC)))) {

		ret = smu_cmn_update_table(smu,

					   SMU_TABLE_ACTIVITY_MONITOR_COEFF,

					   WORKLOAD_PPLIB_COMPUTE_BIT,

					   (void *)(&activity_monitor_external),

					   false);

		if (ret) {

			dev_err(smu->adev->dev, "[%s] Failed to get activity monitor!", __func__);

			return ret;

		}



		ret = smu_cmn_update_table(smu,

					   SMU_TABLE_ACTIVITY_MONITOR_COEFF,

					   WORKLOAD_PPLIB_CUSTOM_BIT,

					   (void *)(&activity_monitor_external),

					   true);

		if (ret) {

			dev_err(smu->adev->dev, "[%s] Failed to set activity monitor!", __func__);

			return ret;

		}



		workload_type = smu_cmn_to_asic_specific_index(smu,

						       CMN2ASIC_MAPPING_WORKLOAD,

						       PP_SMC_POWER_PROFILE_CUSTOM);

	} else {

		/* conv PP_SMC_POWER_PROFILE* to WORKLOAD_PPLIB_*_BIT */

		workload_type = smu_cmn_to_asic_specific_index(smu,

						       CMN2ASIC_MAPPING_WORKLOAD,

						       smu->power_profile_mode);

	}



	if (workload_type < 0)

		return -EINVAL;