[AMD Public Use] yes, i can rename the marco "smu_ppt_xxx" to "smu_ppt_funs". but in kernel code, we can see the same code style, likes: path: //include/linux/cred.h #define task_cred_xxx(task, xxx) \ ({ \ __typeof__(((struct cred *)NULL)->xxx) ___val; \ rcu_read_lock(); \ ___val = __task_cred((task))->xxx; \ rcu_read_unlock(); \ ___val; \ }) #define task_uid(task) (task_cred_xxx((task), uid)) #define task_euid(task) (task_cred_xxx((task), euid)) #define current_cred_xxx(xxx) \ ({ \ current_cred()->xxx; \ }) #define current_uid() (current_cred_xxx(uid)) #define current_gid() (current_cred_xxx(gid)) #define current_euid() (current_cred_xxx(euid)) #define current_egid() (current_cred_xxx(egid)) #define current_suid() (current_cred_xxx(suid)) #define current_sgid() (current_cred_xxx(sgid)) #define current_fsuid() (current_cred_xxx(fsuid)) #define current_fsgid() (current_cred_xxx(fsgid)) #define current_cap() (current_cred_xxx(cap_effective)) #define current_user() (current_cred_xxx(user)) anyway, i will send a new patch to correct it. thanks. Best Regards, Kevin ________________________________ From: Zhang, Hawking Sent: Tuesday, June 16, 2020 6:03 PM To: Wang, Kevin(Yang) ; amd-gfx@lists.freedesktop.org Cc: Deucher, Alexander Subject: RE: [PATCH 2/2] drm/amd/smu: unify smu ppt callback marcos [AMD Public Use] The naming style smu_ppt_xxx makes me confusing..... at first glance, I thought you just have some dummy code or the patch is something still working in progress. It would be better to rename it to smu_ppt_funcs. Regards, Hawking -----Original Message----- From: Wang, Kevin(Yang) Sent: Tuesday, June 16, 2020 15:06 To: amd-gfx@lists.freedesktop.org Cc: Zhang, Hawking ; Deucher, Alexander ; Wang, Kevin(Yang) Subject: [PATCH 2/2] drm/amd/smu: unify smu ppt callback marcos add the new macro "smu_ppt_xxx()" to unify smu callback interfaces Signed-off-by: Kevin Wang --- drivers/gpu/drm/amd/powerplay/smu_internal.h | 266 ++++++------------- 1 file changed, 82 insertions(+), 184 deletions(-) diff --git a/drivers/gpu/drm/amd/powerplay/smu_internal.h b/drivers/gpu/drm/amd/powerplay/smu_internal.h index 880ef54884f8..0f9841bcdcb8 100644 --- a/drivers/gpu/drm/amd/powerplay/smu_internal.h +++ b/drivers/gpu/drm/amd/powerplay/smu_internal.h @@ -25,191 +25,89 @@ #include "amdgpu_smu.h" -#define smu_init_microcode(smu) \ - ((smu)->ppt_funcs->init_microcode ? (smu)->ppt_funcs->init_microcode((smu)) : 0) -#define smu_fini_microcode(smu) \ - ((smu)->ppt_funcs->fini_microcode ? (smu)->ppt_funcs->fini_microcode((smu)) : 0) -#define smu_init_smc_tables(smu) \ - ((smu)->ppt_funcs->init_smc_tables ? (smu)->ppt_funcs->init_smc_tables((smu)) : 0) -#define smu_fini_smc_tables(smu) \ - ((smu)->ppt_funcs->fini_smc_tables ? (smu)->ppt_funcs->fini_smc_tables((smu)) : 0) -#define smu_init_power(smu) \ - ((smu)->ppt_funcs->init_power ? (smu)->ppt_funcs->init_power((smu)) : 0) -#define smu_fini_power(smu) \ - ((smu)->ppt_funcs->fini_power ? (smu)->ppt_funcs->fini_power((smu)) : 0) - -#define smu_setup_pptable(smu) \ - ((smu)->ppt_funcs->setup_pptable ? (smu)->ppt_funcs->setup_pptable((smu)) : 0) -#define smu_powergate_sdma(smu, gate) \ - ((smu)->ppt_funcs->powergate_sdma ? (smu)->ppt_funcs->powergate_sdma((smu), (gate)) : 0) - -#define smu_get_vbios_bootup_values(smu) \ - ((smu)->ppt_funcs->get_vbios_bootup_values ? (smu)->ppt_funcs->get_vbios_bootup_values((smu)) : 0) -#define smu_check_fw_version(smu) \ - ((smu)->ppt_funcs->check_fw_version ? (smu)->ppt_funcs->check_fw_version((smu)) : 0) -#define smu_write_pptable(smu) \ - ((smu)->ppt_funcs->write_pptable ? (smu)->ppt_funcs->write_pptable((smu)) : 0) -#define smu_set_min_dcef_deep_sleep(smu, clk) \ - ((smu)->ppt_funcs->set_min_dcef_deep_sleep ? (smu)->ppt_funcs->set_min_dcef_deep_sleep((smu), (clk)) : 0) -#define smu_set_driver_table_location(smu) \ - ((smu)->ppt_funcs->set_driver_table_location ? (smu)->ppt_funcs->set_driver_table_location((smu)) : 0) -#define smu_set_tool_table_location(smu) \ - ((smu)->ppt_funcs->set_tool_table_location ? (smu)->ppt_funcs->set_tool_table_location((smu)) : 0) -#define smu_notify_memory_pool_location(smu) \ - ((smu)->ppt_funcs->notify_memory_pool_location ? (smu)->ppt_funcs->notify_memory_pool_location((smu)) : 0) -#define smu_gfx_off_control(smu, enable) \ - ((smu)->ppt_funcs->gfx_off_control ? (smu)->ppt_funcs->gfx_off_control((smu), (enable)) : 0) - -#define smu_set_last_dcef_min_deep_sleep_clk(smu) \ - ((smu)->ppt_funcs->set_last_dcef_min_deep_sleep_clk ? (smu)->ppt_funcs->set_last_dcef_min_deep_sleep_clk((smu)) : 0) -#define smu_system_features_control(smu, en) \ - ((smu)->ppt_funcs->system_features_control ? (smu)->ppt_funcs->system_features_control((smu), (en)) : 0) -#define smu_init_max_sustainable_clocks(smu) \ - ((smu)->ppt_funcs->init_max_sustainable_clocks ? (smu)->ppt_funcs->init_max_sustainable_clocks((smu)) : 0) -#define smu_set_default_od_settings(smu) \ - ((smu)->ppt_funcs->set_default_od_settings ? (smu)->ppt_funcs->set_default_od_settings((smu)) : 0) - -#define smu_send_smc_msg_with_param(smu, msg, param, read_arg) \ - ((smu)->ppt_funcs->send_smc_msg_with_param? (smu)->ppt_funcs->send_smc_msg_with_param((smu), (msg), (param), (read_arg)) : 0) - -static inline int smu_send_smc_msg(struct smu_context *smu, enum smu_message_type msg, uint32_t *read_arg) { - return smu_send_smc_msg_with_param(smu, msg, 0, read_arg); -} - -#define smu_alloc_dpm_context(smu) \ - ((smu)->ppt_funcs->alloc_dpm_context ? (smu)->ppt_funcs->alloc_dpm_context((smu)) : 0) -#define smu_init_display_count(smu, count) \ - ((smu)->ppt_funcs->init_display_count ? (smu)->ppt_funcs->init_display_count((smu), (count)) : 0) -#define smu_feature_set_allowed_mask(smu) \ - ((smu)->ppt_funcs->set_allowed_mask? (smu)->ppt_funcs->set_allowed_mask((smu)) : 0) -#define smu_feature_get_enabled_mask(smu, mask, num) \ - ((smu)->ppt_funcs->get_enabled_mask? (smu)->ppt_funcs->get_enabled_mask((smu), (mask), (num)) : 0) -#define smu_is_dpm_running(smu) \ - ((smu)->ppt_funcs->is_dpm_running ? (smu)->ppt_funcs->is_dpm_running((smu)) : 0) -#define smu_notify_display_change(smu) \ - ((smu)->ppt_funcs->notify_display_change? (smu)->ppt_funcs->notify_display_change((smu)) : 0) -#define smu_set_default_dpm_table(smu) \ - ((smu)->ppt_funcs->set_default_dpm_table ? (smu)->ppt_funcs->set_default_dpm_table((smu)) : 0) -#define smu_populate_umd_state_clk(smu) \ - ((smu)->ppt_funcs->populate_umd_state_clk ? (smu)->ppt_funcs->populate_umd_state_clk((smu)) : 0) -#define smu_set_default_od8_settings(smu) \ - ((smu)->ppt_funcs->set_default_od8_settings ? (smu)->ppt_funcs->set_default_od8_settings((smu)) : 0) - -#define smu_get_current_clk_freq(smu, clk_id, value) \ - ((smu)->ppt_funcs->get_current_clk_freq? (smu)->ppt_funcs->get_current_clk_freq((smu), (clk_id), (value)) : 0) - -#define smu_tables_init(smu, tab) \ - ((smu)->ppt_funcs->tables_init ? (smu)->ppt_funcs->tables_init((smu), (tab)) : 0) -#define smu_set_thermal_fan_table(smu) \ - ((smu)->ppt_funcs->set_thermal_fan_table ? (smu)->ppt_funcs->set_thermal_fan_table((smu)) : 0) -#define smu_enable_thermal_alert(smu) \ - ((smu)->ppt_funcs->enable_thermal_alert? (smu)->ppt_funcs->enable_thermal_alert((smu)) : 0) -#define smu_disable_thermal_alert(smu) \ - ((smu)->ppt_funcs->disable_thermal_alert? (smu)->ppt_funcs->disable_thermal_alert((smu)) : 0) - -#define smu_smc_read_sensor(smu, sensor, data, size) \ - ((smu)->ppt_funcs->read_sensor? (smu)->ppt_funcs->read_sensor((smu), (sensor), (data), (size)) : -EINVAL) - -#define smu_pre_display_config_changed(smu) \ - ((smu)->ppt_funcs->pre_display_config_changed ? (smu)->ppt_funcs->pre_display_config_changed((smu)) : 0) -#define smu_display_config_changed(smu) \ - ((smu)->ppt_funcs->display_config_changed ? (smu)->ppt_funcs->display_config_changed((smu)) : 0) -#define smu_apply_clocks_adjust_rules(smu) \ - ((smu)->ppt_funcs->apply_clocks_adjust_rules ? (smu)->ppt_funcs->apply_clocks_adjust_rules((smu)) : 0) -#define smu_notify_smc_display_config(smu) \ - ((smu)->ppt_funcs->notify_smc_display_config ? (smu)->ppt_funcs->notify_smc_display_config((smu)) : 0) -#define smu_force_dpm_limit_value(smu, highest) \ - ((smu)->ppt_funcs->force_dpm_limit_value ? (smu)->ppt_funcs->force_dpm_limit_value((smu), (highest)) : 0) -#define smu_unforce_dpm_levels(smu) \ - ((smu)->ppt_funcs->unforce_dpm_levels ? (smu)->ppt_funcs->unforce_dpm_levels((smu)) : 0) +#define smu_ppt_xxx(intf, ret, smu, args...) \ + ((smu)->ppt_funcs ? ((smu)->ppt_funcs->intf ? (smu)->ppt_funcs->intf(smu, ##args) : ret) : -EINVAL) + +#define smu_init_microcode(smu) smu_ppt_xxx(init_microcode, 0, smu) +#define smu_fini_microcode(smu) smu_ppt_xxx(fini_microcode, 0, smu) +#define smu_init_smc_tables(smu) smu_ppt_xxx(init_smc_tables, 0, smu) +#define smu_fini_smc_tables(smu) smu_ppt_xxx(fini_smc_tables, 0, smu) +#define smu_init_power(smu) smu_ppt_xxx(init_power, 0, smu) +#define smu_fini_power(smu) smu_ppt_xxx(fini_power, 0, smu) +#define smu_setup_pptable(smu) smu_ppt_xxx(setup_pptable, 0, smu) +#define smu_powergate_sdma(smu, gate) smu_ppt_xxx(powergate_sdma, 0, smu, gate) +#define smu_get_vbios_bootup_values(smu) smu_ppt_xxx(get_vbios_bootup_values, 0, smu) +#define smu_check_fw_version(smu) smu_ppt_xxx(check_fw_version, 0, smu) +#define smu_write_pptable(smu) smu_ppt_xxx(write_pptable, 0, smu) +#define smu_set_min_dcef_deep_sleep(smu, clk) smu_ppt_xxx(set_min_dcef_deep_sleep, 0, smu, clk) +#define smu_set_driver_table_location(smu) smu_ppt_xxx(set_driver_table_location, 0, smu) +#define smu_set_tool_table_location(smu) smu_ppt_xxx(set_tool_table_location, 0, smu) +#define smu_notify_memory_pool_location(smu) smu_ppt_xxx(notify_memory_pool_location, 0, smu) +#define smu_gfx_off_control(smu, enable) smu_ppt_xxx(gfx_off_control, 0, smu, enable) +#define smu_set_last_dcef_min_deep_sleep_clk(smu) smu_ppt_xxx(set_last_dcef_min_deep_sleep_clk, 0, smu) +#define smu_system_features_control(smu, en) smu_ppt_xxx(system_features_control, 0, smu, en) +#define smu_init_max_sustainable_clocks(smu) smu_ppt_xxx(init_max_sustainable_clocks, 0, smu) +#define smu_set_default_od_settings(smu) smu_ppt_xxx(set_default_od_settings, 0, smu) +#define smu_send_smc_msg_with_param(smu, msg, param, read_arg) smu_ppt_xxx(send_smc_msg_with_param, 0, smu, msg, param, read_arg) +#define smu_send_smc_msg(smu, msg, read_arg) smu_ppt_xxx(send_smc_msg_with_param, 0, smu, msg, 0, read_arg) +#define smu_alloc_dpm_context(smu) smu_ppt_xxx(alloc_dpm_context, 0, smu) +#define smu_init_display_count(smu, count) smu_ppt_xxx(init_display_count, 0, smu, count) +#define smu_feature_set_allowed_mask(smu) smu_ppt_xxx(set_allowed_mask, 0, smu) +#define smu_feature_get_enabled_mask(smu, mask, num) smu_ppt_xxx(get_enabled_mask, 0, smu, mask, num) +#define smu_is_dpm_running(smu) smu_ppt_xxx(is_dpm_running, 0 , smu) +#define smu_notify_display_change(smu) smu_ppt_xxx(notify_display_change, 0, smu) +#define smu_set_default_dpm_table(smu) smu_ppt_xxx(set_default_dpm_table, 0, smu) +#define smu_populate_umd_state_clk(smu) smu_ppt_xxx(populate_umd_state_clk, 0, smu) +#define smu_set_default_od8_settings(smu) smu_ppt_xxx(set_default_od8_settings, 0, smu) +#define smu_get_current_clk_freq(smu, clk_id, value) smu_ppt_xxx(get_current_clk_freq, 0, smu, clk_id, value) +#define smu_tables_init(smu, tab) smu_ppt_xxx(tables_init, 0, smu, tab) +#define smu_set_thermal_fan_table(smu) smu_ppt_xxx(set_thermal_fan_table, 0, smu) +#define smu_enable_thermal_alert(smu) smu_ppt_xxx(enable_thermal_alert, 0, smu) +#define smu_disable_thermal_alert(smu) smu_ppt_xxx(disable_thermal_alert, 0, smu) +#define smu_smc_read_sensor(smu, sensor, data, size) smu_ppt_xxx(read_sensor, -EINVAL, smu, sensor, data, size) +#define smu_pre_display_config_changed(smu) smu_ppt_xxx(pre_display_config_changed, 0, smu) +#define smu_display_config_changed(smu) smu_ppt_xxx(display_config_changed, 0 , smu) +#define smu_apply_clocks_adjust_rules(smu) smu_ppt_xxx(apply_clocks_adjust_rules, 0, smu) +#define smu_notify_smc_display_config(smu) smu_ppt_xxx(notify_smc_display_config, 0, smu) +#define smu_force_dpm_limit_value(smu, highest) smu_ppt_xxx(force_dpm_limit_value, 0, smu, highest) +#define smu_unforce_dpm_levels(smu) smu_ppt_xxx(unforce_dpm_levels, 0, smu) +#define smu_set_cpu_power_state(smu) smu_ppt_xxx(set_cpu_power_state, 0, smu) +#define smu_msg_get_index(smu, msg) smu_ppt_xxx(get_smu_msg_index, -EINVAL, smu, msg) +#define smu_clk_get_index(smu, clk) smu_ppt_xxx(get_smu_clk_index, -EINVAL, smu, clk) +#define smu_feature_get_index(smu, fea) smu_ppt_xxx(get_smu_feature_index, -EINVAL, smu, fea) +#define smu_table_get_index(smu, tab) smu_ppt_xxx(get_smu_table_index, -EINVAL, smu, tab) +#define smu_power_get_index(smu, src) smu_ppt_xxx(get_smu_power_index, -EINVAL, smu, src) +#define smu_workload_get_type(smu, type) smu_ppt_xxx(get_smu_power_index, -EINVAL, smu, type) +#define smu_run_btc(smu) smu_ppt_xxx(run_btc, 0, smu) +#define smu_get_allowed_feature_mask(smu, feature_mask, num) smu_ppt_xxx(get_allowed_feature_mask, 0, smu, feature_mask, num) +#define smu_store_cc6_data(smu, st, cc6_dis, pst_dis, pst_sw_dis) smu_ppt_xxx(store_cc6_data, 0, smu, st, cc6_dis, pst_dis, pst_sw_dis) +#define smu_get_dal_power_level(smu, clocks) smu_ppt_xxx(get_dal_power_level, 0, smu, clocks) +#define smu_get_perf_level(smu, designation, level) smu_ppt_xxx(get_perf_level, 0, smu, designation, level) +#define smu_get_current_shallow_sleep_clocks(smu, clocks) smu_ppt_xxx(get_current_shallow_sleep_clocks, 0, smu, clocks) +#define smu_dpm_set_vcn_enable(smu, enable) smu_ppt_xxx(dpm_set_vcn_enable, 0, smu, enable) +#define smu_dpm_set_jpeg_enable(smu, enable) smu_ppt_xxx(dpm_set_jpeg_enable, 0, smu, enable) +#define smu_set_watermarks_table(smu, tab, clock_ranges) smu_ppt_xxx(set_watermarks_table, 0, smu, tab, clock_ranges) +#define smu_get_current_clk_freq_by_table(smu, clk_type, value) smu_ppt_xxx(get_current_clk_freq_by_table, 0, smu, clk_type, value) +#define smu_thermal_temperature_range_update(smu, range, rw) smu_ppt_xxx(thermal_temperature_range_update, 0, smu, range, rw) +#define smu_get_thermal_temperature_range(smu, range) smu_ppt_xxx(get_thermal_temperature_range, 0, smu, range) +#define smu_register_irq_handler(smu) smu_ppt_xxx(register_irq_handler, 0, smu) +#define smu_get_dpm_ultimate_freq(smu, param, min, max) smu_ppt_xxx(get_dpm_ultimate_freq, 0, smu, param, min, max) +#define smu_asic_set_performance_level(smu, level) smu_ppt_xxx(set_performance_level, -EINVAL, smu, level) +#define smu_dump_pptable(smu) smu_ppt_xxx(dump_pptable, 0, smu) +#define smu_get_dpm_clk_limited(smu, clk_type, dpm_level, freq) smu_ppt_xxx(get_dpm_clk_limited, -EINVAL, smu, clk_type, dpm_level, freq) +#define smu_set_soft_freq_limited_range(smu, clk_type, min, max) smu_ppt_xxx(set_soft_freq_limited_range, -EINVAL, smu, clk_type, min, max) +#define smu_override_pcie_parameters(smu) smu_ppt_xxx(override_pcie_parameters, 0, smu) +#define smu_update_pcie_parameters(smu, pcie_gen_cap, pcie_width_cap) smu_ppt_xxx(update_pcie_parameters, 0, smu, pcie_gen_cap, pcie_width_cap) +#define smu_set_thermal_range(smu, range) smu_ppt_xxx(set_thermal_range, 0, smu, range) +#define smu_disable_umc_cdr_12gbps_workaround(smu) smu_ppt_xxx(disable_umc_cdr_12gbps_workaround, 0, smu) +#define smu_set_power_source(smu, power_src) smu_ppt_xxx(set_power_source, 0, smu, power_src) +#define smu_i2c_eeprom_init(smu, control) smu_ppt_xxx(i2c_eeprom_init, 0, smu, control) +#define smu_i2c_eeprom_fini(smu, control) smu_ppt_xxx(i2c_eeprom_fini, 0, smu, control) +#define smu_get_unique_id(smu) smu_ppt_xxx(get_unique_id, 0, smu) +#define smu_log_thermal_throttling(smu) smu_ppt_xxx(log_thermal_throttling_event, 0, smu) +#define smu_get_asic_power_limits(smu) smu_ppt_xxx(get_power_limit, 0, smu) #define smu_get_profiling_clk_mask(smu, level, sclk_mask, mclk_mask, soc_mask) \ - ((smu)->ppt_funcs->get_profiling_clk_mask ? (smu)->ppt_funcs->get_profiling_clk_mask((smu), (level), (sclk_mask), (mclk_mask), (soc_mask)) : 0) -#define smu_set_cpu_power_state(smu) \ - ((smu)->ppt_funcs->set_cpu_power_state ? (smu)->ppt_funcs->set_cpu_power_state((smu)) : 0) - -#define smu_msg_get_index(smu, msg) \ - ((smu)->ppt_funcs? ((smu)->ppt_funcs->get_smu_msg_index? (smu)->ppt_funcs->get_smu_msg_index((smu), (msg)) : -EINVAL) : -EINVAL) -#define smu_clk_get_index(smu, msg) \ - ((smu)->ppt_funcs? ((smu)->ppt_funcs->get_smu_clk_index? (smu)->ppt_funcs->get_smu_clk_index((smu), (msg)) : -EINVAL) : -EINVAL) -#define smu_feature_get_index(smu, msg) \ - ((smu)->ppt_funcs? ((smu)->ppt_funcs->get_smu_feature_index? (smu)->ppt_funcs->get_smu_feature_index((smu), (msg)) : -EINVAL) : -EINVAL) -#define smu_table_get_index(smu, tab) \ - ((smu)->ppt_funcs? ((smu)->ppt_funcs->get_smu_table_index? (smu)->ppt_funcs->get_smu_table_index((smu), (tab)) : -EINVAL) : -EINVAL) -#define smu_power_get_index(smu, src) \ - ((smu)->ppt_funcs? ((smu)->ppt_funcs->get_smu_power_index? (smu)->ppt_funcs->get_smu_power_index((smu), (src)) : -EINVAL) : -EINVAL) -#define smu_workload_get_type(smu, profile) \ - ((smu)->ppt_funcs? ((smu)->ppt_funcs->get_workload_type? (smu)->ppt_funcs->get_workload_type((smu), (profile)) : -EINVAL) : -EINVAL) -#define smu_run_btc(smu) \ - ((smu)->ppt_funcs? ((smu)->ppt_funcs->run_btc? (smu)->ppt_funcs->run_btc((smu)) : 0) : 0) -#define smu_get_allowed_feature_mask(smu, feature_mask, num) \ - ((smu)->ppt_funcs? ((smu)->ppt_funcs->get_allowed_feature_mask? (smu)->ppt_funcs->get_allowed_feature_mask((smu), (feature_mask), (num)) : 0) : 0) - - -#define smu_store_cc6_data(smu, st, cc6_dis, pst_dis, pst_sw_dis) \ - ((smu)->ppt_funcs->store_cc6_data ? (smu)->ppt_funcs->store_cc6_data((smu), (st), (cc6_dis), (pst_dis), (pst_sw_dis)) : 0) - -#define smu_get_dal_power_level(smu, clocks) \ - ((smu)->ppt_funcs->get_dal_power_level ? (smu)->ppt_funcs->get_dal_power_level((smu), (clocks)) : 0) -#define smu_get_perf_level(smu, designation, level) \ - ((smu)->ppt_funcs->get_perf_level ? (smu)->ppt_funcs->get_perf_level((smu), (designation), (level)) : 0) -#define smu_get_current_shallow_sleep_clocks(smu, clocks) \ - ((smu)->ppt_funcs->get_current_shallow_sleep_clocks ? (smu)->ppt_funcs->get_current_shallow_sleep_clocks((smu), (clocks)) : 0) - -#define smu_dpm_set_vcn_enable(smu, enable) \ - ((smu)->ppt_funcs->dpm_set_vcn_enable ? (smu)->ppt_funcs->dpm_set_vcn_enable((smu), (enable)) : 0) -#define smu_dpm_set_jpeg_enable(smu, enable) \ - ((smu)->ppt_funcs->dpm_set_jpeg_enable ? (smu)->ppt_funcs->dpm_set_jpeg_enable((smu), (enable)) : 0) - -#define smu_set_watermarks_table(smu, tab, clock_ranges) \ - ((smu)->ppt_funcs->set_watermarks_table ? (smu)->ppt_funcs->set_watermarks_table((smu), (tab), (clock_ranges)) : 0) -#define smu_get_current_clk_freq_by_table(smu, clk_type, value) \ - ((smu)->ppt_funcs->get_current_clk_freq_by_table ? (smu)->ppt_funcs->get_current_clk_freq_by_table((smu), (clk_type), (value)) : 0) -#define smu_thermal_temperature_range_update(smu, range, rw) \ - ((smu)->ppt_funcs->thermal_temperature_range_update? (smu)->ppt_funcs->thermal_temperature_range_update((smu), (range), (rw)) : 0) -#define smu_get_thermal_temperature_range(smu, range) \ - ((smu)->ppt_funcs->get_thermal_temperature_range? (smu)->ppt_funcs->get_thermal_temperature_range((smu), (range)) : 0) -#define smu_register_irq_handler(smu) \ - ((smu)->ppt_funcs->register_irq_handler ? (smu)->ppt_funcs->register_irq_handler(smu) : 0) - -#define smu_get_dpm_ultimate_freq(smu, param, min, max) \ - ((smu)->ppt_funcs->get_dpm_ultimate_freq ? (smu)->ppt_funcs->get_dpm_ultimate_freq((smu), (param), (min), (max)) : 0) - -#define smu_asic_set_performance_level(smu, level) \ - ((smu)->ppt_funcs->set_performance_level? (smu)->ppt_funcs->set_performance_level((smu), (level)) : -EINVAL); -#define smu_dump_pptable(smu) \ - ((smu)->ppt_funcs->dump_pptable ? (smu)->ppt_funcs->dump_pptable((smu)) : 0) -#define smu_get_dpm_clk_limited(smu, clk_type, dpm_level, freq) \ - ((smu)->ppt_funcs->get_dpm_clk_limited ? (smu)->ppt_funcs->get_dpm_clk_limited((smu), (clk_type), (dpm_level), (freq)) : -EINVAL) - -#define smu_set_soft_freq_limited_range(smu, clk_type, min, max) \ - ((smu)->ppt_funcs->set_soft_freq_limited_range ? (smu)->ppt_funcs->set_soft_freq_limited_range((smu), (clk_type), (min), (max)) : -EINVAL) - -#define smu_override_pcie_parameters(smu) \ - ((smu)->ppt_funcs->override_pcie_parameters ? (smu)->ppt_funcs->override_pcie_parameters((smu)) : 0) - -#define smu_update_pcie_parameters(smu, pcie_gen_cap, pcie_width_cap) \ - ((smu)->ppt_funcs->update_pcie_parameters ? (smu)->ppt_funcs->update_pcie_parameters((smu), (pcie_gen_cap), (pcie_width_cap)) : 0) - -#define smu_set_thermal_range(smu, range) \ - ((smu)->ppt_funcs->set_thermal_range ? (smu)->ppt_funcs->set_thermal_range((smu), (range)) : 0) - -#define smu_disable_umc_cdr_12gbps_workaround(smu) \ - ((smu)->ppt_funcs->disable_umc_cdr_12gbps_workaround ? (smu)->ppt_funcs->disable_umc_cdr_12gbps_workaround((smu)) : 0) - -#define smu_set_power_source(smu, power_src) \ - ((smu)->ppt_funcs->set_power_source ? (smu)->ppt_funcs->set_power_source((smu), (power_src)) : 0) - -#define smu_i2c_eeprom_init(smu, control) \ - ((smu)->ppt_funcs->i2c_eeprom_init ? (smu)->ppt_funcs->i2c_eeprom_init((smu), (control)) : 0) -#define smu_i2c_eeprom_fini(smu, control) \ - ((smu)->ppt_funcs->i2c_eeprom_fini ? (smu)->ppt_funcs->i2c_eeprom_fini((smu), (control)) : 0) -#define smu_get_unique_id(smu) \ - ((smu)->ppt_funcs->get_unique_id ? (smu)->ppt_funcs->get_unique_id((smu)) : 0) - -#define smu_log_thermal_throttling(smu) \ - ((smu)->ppt_funcs->log_thermal_throttling_event ? (smu)->ppt_funcs->log_thermal_throttling_event((smu)) : 0) + smu_ppt_xxx(get_profiling_clk_mask, 0, smu, level, sclk_mask, mclk_mask, soc_mask) -#define smu_get_asic_power_limits(smu) \ - ((smu)->ppt_funcs->get_power_limit ? (smu)->ppt_funcs->get_power_limit((smu)) : 0) #endif -- 2.17.1