|
[Public]
VDDGFX should be matched to VDDC (e.g, SMU7_MAX_LEVELS_VDDC).
Alex
From: amd-gfx <amd-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx> on behalf of Mario Limonciello <mario.limonciello@xxxxxxx>
Sent: Thursday, December 14, 2023 4:11 PM To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx <amd-gfx@xxxxxxxxxxxxxxxxxxxxx> Cc: Limonciello, Mario <Mario.Limonciello@xxxxxxx> Subject: [PATCH] drm/amd: Add missing definitions for `SMU_MAX_LEVELS_VDDGFX` It is reported that on a Topaz dGPU the kernel emits:
amdgpu: can't get the mac of 5 This is because there is no definition for max levels of VDDGFX declared for SMU71 or SMU7. There is however an unused definition of VDDNB. Use this to return the max levels for VDDGFX. Link: https://gitlab.freedesktop.org/drm/amd/-/issues/3049 Signed-off-by: Mario Limonciello <mario.limonciello@xxxxxxx> --- drivers/gpu/drm/amd/pm/powerplay/smumgr/ci_smumgr.c | 2 ++ drivers/gpu/drm/amd/pm/powerplay/smumgr/iceland_smumgr.c | 2 ++ 2 files changed, 4 insertions(+) diff --git a/drivers/gpu/drm/amd/pm/powerplay/smumgr/ci_smumgr.c b/drivers/gpu/drm/amd/pm/powerplay/smumgr/ci_smumgr.c index 9e4228232f02..c5bccd382196 100644 --- a/drivers/gpu/drm/amd/pm/powerplay/smumgr/ci_smumgr.c +++ b/drivers/gpu/drm/amd/pm/powerplay/smumgr/ci_smumgr.c @@ -2303,6 +2303,8 @@ static uint32_t ci_get_mac_definition(uint32_t value) return SMU7_MAX_LEVELS_VDDCI; case SMU_MAX_LEVELS_MVDD: return SMU7_MAX_LEVELS_MVDD; + case SMU_MAX_LEVELS_VDDGFX: + return SMU7_MAX_LEVELS_VDDNB; } pr_debug("can't get the mac of %x\n", value); diff --git a/drivers/gpu/drm/amd/pm/powerplay/smumgr/iceland_smumgr.c b/drivers/gpu/drm/amd/pm/powerplay/smumgr/iceland_smumgr.c index 97d9802fe673..c9115eaa63c4 100644 --- a/drivers/gpu/drm/amd/pm/powerplay/smumgr/iceland_smumgr.c +++ b/drivers/gpu/drm/amd/pm/powerplay/smumgr/iceland_smumgr.c @@ -2268,6 +2268,8 @@ static uint32_t iceland_get_mac_definition(uint32_t value) return SMU71_MAX_LEVELS_VDDCI; case SMU_MAX_LEVELS_MVDD: return SMU71_MAX_LEVELS_MVDD; + case SMU_MAX_LEVELS_VDDGFX: + return SMU71_MAX_LEVELS_VDDNB; } pr_warn("can't get the mac of %x\n", value); -- 2.34.1 |
