diff options
| author | Tom St Denis <tom.stdenis@amd.com> | 2017-09-05 07:55:48 -0400 |
|---|---|---|
| committer | Alex Deucher <alexander.deucher@amd.com> | 2017-09-12 14:23:18 -0400 |
| commit | 64d03abe6ee36ec48e997743e9397ae160eb508a (patch) | |
| tree | d8070fd7b1d40fa8f9091adbd99e4712a7a7b9cf /drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c | |
| parent | a216ab09955d6b77f3af4f0aba9255c5ddf382f5 (diff) | |
drm/amd/powerplay: Fix psm_set_user_performance_state()
We now pass a pointer to a pointer which seems to be
what they meant in the first place. The previous version
was modifying a pointer passed by value.
Fixes bug that was introduced by
commit 332798d40c2e91:drm/amd/powerplay: delete eventmgr layer in poweprlay
Signed-off-by: Tom St Denis <tom.stdenis@amd.com>
Reviewed-By: Rex Zhu <Rex.Zhu@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c')
| -rw-r--r-- | drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c b/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c index c6157bcdf7d6..4f1b932361b2 100644 --- a/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/hwmgr.c @@ -294,7 +294,7 @@ int hwmgr_handle_task(struct pp_instance *handle, enum amd_pp_task task_id, { enum amd_pm_state_type ps; enum PP_StateUILabel requested_ui_label; - struct pp_power_state *requested_ps; + struct pp_power_state *requested_ps = NULL; if (input == NULL) { ret = -EINVAL; @@ -303,7 +303,7 @@ int hwmgr_handle_task(struct pp_instance *handle, enum amd_pp_task task_id, ps = *(unsigned long *)input; requested_ui_label = power_state_convert(ps); - ret = psm_set_user_performance_state(hwmgr, requested_ui_label, requested_ps); + ret = psm_set_user_performance_state(hwmgr, requested_ui_label, &requested_ps); if (ret) return ret; ret = psm_adjust_power_state_dynamic(hwmgr, false, requested_ps); |