Commit 61b396b9 authored by Alex Deucher's avatar Alex Deucher
Browse files

drm/amdgpu/pm/smu_v13.0: convert IP version checking



Use IP versions rather than asic_type to differentiate
IP version specific features.

Acked-by: default avatarChristian König <christian.koenig@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 6b726a0a
Loading
Loading
Loading
Loading
+13 −9
Original line number Diff line number Diff line
@@ -89,12 +89,13 @@ int smu_v13_0_init_microcode(struct smu_context *smu)
	if (amdgpu_sriov_vf(adev))
		return 0;

	switch (adev->asic_type) {
	case CHIP_ALDEBARAN:
	switch (adev->ip_versions[MP1_HWIP]) {
	case IP_VERSION(13, 0, 2):
		chip_name = "aldebaran";
		break;
	default:
		dev_err(adev->dev, "Unsupported ASIC type %d\n", adev->asic_type);
		dev_err(adev->dev, "Unsupported IP version 0x%x\n",
			adev->ip_versions[MP1_HWIP]);
		return -EINVAL;
	}

@@ -210,15 +211,17 @@ int smu_v13_0_check_fw_version(struct smu_context *smu)
	smu_minor = (smu_version >> 8) & 0xff;
	smu_debug = (smu_version >> 0) & 0xff;

	switch (smu->adev->asic_type) {
	case CHIP_ALDEBARAN:
	switch (smu->adev->ip_versions[MP1_HWIP]) {
	case IP_VERSION(13, 0, 2):
		smu->smc_driver_if_version = SMU13_DRIVER_IF_VERSION_ALDE;
		break;
	case CHIP_YELLOW_CARP:
	case IP_VERSION(13, 0, 1):
	case IP_VERSION(13, 0, 3):
		smu->smc_driver_if_version = SMU13_DRIVER_IF_VERSION_YELLOW_CARP;
		break;
	default:
		dev_err(smu->adev->dev, "smu unsupported asic type:%d.\n", smu->adev->asic_type);
		dev_err(smu->adev->dev, "smu unsupported IP version: 0x%x.\n",
			smu->adev->ip_versions[MP1_HWIP]);
		smu->smc_driver_if_version = SMU13_DRIVER_IF_VERSION_INV;
		break;
	}
@@ -740,8 +743,9 @@ int smu_v13_0_gfx_off_control(struct smu_context *smu, bool enable)
	int ret = 0;
	struct amdgpu_device *adev = smu->adev;

	switch (adev->asic_type) {
	case CHIP_YELLOW_CARP:
	switch (adev->ip_versions[MP1_HWIP]) {
	case IP_VERSION(13, 0, 1):
	case IP_VERSION(13, 0, 3):
		if (!(adev->pm.pp_feature & PP_GFXOFF_MASK))
			return 0;
		if (enable)