Skip to content

L
linux
Commit 2a507105 authored by Rex Zhu's avatar Rex Zhu Committed by Alex Deucher
Browse files
Options

drm/amd/powerplay: add global PowerPlay mutex. 

Acked-by: default avatarChristian König <christian.koenig@amd.com>
Signed-off-by: default avatarRex Zhu <Rex.Zhu@amd.com>
Reviewed-by: default avatarAlex Deucher <alexander.deucher@amd.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 0d2c7569
Hide whitespace changes
Inline Side-by-side
Showing with 132 additions and 61 deletions
drivers/gpu/drm/amd/powerplay/amd_powerplay.c
View file @ 2a507105
...@@ -341,8 +341,9 @@ static int pp_dpm_force_performance_level(void *handle, ...@@ -341,8 +341,9 @@ static int pp_dpm_force_performance_level(void *handle,
return 0; return 0;
} }
mutex_lock(&pp_handle->pp_lock);
hwmgr->hwmgr_func->force_dpm_level(hwmgr, level); hwmgr->hwmgr_func->force_dpm_level(hwmgr, level);
mutex_unlock(&pp_handle->pp_lock);
return 0; return 0;
} }
...@@ -352,6 +353,7 @@ static enum amd_dpm_forced_level pp_dpm_get_performance_level( ...@@ -352,6 +353,7 @@ static enum amd_dpm_forced_level pp_dpm_get_performance_level(
struct pp_hwmgr *hwmgr; struct pp_hwmgr *hwmgr;
struct pp_instance *pp_handle = (struct pp_instance *)handle; struct pp_instance *pp_handle = (struct pp_instance *)handle;
int ret = 0; int ret = 0;
enum amd_dpm_forced_level level;
ret = pp_check(pp_handle); ret = pp_check(pp_handle);
...@@ -359,8 +361,10 @@ static enum amd_dpm_forced_level pp_dpm_get_performance_level( ...@@ -359,8 +361,10 @@ static enum amd_dpm_forced_level pp_dpm_get_performance_level(
return ret; return ret;
hwmgr = pp_handle->hwmgr; hwmgr = pp_handle->hwmgr;
mutex_lock(&pp_handle->pp_lock);
return hwmgr->dpm_level; level = hwmgr->dpm_level;
mutex_unlock(&pp_handle->pp_lock);
return level;
} }
static int pp_dpm_get_sclk(void *handle, bool low) static int pp_dpm_get_sclk(void *handle, bool low)
...@@ -380,8 +384,10 @@ static int pp_dpm_get_sclk(void *handle, bool low) ...@@ -380,8 +384,10 @@ static int pp_dpm_get_sclk(void *handle, bool low)
pr_info("%s was not implemented.\n", __func__); pr_info("%s was not implemented.\n", __func__);
return 0; return 0;
} }
mutex_lock(&pp_handle->pp_lock);
return hwmgr->hwmgr_func->get_sclk(hwmgr, low); ret = hwmgr->hwmgr_func->get_sclk(hwmgr, low);
mutex_unlock(&pp_handle->pp_lock);
return ret;
} }
static int pp_dpm_get_mclk(void *handle, bool low) static int pp_dpm_get_mclk(void *handle, bool low)
...@@ -401,8 +407,10 @@ static int pp_dpm_get_mclk(void *handle, bool low) ...@@ -401,8 +407,10 @@ static int pp_dpm_get_mclk(void *handle, bool low)
pr_info("%s was not implemented.\n", __func__); pr_info("%s was not implemented.\n", __func__);
return 0; return 0;
} }
mutex_lock(&pp_handle->pp_lock);
return hwmgr->hwmgr_func->get_mclk(hwmgr, low); ret = hwmgr->hwmgr_func->get_mclk(hwmgr, low);
mutex_unlock(&pp_handle->pp_lock);
return ret;
} }
static int pp_dpm_powergate_vce(void *handle, bool gate) static int pp_dpm_powergate_vce(void *handle, bool gate)
...@@ -422,8 +430,10 @@ static int pp_dpm_powergate_vce(void *handle, bool gate) ...@@ -422,8 +430,10 @@ static int pp_dpm_powergate_vce(void *handle, bool gate)
pr_info("%s was not implemented.\n", __func__); pr_info("%s was not implemented.\n", __func__);
return 0; return 0;
} }
mutex_lock(&pp_handle->pp_lock);
return hwmgr->hwmgr_func->powergate_vce(hwmgr, gate); ret = hwmgr->hwmgr_func->powergate_vce(hwmgr, gate);
mutex_unlock(&pp_handle->pp_lock);
return ret;
} }
static int pp_dpm_powergate_uvd(void *handle, bool gate) static int pp_dpm_powergate_uvd(void *handle, bool gate)
...@@ -443,8 +453,10 @@ static int pp_dpm_powergate_uvd(void *handle, bool gate) ...@@ -443,8 +453,10 @@ static int pp_dpm_powergate_uvd(void *handle, bool gate)
pr_info("%s was not implemented.\n", __func__); pr_info("%s was not implemented.\n", __func__);
return 0; return 0;
} }
mutex_lock(&pp_handle->pp_lock);
return hwmgr->hwmgr_func->powergate_uvd(hwmgr, gate); ret = hwmgr->hwmgr_func->powergate_uvd(hwmgr, gate);
mutex_unlock(&pp_handle->pp_lock);
return ret;
} }
static enum PP_StateUILabel power_state_convert(enum amd_pm_state_type state) static enum PP_StateUILabel power_state_convert(enum amd_pm_state_type state)
...@@ -472,7 +484,7 @@ static int pp_dpm_dispatch_tasks(void *handle, enum amd_pp_event event_id, ...@@ -472,7 +484,7 @@ static int pp_dpm_dispatch_tasks(void *handle, enum amd_pp_event event_id,
if (ret != 0) if (ret != 0)
return ret; return ret;
mutex_lock(&pp_handle->pp_lock);
switch (event_id) { switch (event_id) {
case AMD_PP_EVENT_DISPLAY_CONFIG_CHANGE: case AMD_PP_EVENT_DISPLAY_CONFIG_CHANGE:
ret = pem_handle_event(pp_handle->eventmgr, event_id, &data); ret = pem_handle_event(pp_handle->eventmgr, event_id, &data);
...@@ -498,6 +510,7 @@ static int pp_dpm_dispatch_tasks(void *handle, enum amd_pp_event event_id, ...@@ -498,6 +510,7 @@ static int pp_dpm_dispatch_tasks(void *handle, enum amd_pp_event event_id,
default: default:
break; break;
} }
mutex_unlock(&pp_handle->pp_lock);
return ret; return ret;
} }
...@@ -507,6 +520,7 @@ static enum amd_pm_state_type pp_dpm_get_current_power_state(void *handle) ...@@ -507,6 +520,7 @@ static enum amd_pm_state_type pp_dpm_get_current_power_state(void *handle)
struct pp_power_state *state; struct pp_power_state *state;
struct pp_instance *pp_handle = (struct pp_instance *)handle; struct pp_instance *pp_handle = (struct pp_instance *)handle;
int ret = 0; int ret = 0;
enum amd_pm_state_type pm_type;
ret = pp_check(pp_handle); ret = pp_check(pp_handle);
...@@ -518,21 +532,26 @@ static enum amd_pm_state_type pp_dpm_get_current_power_state(void *handle) ...@@ -518,21 +532,26 @@ static enum amd_pm_state_type pp_dpm_get_current_power_state(void *handle)
if (hwmgr->current_ps == NULL) if (hwmgr->current_ps == NULL)
return -EINVAL; return -EINVAL;
mutex_lock(&pp_handle->pp_lock);
state = hwmgr->current_ps; state = hwmgr->current_ps;
switch (state->classification.ui_label) { switch (state->classification.ui_label) {
case PP_StateUILabel_Battery: case PP_StateUILabel_Battery:
return POWER_STATE_TYPE_BATTERY; pm_type = POWER_STATE_TYPE_BATTERY;
case PP_StateUILabel_Balanced: case PP_StateUILabel_Balanced:
return POWER_STATE_TYPE_BALANCED; pm_type = POWER_STATE_TYPE_BALANCED;
case PP_StateUILabel_Performance: case PP_StateUILabel_Performance:
return POWER_STATE_TYPE_PERFORMANCE; pm_type = POWER_STATE_TYPE_PERFORMANCE;
default: default:
if (state->classification.flags & PP_StateClassificationFlag_Boot) if (state->classification.flags & PP_StateClassificationFlag_Boot)
return POWER_STATE_TYPE_INTERNAL_BOOT; pm_type = POWER_STATE_TYPE_INTERNAL_BOOT;
else else
return POWER_STATE_TYPE_DEFAULT; pm_type = POWER_STATE_TYPE_DEFAULT;
} }
mutex_unlock(&pp_handle->pp_lock);
return pm_type;
} }
static int pp_dpm_set_fan_control_mode(void *handle, uint32_t mode) static int pp_dpm_set_fan_control_mode(void *handle, uint32_t mode)
...@@ -552,8 +571,10 @@ static int pp_dpm_set_fan_control_mode(void *handle, uint32_t mode) ...@@ -552,8 +571,10 @@ static int pp_dpm_set_fan_control_mode(void *handle, uint32_t mode)
pr_info("%s was not implemented.\n", __func__); pr_info("%s was not implemented.\n", __func__);
return 0; return 0;
} }
mutex_lock(&pp_handle->pp_lock);
return hwmgr->hwmgr_func->set_fan_control_mode(hwmgr, mode); ret = hwmgr->hwmgr_func->set_fan_control_mode(hwmgr, mode);
mutex_unlock(&pp_handle->pp_lock);
return ret;
} }
static int pp_dpm_get_fan_control_mode(void *handle) static int pp_dpm_get_fan_control_mode(void *handle)
...@@ -573,8 +594,10 @@ static int pp_dpm_get_fan_control_mode(void *handle) ...@@ -573,8 +594,10 @@ static int pp_dpm_get_fan_control_mode(void *handle)
pr_info("%s was not implemented.\n", __func__); pr_info("%s was not implemented.\n", __func__);
return 0; return 0;
} }
mutex_lock(&pp_handle->pp_lock);
return hwmgr->hwmgr_func->get_fan_control_mode(hwmgr); ret = hwmgr->hwmgr_func->get_fan_control_mode(hwmgr);
mutex_unlock(&pp_handle->pp_lock);
return ret;
} }
static int pp_dpm_set_fan_speed_percent(void *handle, uint32_t percent) static int pp_dpm_set_fan_speed_percent(void *handle, uint32_t percent)
...@@ -594,8 +617,10 @@ static int pp_dpm_set_fan_speed_percent(void *handle, uint32_t percent) ...@@ -594,8 +617,10 @@ static int pp_dpm_set_fan_speed_percent(void *handle, uint32_t percent)
pr_info("%s was not implemented.\n", __func__); pr_info("%s was not implemented.\n", __func__);
return 0; return 0;
} }
mutex_lock(&pp_handle->pp_lock);
return hwmgr->hwmgr_func->set_fan_speed_percent(hwmgr, percent); ret = hwmgr->hwmgr_func->set_fan_speed_percent(hwmgr, percent);
mutex_unlock(&pp_handle->pp_lock);
return ret;
} }
static int pp_dpm_get_fan_speed_percent(void *handle, uint32_t *speed) static int pp_dpm_get_fan_speed_percent(void *handle, uint32_t *speed)
...@@ -616,7 +641,10 @@ static int pp_dpm_get_fan_speed_percent(void *handle, uint32_t *speed) ...@@ -616,7 +641,10 @@ static int pp_dpm_get_fan_speed_percent(void *handle, uint32_t *speed)
return 0; return 0;
} }
return hwmgr->hwmgr_func->get_fan_speed_percent(hwmgr, speed); mutex_lock(&pp_handle->pp_lock);
ret = hwmgr->hwmgr_func->get_fan_speed_percent(hwmgr, speed);
mutex_unlock(&pp_handle->pp_lock);
return ret;
} }
static int pp_dpm_get_fan_speed_rpm(void *handle, uint32_t *rpm) static int pp_dpm_get_fan_speed_rpm(void *handle, uint32_t *rpm)
...@@ -635,7 +663,10 @@ static int pp_dpm_get_fan_speed_rpm(void *handle, uint32_t *rpm) ...@@ -635,7 +663,10 @@ static int pp_dpm_get_fan_speed_rpm(void *handle, uint32_t *rpm)
if (hwmgr->hwmgr_func->get_fan_speed_rpm == NULL) if (hwmgr->hwmgr_func->get_fan_speed_rpm == NULL)
return -EINVAL; return -EINVAL;
return hwmgr->hwmgr_func->get_fan_speed_rpm(hwmgr, rpm); mutex_lock(&pp_handle->pp_lock);
ret = hwmgr->hwmgr_func->get_fan_speed_rpm(hwmgr, rpm);
mutex_unlock(&pp_handle->pp_lock);
return ret;
} }
static int pp_dpm_get_temperature(void *handle) static int pp_dpm_get_temperature(void *handle)
...@@ -655,8 +686,10 @@ static int pp_dpm_get_temperature(void *handle) ...@@ -655,8 +686,10 @@ static int pp_dpm_get_temperature(void *handle)
pr_info("%s was not implemented.\n", __func__); pr_info("%s was not implemented.\n", __func__);
return 0; return 0;
} }
mutex_lock(&pp_handle->pp_lock);
return hwmgr->hwmgr_func->get_temperature(hwmgr); ret = hwmgr->hwmgr_func->get_temperature(hwmgr);
mutex_unlock(&pp_handle->pp_lock);
return ret;
} }
static int pp_dpm_get_pp_num_states(void *handle, static int pp_dpm_get_pp_num_states(void *handle,
...@@ -677,6 +710,8 @@ static int pp_dpm_get_pp_num_states(void *handle, ...@@ -677,6 +710,8 @@ static int pp_dpm_get_pp_num_states(void *handle,
if (hwmgr->ps == NULL) if (hwmgr->ps == NULL)
return -EINVAL; return -EINVAL;
mutex_lock(&pp_handle->pp_lock);
data->nums = hwmgr->num_ps; data->nums = hwmgr->num_ps;
for (i = 0; i < hwmgr->num_ps; i++) { for (i = 0; i < hwmgr->num_ps; i++) {
...@@ -699,7 +734,7 @@ static int pp_dpm_get_pp_num_states(void *handle, ...@@ -699,7 +734,7 @@ static int pp_dpm_get_pp_num_states(void *handle,
data->states[i] = POWER_STATE_TYPE_DEFAULT; data->states[i] = POWER_STATE_TYPE_DEFAULT;
} }
} }
mutex_unlock(&pp_handle->pp_lock);
return 0; return 0;
} }
...@@ -708,6 +743,7 @@ static int pp_dpm_get_pp_table(void *handle, char **table) ...@@ -708,6 +743,7 @@ static int pp_dpm_get_pp_table(void *handle, char **table)
struct pp_hwmgr *hwmgr; struct pp_hwmgr *hwmgr;
struct pp_instance *pp_handle = (struct pp_instance *)handle; struct pp_instance *pp_handle = (struct pp_instance *)handle;
int ret = 0; int ret = 0;
int size = 0;
ret = pp_check(pp_handle); ret = pp_check(pp_handle);
...@@ -719,9 +755,11 @@ static int pp_dpm_get_pp_table(void *handle, char **table) ...@@ -719,9 +755,11 @@ static int pp_dpm_get_pp_table(void *handle, char **table)
if (!hwmgr->soft_pp_table) if (!hwmgr->soft_pp_table)
return -EINVAL; return -EINVAL;
mutex_lock(&pp_handle->pp_lock);
*table = (char *)hwmgr->soft_pp_table; *table = (char *)hwmgr->soft_pp_table;
size = hwmgr->soft_pp_table_size;
return hwmgr->soft_pp_table_size; mutex_unlock(&pp_handle->pp_lock);
return size;
} }
static int pp_dpm_set_pp_table(void *handle, const char *buf, size_t size) static int pp_dpm_set_pp_table(void *handle, const char *buf, size_t size)
...@@ -736,19 +774,21 @@ static int pp_dpm_set_pp_table(void *handle, const char *buf, size_t size) ...@@ -736,19 +774,21 @@ static int pp_dpm_set_pp_table(void *handle, const char *buf, size_t size)
return ret; return ret;
hwmgr = pp_handle->hwmgr; hwmgr = pp_handle->hwmgr;
mutex_lock(&pp_handle->pp_lock);
if (!hwmgr->hardcode_pp_table) { if (!hwmgr->hardcode_pp_table) {
hwmgr->hardcode_pp_table = kmemdup(hwmgr->soft_pp_table, hwmgr->hardcode_pp_table = kmemdup(hwmgr->soft_pp_table,
hwmgr->soft_pp_table_size, hwmgr->soft_pp_table_size,
GFP_KERNEL); GFP_KERNEL);
if (!hwmgr->hardcode_pp_table) {
if (!hwmgr->hardcode_pp_table) mutex_unlock(&pp_handle->pp_lock);
return -ENOMEM; return -ENOMEM;
}
} }
memcpy(hwmgr->hardcode_pp_table, buf, size); memcpy(hwmgr->hardcode_pp_table, buf, size);
hwmgr->soft_pp_table = hwmgr->hardcode_pp_table; hwmgr->soft_pp_table = hwmgr->hardcode_pp_table;
mutex_unlock(&pp_handle->pp_lock);
ret = amd_powerplay_reset(handle); ret = amd_powerplay_reset(handle);
if (ret) if (ret)
...@@ -781,8 +821,10 @@ static int pp_dpm_force_clock_level(void *handle, ...@@ -781,8 +821,10 @@ static int pp_dpm_force_clock_level(void *handle,
pr_info("%s was not implemented.\n", __func__); pr_info("%s was not implemented.\n", __func__);
return 0; return 0;
} }
mutex_lock(&pp_handle->pp_lock);
return hwmgr->hwmgr_func->force_clock_level(hwmgr, type, mask); hwmgr->hwmgr_func->force_clock_level(hwmgr, type, mask);
mutex_unlock(&pp_handle->pp_lock);
return ret;
} }
static int pp_dpm_print_clock_levels(void *handle, static int pp_dpm_print_clock_levels(void *handle,
...@@ -803,7 +845,10 @@ static int pp_dpm_print_clock_levels(void *handle, ...@@ -803,7 +845,10 @@ static int pp_dpm_print_clock_levels(void *handle,
pr_info("%s was not implemented.\n", __func__); pr_info("%s was not implemented.\n", __func__);
return 0; return 0;
} }
return hwmgr->hwmgr_func->print_clock_levels(hwmgr, type, buf); mutex_lock(&pp_handle->pp_lock);
ret = hwmgr->hwmgr_func->print_clock_levels(hwmgr, type, buf);
mutex_unlock(&pp_handle->pp_lock);
return ret;
} }
static int pp_dpm_get_sclk_od(void *handle) static int pp_dpm_get_sclk_od(void *handle)
...@@ -823,8 +868,10 @@ static int pp_dpm_get_sclk_od(void *handle) ...@@ -823,8 +868,10 @@ static int pp_dpm_get_sclk_od(void *handle)
pr_info("%s was not implemented.\n", __func__); pr_info("%s was not implemented.\n", __func__);
return 0; return 0;
} }
mutex_lock(&pp_handle->pp_lock);
return hwmgr->hwmgr_func->get_sclk_od(hwmgr); ret = hwmgr->hwmgr_func->get_sclk_od(hwmgr);
mutex_unlock(&pp_handle->pp_lock);
return ret;
} }
static int pp_dpm_set_sclk_od(void *handle, uint32_t value) static int pp_dpm_set_sclk_od(void *handle, uint32_t value)
...@@ -845,7 +892,10 @@ static int pp_dpm_set_sclk_od(void *handle, uint32_t value) ...@@ -845,7 +892,10 @@ static int pp_dpm_set_sclk_od(void *handle, uint32_t value)
return 0; return 0;
} }
return hwmgr->hwmgr_func->set_sclk_od(hwmgr, value); mutex_lock(&pp_handle->pp_lock);
ret = hwmgr->hwmgr_func->set_sclk_od(hwmgr, value);
mutex_lock(&pp_handle->pp_lock);
return ret;
} }
static int pp_dpm_get_mclk_od(void *handle) static int pp_dpm_get_mclk_od(void *handle)
...@@ -865,8 +915,10 @@ static int pp_dpm_get_mclk_od(void *handle) ...@@ -865,8 +915,10 @@ static int pp_dpm_get_mclk_od(void *handle)
pr_info("%s was not implemented.\n", __func__); pr_info("%s was not implemented.\n", __func__);
return 0; return 0;
} }
mutex_lock(&pp_handle->pp_lock);
return hwmgr->hwmgr_func->get_mclk_od(hwmgr); ret = hwmgr->hwmgr_func->get_mclk_od(hwmgr);
mutex_unlock(&pp_handle->pp_lock);
return ret;
} }
static int pp_dpm_set_mclk_od(void *handle, uint32_t value) static int pp_dpm_set_mclk_od(void *handle, uint32_t value)
...@@ -886,8 +938,10 @@ static int pp_dpm_set_mclk_od(void *handle, uint32_t value) ...@@ -886,8 +938,10 @@ static int pp_dpm_set_mclk_od(void *handle, uint32_t value)
pr_info("%s was not implemented.\n", __func__); pr_info("%s was not implemented.\n", __func__);
return 0; return 0;
} }
mutex_lock(&pp_handle->pp_lock);
return hwmgr->hwmgr_func->set_mclk_od(hwmgr, value); ret = hwmgr->hwmgr_func->set_mclk_od(hwmgr, value);
mutex_unlock(&pp_handle->pp_lock);
return ret;
} }
static int pp_dpm_read_sensor(void *handle, int idx, static int pp_dpm_read_sensor(void *handle, int idx,
...@@ -909,7 +963,11 @@ static int pp_dpm_read_sensor(void *handle, int idx, ...@@ -909,7 +963,11 @@ static int pp_dpm_read_sensor(void *handle, int idx,
return 0; return 0;
} }
return hwmgr->hwmgr_func->read_sensor(hwmgr, idx, value, size); mutex_lock(&pp_handle->pp_lock);
ret = hwmgr->hwmgr_func->read_sensor(hwmgr, idx, value, size);
mutex_unlock(&pp_handle->pp_lock);
return ret;
} }
static struct amd_vce_state* static struct amd_vce_state*
...@@ -1114,8 +1172,8 @@ int amd_powerplay_create(struct amd_pp_init *pp_init, ...@@ -1114,8 +1172,8 @@ int amd_powerplay_create(struct amd_pp_init *pp_init,
instance->pm_en = pp_init->pm_en; instance->pm_en = pp_init->pm_en;
instance->feature_mask = pp_init->feature_mask; instance->feature_mask = pp_init->feature_mask;
instance->device = pp_init->device; instance->device = pp_init->device;
mutex_init(&instance->pp_lock);
*handle = instance; *handle = instance;
return 0; return 0;
} }
...@@ -1186,9 +1244,9 @@ int amd_powerplay_display_configuration_change(void *handle, ...@@ -1186,9 +1244,9 @@ int amd_powerplay_display_configuration_change(void *handle,
return ret; return ret;
hwmgr = pp_handle->hwmgr; hwmgr = pp_handle->hwmgr;
mutex_lock(&pp_handle->pp_lock);
phm_store_dal_configuration_data(hwmgr, display_config); phm_store_dal_configuration_data(hwmgr, display_config);
mutex_unlock(&pp_handle->pp_lock);
return 0; return 0;
} }
...@@ -1209,7 +1267,10 @@ int amd_powerplay_get_display_power_level(void *handle, ...@@ -1209,7 +1267,10 @@ int amd_powerplay_get_display_power_level(void *handle,
if (output == NULL) if (output == NULL)
return -EINVAL; return -EINVAL;
return phm_get_dal_power_level(hwmgr, output); mutex_lock(&pp_handle->pp_lock);
ret = phm_get_dal_power_level(hwmgr, output);
mutex_unlock(&pp_handle->pp_lock);
return ret;
} }
int amd_powerplay_get_current_clocks(void *handle, int amd_powerplay_get_current_clocks(void *handle,
...@@ -1228,14 +1289,22 @@ int amd_powerplay_get_current_clocks(void *handle, ...@@ -1228,14 +1289,22 @@ int amd_powerplay_get_current_clocks(void *handle,
hwmgr = pp_handle->hwmgr; hwmgr = pp_handle->hwmgr;
mutex_lock(&pp_handle->pp_lock);
phm_get_dal_power_level(hwmgr, &simple_clocks); phm_get_dal_power_level(hwmgr, &simple_clocks);
if (phm_cap_enabled(hwmgr->platform_descriptor.platformCaps, PHM_PlatformCaps_PowerContainment)) { if (phm_cap_enabled(hwmgr->platform_descriptor.platformCaps,
if (0 != phm_get_clock_info(hwmgr, &hwmgr->current_ps->hardware, &hw_clocks, PHM_PerformanceLevelDesignation_PowerContainment)) PHM_PlatformCaps_PowerContainment))
PP_ASSERT_WITH_CODE(0, "Error in PHM_GetPowerContainmentClockInfo", return -1); ret = phm_get_clock_info(hwmgr, &hwmgr->current_ps->hardware,
} else { &hw_clocks, PHM_PerformanceLevelDesignation_PowerContainment);
if (0 != phm_get_clock_info(hwmgr, &hwmgr->current_ps->hardware, &hw_clocks, PHM_PerformanceLevelDesignation_Activity)) else
PP_ASSERT_WITH_CODE(0, "Error in PHM_GetClockInfo", return -1); ret = phm_get_clock_info(hwmgr, &hwmgr->current_ps->hardware,
&hw_clocks, PHM_PerformanceLevelDesignation_Activity);
if (ret != 0) {
pr_info("Error in phm_get_clock_info \n");
mutex_unlock(&pp_handle->pp_lock);
return -EINVAL;
} }
clocks->min_engine_clock = hw_clocks.min_eng_clk; clocks->min_engine_clock = hw_clocks.min_eng_clk;
...@@ -1254,14 +1323,12 @@ int amd_powerplay_get_current_clocks(void *handle, ...@@ -1254,14 +1323,12 @@ int amd_powerplay_get_current_clocks(void *handle,
clocks->max_engine_clock_in_sr = hw_clocks.max_eng_clk; clocks->max_engine_clock_in_sr = hw_clocks.max_eng_clk;
clocks->min_engine_clock_in_sr = hw_clocks.min_eng_clk; clocks->min_engine_clock_in_sr = hw_clocks.min_eng_clk;
} }
mutex_unlock(&pp_handle->pp_lock);
return 0; return 0;
} }
int amd_powerplay_get_clock_by_type(void *handle, enum amd_pp_clock_type type, struct amd_pp_clocks *clocks) int amd_powerplay_get_clock_by_type(void *handle, enum amd_pp_clock_type type, struct amd_pp_clocks *clocks)
{ {
int result = -1;
struct pp_hwmgr *hwmgr; struct pp_hwmgr *hwmgr;
struct pp_instance *pp_handle = (struct pp_instance *)handle; struct pp_instance *pp_handle = (struct pp_instance *)handle;
int ret = 0; int ret = 0;
...@@ -1276,9 +1343,10 @@ int amd_powerplay_get_clock_by_type(void *handle, enum amd_pp_clock_type type, s ...@@ -1276,9 +1343,10 @@ int amd_powerplay_get_clock_by_type(void *handle, enum amd_pp_clock_type type, s
if (clocks == NULL) if (clocks == NULL)
return -EINVAL; return -EINVAL;
result = phm_get_clock_by_type(hwmgr, type, clocks); mutex_lock(&pp_handle->pp_lock);
ret = phm_get_clock_by_type(hwmgr, type, clocks);
return result; mutex_unlock(&pp_handle->pp_lock);
return ret;
} }
int amd_powerplay_get_display_mode_validation_clocks(void *handle, int amd_powerplay_get_display_mode_validation_clocks(void *handle,
...@@ -1295,13 +1363,15 @@ int amd_powerplay_get_display_mode_validation_clocks(void *handle, ...@@ -1295,13 +1363,15 @@ int amd_powerplay_get_display_mode_validation_clocks(void *handle,
hwmgr = pp_handle->hwmgr; hwmgr = pp_handle->hwmgr;
if (clocks == NULL) if (clocks == NULL)
return -EINVAL; return -EINVAL;
mutex_lock(&pp_handle->pp_lock);
if (phm_cap_enabled(hwmgr->platform_descriptor.platformCaps, PHM_PlatformCaps_DynamicPatchPowerState)) if (phm_cap_enabled(hwmgr->platform_descriptor.platformCaps, PHM_PlatformCaps_DynamicPatchPowerState))
ret = phm_get_max_high_clocks(hwmgr, clocks); ret = phm_get_max_high_clocks(hwmgr, clocks);
mutex_unlock(&pp_handle->pp_lock);
return ret; return ret;
} }
drivers/gpu/drm/amd/powerplay/inc/pp_instance.h
View file @ 2a507105
...@@ -39,6 +39,7 @@ struct pp_instance { ...@@ -39,6 +39,7 @@ struct pp_instance {
struct pp_smumgr *smu_mgr; struct pp_smumgr *smu_mgr;
struct pp_hwmgr *hwmgr; struct pp_hwmgr *hwmgr;
struct pp_eventmgr *eventmgr; struct pp_eventmgr *eventmgr;
struct mutex pp_lock;
}; };
#endif #endif
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment
GitLab Nexedi Edition | About GitLab | About Nexedi | 沪ICP备2021021310号-2 | 沪ICP备2021021310号-7