Lijo pointed out to me that
commit ed4efe426a49 ("drm/amd: Restore cached power limit during resume")
commit 796ff8a7e01b ("drm/amd: Restore cached manual clock settings during
resume")
both should be unnecessary because smu_restore_dpm_user_profile() already
restores them. However in looking at the code there is a case that isn't
properly restored - the FAST PPT limits supported by Van Gogh. The nuance
isn't immediately obvious because the limit variable is overloaded with
limit type.
v5:
* Re-order limit of zero handling.
* Drop first and last patch, which were already merged.
Mario Limonciello (4):
drm/amd: Stop overloading power limit with limit type
drm/amd: Remove second call to set_power_limit()
drm/amd: Save and restore all limit types
drm/amd: Drop calls to restore power limit and clock from smu_resume()
.../gpu/drm/amd/include/kgd_pp_interface.h | 2 +-
drivers/gpu/drm/amd/pm/amdgpu_dpm.c | 3 +-
drivers/gpu/drm/amd/pm/amdgpu_pm.c | 3 +-
drivers/gpu/drm/amd/pm/inc/amdgpu_dpm.h | 2 +-
.../gpu/drm/amd/pm/powerplay/amd_powerplay.c | 2 +-
drivers/gpu/drm/amd/pm/swsmu/amdgpu_smu.c | 59 ++++++++-----------
drivers/gpu/drm/amd/pm/swsmu/inc/amdgpu_smu.h | 3 +-
.../gpu/drm/amd/pm/swsmu/smu11/vangogh_ppt.c | 1 -
8 files changed, 31 insertions(+), 44 deletions(-)
--
2.51.0
