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/pp_psm.c | |
parent | drm/amdgpu: fix userptr put_page handling (diff) | |
download | linux-dev-64d03abe6ee36ec48e997743e9397ae160eb508a.tar.xz linux-dev-64d03abe6ee36ec48e997743e9397ae160eb508a.zip |
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/pp_psm.c')
-rw-r--r-- | drivers/gpu/drm/amd/powerplay/hwmgr/pp_psm.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/pp_psm.c b/drivers/gpu/drm/amd/powerplay/hwmgr/pp_psm.c index 7656324957a8..167cdc321db2 100644 --- a/drivers/gpu/drm/amd/powerplay/hwmgr/pp_psm.c +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/pp_psm.c @@ -188,19 +188,19 @@ int psm_set_performance_states(struct pp_hwmgr *hwmgr) int psm_set_user_performance_state(struct pp_hwmgr *hwmgr, enum PP_StateUILabel label_id, - struct pp_power_state *state) + struct pp_power_state **state) { int table_entries; int i; table_entries = hwmgr->num_ps; - state = hwmgr->ps; + *state = hwmgr->ps; restart_search: for (i = 0; i < table_entries; i++) { - if (state->classification.ui_label & label_id) + if ((*state)->classification.ui_label & label_id) return 0; - state = (struct pp_power_state *)((unsigned long)state + hwmgr->ps_size); + *state = (struct pp_power_state *)((uintptr_t)*state + hwmgr->ps_size); } switch (label_id) { |