Sorry, please ignore this patch as the new logic is wrong. Best Regards Rex -----Original Message----- From: amd-gfx [mailto:amd-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx] On Behalf Of Rex Zhu Sent: Tuesday, January 02, 2018 3:54 PM To: amd-gfx at lists.freedesktop.org Cc: Zhu, Rex Subject: [PATCH 3/3] drm/amd/pp: Refine pp_dpm_set_pp_table logic Disable avfs control before set user defined pp table. Change-Id: I0a63aa1def82b166d3f2aa542c3a190339270236 Signed-off-by: Rex Zhu <Rex.Zhu at amd.com> --- drivers/gpu/drm/amd/powerplay/amd_powerplay.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/drivers/gpu/drm/amd/powerplay/amd_powerplay.c b/drivers/gpu/drm/amd/powerplay/amd_powerplay.c index fa9d161..b6c18cf 100644 --- a/drivers/gpu/drm/amd/powerplay/amd_powerplay.c +++ b/drivers/gpu/drm/amd/powerplay/amd_powerplay.c @@ -828,16 +828,16 @@ static int pp_dpm_set_pp_table(void *handle, const char *buf, size_t size) hwmgr->soft_pp_table = hwmgr->hardcode_pp_table; mutex_unlock(&pp_handle->pp_lock); - ret = amd_powerplay_reset(handle); - if (ret) - return ret; - if (hwmgr->hwmgr_func->avfs_control) { ret = hwmgr->hwmgr_func->avfs_control(hwmgr, false); if (ret) return ret; } + ret = amd_powerplay_reset(handle); + if (ret) + return ret; + return 0; } -- 1.9.1 _______________________________________________ amd-gfx mailing list amd-gfx at lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/amd-gfx
