Commit e3746696 authored Feb 24, 2021 by Alex Deucher

drm/amdgpu/pm: make unsupported power profile messages debug

Making them an error confuses users and the errors are harmless
as not all asics support all profiles.

Bug: https://gitlab.freedesktop.org/drm/amd/-/issues/1488


Acked-by: Nirmoy Das <nirmoy.das@amd.com>
Reviewed-by: Evan Quan <evan.quan@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>

parent 0c61ac81

drivers/gpu/drm/amd/pm/swsmu/smu11/arcturus_ppt.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -1322,7 +1322,7 @@ static int arcturus_set_power_profile_mode(struct smu_context *smu,
		CMN2ASIC_MAPPING_WORKLOAD,
		profile_mode);
		if (workload_type < 0) {
		dev_err(smu->adev->dev, "Unsupported power profile mode %d on arcturus\n", profile_mode);
		dev_dbg(smu->adev->dev, "Unsupported power profile mode %d on arcturus\n", profile_mode);
		return -EINVAL;
		}

drivers/gpu/drm/amd/pm/swsmu/smu11/vangogh_ppt.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -810,7 +810,7 @@ static int vangogh_set_power_profile_mode(struct smu_context smu, long input,
		CMN2ASIC_MAPPING_WORKLOAD,
		profile_mode);
		if (workload_type < 0) {
		dev_err_once(smu->adev->dev, "Unsupported power profile mode %d on VANGOGH\n",
		dev_dbg(smu->adev->dev, "Unsupported power profile mode %d on VANGOGH\n",
		profile_mode);
		return -EINVAL;
		}

drivers/gpu/drm/amd/pm/swsmu/smu12/renoir_ppt.c

+1 −1

Original line number	Diff line number	Diff line
		@@ -844,7 +844,7 @@ static int renoir_set_power_profile_mode(struct smu_context smu, long input, u
		* TODO: If some case need switch to powersave/default power mode
		* then can consider enter WORKLOAD_COMPUTE/WORKLOAD_CUSTOM for power saving.
		*/
		dev_err_once(smu->adev->dev, "Unsupported power profile mode %d on RENOIR\n", profile_mode);
		dev_dbg(smu->adev->dev, "Unsupported power profile mode %d on RENOIR\n", profile_mode);
		return -EINVAL;
		}