Sorry, please ignore this patch as the new logic is wrong. Best Regards Rex
-----Original Message----- From: amd-gfx [mailto:[email protected]] On Behalf Of Rex Zhu Sent: Tuesday, January 02, 2018 3:54 PM To: [email protected] 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 <[email protected]> --- 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 [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx _______________________________________________ amd-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/amd-gfx
