[PATCH] drm/amd/pm: add pptable_funcs documentation (v2)

Quan, Evan Evan.Quan at amd.com
Fri Dec 18 02:51:28 UTC 2020


[AMD Official Use Only - Internal Distribution Only]

Thanks for adding this.
Except one comment below, the patch is reviewed-by: Evan Quan <evan.quan at amd.com>

-----Original Message-----
From: amd-gfx <amd-gfx-bounces at lists.freedesktop.org> On Behalf Of Ryan Taylor
Sent: Friday, December 18, 2020 3:09 AM
To: amd-gfx at lists.freedesktop.org
Cc: Taylor, Ryan <Ryan.Taylor at amd.com>
Subject: [PATCH] drm/amd/pm: add pptable_funcs documentation (v2)

Documents the hooks in struct pptable_funcs.

v2: Improved documentation accuracy.
Signed-off-by: Ryan Taylor <Ryan.Taylor at amd.com>
---
 drivers/gpu/drm/amd/pm/inc/amdgpu_smu.h | 525 ++++++++++++++++++++++++
 1 file changed, 525 insertions(+)

diff --git a/drivers/gpu/drm/amd/pm/inc/amdgpu_smu.h b/drivers/gpu/drm/amd/pm/inc/amdgpu_smu.h
index 4bdbcce7092d..b6578550da46 100644
--- a/drivers/gpu/drm/amd/pm/inc/amdgpu_smu.h
+++ b/drivers/gpu/drm/amd/pm/inc/amdgpu_smu.h
@@ -469,120 +469,645 @@ struct smu_context

 struct i2c_adapter;

+/**
+ * struct pptable_funcs - Callbacks used to interact with the SMU.
+ */
 struct pptable_funcs {
+/**
+ * @run_btc: Calibrate voltage/frequency curve to fit the system's
+ *           power delivery and voltage margins. Required for adaptive
+ *           voltage frequency scaling (AVFS).
+ */
 int (*run_btc)(struct smu_context *smu);
+
+/**
+ * @get_allowed_feature_mask: Get allowed feature mask.
+ * &feature_mask: Array to store feature mask.
+ * &num: Elements in &feature_mask.
+ */
 int (*get_allowed_feature_mask)(struct smu_context *smu, uint32_t *feature_mask, uint32_t num);
+
+/**
+ * @get_current_power_state: Get the current power state.
+ *
+ * Return: Current power state on success, negative errno on failure.
+ */
 enum amd_pm_state_type (*get_current_power_state)(struct smu_context *smu);
+
+/**
+ * @set_default_dpm_table: Populate the dpm table with defaults.
+ */
 int (*set_default_dpm_table)(struct smu_context *smu);
+
 int (*set_power_state)(struct smu_context *smu);
+
+/**
+ * @populate_umd_state_clk: Populate the UMD power state table with
+ *                          defaults.
+ */
 int (*populate_umd_state_clk)(struct smu_context *smu);
+
+/**
+ * @print_clk_levels: Print DPM clock levels for a clock domain
+ *                    to buffer. Star current level.
+ *
+ * Used for sysfs interfaces.
+ */
 int (*print_clk_levels)(struct smu_context *smu, enum smu_clk_type clk_type, char *buf);
+
+/**
+ * @force_clk_levels: Set a range of allowed DPM levels for a clock
+ *                    domain.
+ * &clk_type: Clock domain.
+ * &mask: Range of allowed DPM levels.
+ */
 int (*force_clk_levels)(struct smu_context *smu, enum smu_clk_type clk_type, uint32_t mask);
+
+/**
+ * @od_edit_dpm_table: Edit the custom overdrive DPM table.
+ * &type: Type of edit.
+ * &input: Edit parameters.
+ * &size: Size of &input.
+ */
 int (*od_edit_dpm_table)(struct smu_context *smu,
  enum PP_OD_DPM_TABLE_COMMAND type,
  long *input, uint32_t size);
+
+/**
+ * @get_clock_by_type_with_latency: Get the speed and latency of a clock
+ *                                  domain.
+ */
 int (*get_clock_by_type_with_latency)(struct smu_context *smu,
       enum smu_clk_type clk_type,
       struct
       pp_clock_levels_with_latency
       *clocks);
+/**
+ * @get_clock_by_type_with_voltage: Get the speed and voltage of a clock
+ *                                  domain.
+ */
+int (*get_clock_by_type_with_voltage)(struct smu_context *smu,
+      enum amd_pp_clock_type type,
+      struct
+      pp_clock_levels_with_voltage
+      *clocks);
+
+/**
+ * @get_power_profile_mode: Print all power profile modes to
+ *                          buffer. Star current mode.
+ */
 int (*get_power_profile_mode)(struct smu_context *smu, char *buf);
+
+/**
+ * @set_power_profile_mode: Set a power profile mode. Also used to
+ *                          create/set custom power profile modes.
+ * &input: Power profile mode parameters.
+ * &size: Size of &input.
+ */
 int (*set_power_profile_mode)(struct smu_context *smu, long *input, uint32_t size);
+
+/**
+ * @dpm_set_vcn_enable: Enable/disable VCN engine dynamic power
+ *                      management.
+ */
 int (*dpm_set_vcn_enable)(struct smu_context *smu, bool enable);
+
+/**
+ * @dpm_set_jpeg_enable: Enable/disable JPEG engine dynamic power
+ *                       management.
+ */
 int (*dpm_set_jpeg_enable)(struct smu_context *smu, bool enable);
+
+/**
+ * @read_sensor: Read data from a sensor.
+ * &sensor: Sensor to read data from.
+ * &data: Sensor reading.
+ * &size: Size of &data.
+ */
 int (*read_sensor)(struct smu_context *smu, enum amd_pp_sensors sensor,
    void *data, uint32_t *size);
+
+/**
+ * @pre_display_config_changed: Prepare GPU for a display configuration
+ *                              change.
+ *
+ * Disable display tracking and pin memory clock speed to maximum. Used
+ * in display component synchronization.
+ */
 int (*pre_display_config_changed)(struct smu_context *smu);
+
+/**
+ * @display_config_changed: Notify the SMU of the current display
+ *                          configuration.
+ *
+ * Allows SMU to properly track blanking periods for memory clock
+ * adjustment. Used in display component synchronization.
+ */
 int (*display_config_changed)(struct smu_context *smu);
+
 int (*apply_clocks_adjust_rules)(struct smu_context *smu);
+
+/**
+ * @notify_smc_display_config: Applies display requirements to the
+ *                             current power state.
+ *
+ * Optimize deep sleep DCEFclk and mclk for the current display
+ * configuration. Used in display component synchronization.
+ */
 int (*notify_smc_display_config)(struct smu_context *smu);
+
+/**
+ * @is_dpm_running: Check if DPM is running.
+ *
+ * Return: True if DPM is running, false otherwise.
+ */
 bool (*is_dpm_running)(struct smu_context *smu);
+
+/**
+ * @get_fan_speed_rpm: Get the current fan speed in RPM.
+ */
 int (*get_fan_speed_rpm)(struct smu_context *smu, uint32_t *speed);
+
+/**
+ * @set_watermarks_table: Configure and upload the watermarks tables to
+ *                        the SMU.
+ */
 int (*set_watermarks_table)(struct smu_context *smu,
     struct pp_smu_wm_range_sets *clock_ranges);
+
+/**
+ * @get_thermal_temperature_range: Get safe thermal limits in Celcius.
+ */
 int (*get_thermal_temperature_range)(struct smu_context *smu, struct smu_temperature_range *range);
+
+/**
+ * @get_uclk_dpm_states: Get memory clock DPM levels in kHz.
+ * &clocks_in_khz: Array of DPM levels.
+ * &num_states: Elements in &clocks_in_khz.
+ */
 int (*get_uclk_dpm_states)(struct smu_context *smu, uint32_t *clocks_in_khz, uint32_t *num_states);
+
+/**
+ * @set_default_od_settings: Set the overdrive tables to defaults.
+ */
[Quan, Evan] Better to describe this a "retrieve the default overdrive settings from SMU"
 int (*set_default_od_settings)(struct smu_context *smu);
+
+/**
+ * @set_performance_level: Set a performance level.
+ */
 int (*set_performance_level)(struct smu_context *smu, enum amd_dpm_forced_level level);
+
+/**
+ * @display_disable_memory_clock_switch: Enable/disable dynamic memory
+ *                                       clock switching.
+ *
+ * Disabling this feature forces memory clock speed to maximum.
+ * Enabling sets the minimum memory clock capable of driving the
+ * current display configuration.
+ */
 int (*display_disable_memory_clock_switch)(struct smu_context *smu, bool disable_memory_clock_switch);
+
+/**
+ * @dump_pptable: Print the power play table to the system log.
+ */
 void (*dump_pptable)(struct smu_context *smu);
+
+/**
+ * @get_power_limit: Get the device's power limits.
+ */
 int (*get_power_limit)(struct smu_context *smu);
+
+/**
+ * @set_df_cstate: Set data fabric cstate.
+ */
 int (*set_df_cstate)(struct smu_context *smu, enum pp_df_cstate state);
+
+/**
+ * @allow_xgmi_power_down: Enable/disable external global memory
+ *                         interconnect power down.
+ */
 int (*allow_xgmi_power_down)(struct smu_context *smu, bool en);
+
+/**
+ * @update_pcie_parameters: Update and upload the system's PCIe
+ *                          capabilites to the SMU.
+ * &pcie_gen_cap: Maximum allowed PCIe generation.
+ * &pcie_width_cap: Maximum allowed PCIe width.
+ */
 int (*update_pcie_parameters)(struct smu_context *smu, uint32_t pcie_gen_cap, uint32_t pcie_width_cap);
+
+/**
+ * @i2c_init: Initialize i2c.
+ *
+ * The i2c bus is used internally by the SMU voltage regulators and
+ * other devices. The i2c's EEPROM also stores bad page tables on boards
+ * with ECC.
+ */
 int (*i2c_init)(struct smu_context *smu, struct i2c_adapter *control);
+
+/**
+ * @i2c_fini: Tear down i2c.
+ */
 void (*i2c_fini)(struct smu_context *smu, struct i2c_adapter *control);
+
+/**
+ * @get_unique_id: Get the GPU's unique id. Used for asset tracking.
+ */
 void (*get_unique_id)(struct smu_context *smu);
+
+/**
+ * @get_dpm_clock_table: Get a copy of the DPM clock table.
+ *
+ * Used by display component in bandwidth and watermark calculations.
+ */
 int (*get_dpm_clock_table)(struct smu_context *smu, struct dpm_clocks *clock_table);
+
+/**
+ * @init_microcode: Request the SMU's firmware from the kernel.
+ */
 int (*init_microcode)(struct smu_context *smu);
+
+/**
+ * @load_microcode: Load firmware onto the SMU.
+ */
 int (*load_microcode)(struct smu_context *smu);
+
+/**
+ * @fini_microcode: Release the SMU's firmware.
+ */
 void (*fini_microcode)(struct smu_context *smu);
+
+/**
+ * @init_smc_tables: Initialize the SMU tables.
+ */
 int (*init_smc_tables)(struct smu_context *smu);
+
+/**
+ * @fini_smc_tables: Release the SMU tables.
+ */
 int (*fini_smc_tables)(struct smu_context *smu);
+
+/**
+ * @init_power: Initialize the power gate table context.
+ */
 int (*init_power)(struct smu_context *smu);
+
+/**
+ * @fini_power: Release the power gate table context.
+ */
 int (*fini_power)(struct smu_context *smu);
+
+/**
+ * @check_fw_status: Check the SMU's firmware status.
+ *
+ * Return: Zero if check passes, negative errno on failure.
+ */
 int (*check_fw_status)(struct smu_context *smu);
+
+/**
+ * @setup_pptable: Initialize the power play table and populate it with
+ *                 default values.
+ */
 int (*setup_pptable)(struct smu_context *smu);
+
+/**
+ * @get_vbios_bootup_values: Get default boot values from the VBIOS.
+ */
 int (*get_vbios_bootup_values)(struct smu_context *smu);
+
+/**
+ * @check_fw_version: Print driver and SMU interface versions to the
+ *                    system log.
+ *
+ * Interface mismatch is not a critical failure.
+ */
 int (*check_fw_version)(struct smu_context *smu);
+
+/**
+ * @powergate_sdma: Power up/down system direct memory access.
+ */
 int (*powergate_sdma)(struct smu_context *smu, bool gate);
+
+/**
+ * @set_gfx_cgpg: Enable/disable graphics engine course grain power
+ *                gating.
+ */
 int (*set_gfx_cgpg)(struct smu_context *smu, bool enable);
+
+/**
+ * @write_pptable: Write the power play table to the SMU.
+ */
 int (*write_pptable)(struct smu_context *smu);
+
+/**
+ * @set_driver_table_location: Send the location of the driver table to
+ *                             the SMU.
+ */
 int (*set_driver_table_location)(struct smu_context *smu);
+
+/**
+ * @set_tool_table_location: Send the location of the tool table to the
+ *                           SMU.
+ */
 int (*set_tool_table_location)(struct smu_context *smu);
+
+/**
+ * @notify_memory_pool_location: Send the location of the memory pool to
+ *                               the SMU.
+ */
 int (*notify_memory_pool_location)(struct smu_context *smu);
+
+/**
+ * @system_features_control: Enable/disable all SMU features.
+ */
 int (*system_features_control)(struct smu_context *smu, bool en);
+
+/**
+ * @send_smc_msg_with_param: Send a message with a parameter to the SMU.
+ * &msg: Type of message.
+ * &param: Message parameter.
+ * &read_arg: SMU response (optional).
+ */
 int (*send_smc_msg_with_param)(struct smu_context *smu,
        enum smu_message_type msg, uint32_t param, uint32_t *read_arg);
+
+/**
+ * @send_smc_msg: Send a message to the SMU.
+ * &msg: Type of message.
+ * &read_arg: SMU response (optional).
+ */
 int (*send_smc_msg)(struct smu_context *smu,
     enum smu_message_type msg,
     uint32_t *read_arg);
+
+/**
+ * @init_display_count: Notify the SMU of the number of display
+ *                      components in current display configuration.
+ */
 int (*init_display_count)(struct smu_context *smu, uint32_t count);
+
+/**
+ * @set_allowed_mask: Notify the SMU of the features currently allowed
+ *                    by the driver.
+ */
 int (*set_allowed_mask)(struct smu_context *smu);
+
+/**
+ * @get_enabled_mask: Get a mask of features that are currently enabled
+ *                    on the SMU.
+ * &feature_mask: Array representing enabled feature mask.
+ * &num: Elements in &feature_mask.
+ */
 int (*get_enabled_mask)(struct smu_context *smu, uint32_t *feature_mask, uint32_t num);
+
+/**
+ * @feature_is_enabled: Test if a feature is enabled.
+ *
+ * Return: One if enabled, zero if disabled.
+ */
 int (*feature_is_enabled)(struct smu_context *smu, enum smu_feature_mask mask);
+
+/**
+ * @disable_all_features_with_exception: Disable all features with
+ *                                       exception to those in &mask.
+ */
 int (*disable_all_features_with_exception)(struct smu_context *smu, enum smu_feature_mask mask);
+
+/**
+ * @notify_display_change: Enable fast memory clock switching.
+ *
+ * Allows for fine grained memory clock switching but has more stringent
+ * timing requirements.
+ */
 int (*notify_display_change)(struct smu_context *smu);
+
+/**
+ * @set_power_limit: Set power limit in watts.
+ */
 int (*set_power_limit)(struct smu_context *smu, uint32_t n);
+
+/**
+ * @init_max_sustainable_clocks: Populate max sustainable clock speed
+ *                               table with values from the SMU.
+ */
 int (*init_max_sustainable_clocks)(struct smu_context *smu);
+
+/**
+ * @enable_thermal_alert: Enable thermal alert interrupts.
+ */
 int (*enable_thermal_alert)(struct smu_context *smu);
+
+/**
+ * @disable_thermal_alert: Disable thermal alert interrupts.
+ */
 int (*disable_thermal_alert)(struct smu_context *smu);
+
+/**
+ * @set_min_dcef_deep_sleep: Set a minimum display fabric deep sleep
+ *                           clock speed in MHz.
+ */
 int (*set_min_dcef_deep_sleep)(struct smu_context *smu, uint32_t clk);
+
+/**
+ * @display_clock_voltage_request: Set a hard minimum frequency
+ * for a clock domain.
+ */
 int (*display_clock_voltage_request)(struct smu_context *smu, struct
      pp_display_clock_request
      *clock_req);
+
+/**
+ * @get_fan_control_mode: Get the current fan control mode.
+ */
 uint32_t (*get_fan_control_mode)(struct smu_context *smu);
+
+/**
+ * @set_fan_control_mode: Set the fan control mode.
+ */
 int (*set_fan_control_mode)(struct smu_context *smu, uint32_t mode);
+
+/**
+ * @set_fan_speed_rpm: Set a static fan speed in RPM.
+ */
 int (*set_fan_speed_rpm)(struct smu_context *smu, uint32_t speed);
+
+/**
+ * @set_xgmi_pstate: Set inter-chip global memory interconnect pstate.
+ * &pstate: Pstate to set. D0 if Nonzero, D3 otherwise.
+ */
 int (*set_xgmi_pstate)(struct smu_context *smu, uint32_t pstate);
+
+/**
+ * @gfx_off_control: Enable/disable graphics engine poweroff.
+ */
 int (*gfx_off_control)(struct smu_context *smu, bool enable);
+
+
+/**
+ * @get_gfx_off_status: Get graphics engine poweroff status.
+ *
+ * Return:
+ * 0 - GFXOFF(default).
+ * 1 - Transition out of GFX State.
+ * 2 - Not in GFXOFF.
+ * 3 - Transition into GFXOFF.
+ */
 uint32_t (*get_gfx_off_status)(struct smu_context *smu);
+
+/**
+ * @register_irq_handler: Register interupt request handlers.
+ */
 int (*register_irq_handler)(struct smu_context *smu);
+
+/**
+ * @set_azalia_d3_pme: Wake the audio decode engine from d3 sleep.
+ */
 int (*set_azalia_d3_pme)(struct smu_context *smu);
+
+/**
+ * @get_max_sustainable_clocks_by_dc: Get a copy of the max sustainable
+ *                                    clock speeds table.
+ *
+ * Provides a way for the display component (DC) to get the max
+ * sustainable clocks from the SMU.
+ */
 int (*get_max_sustainable_clocks_by_dc)(struct smu_context *smu, struct pp_smu_nv_clock_table *max_clocks);
+
+/**
+ * @baco_is_support: Check if GPU supports BACO (Bus Active, Chip Off).
+ */
 bool (*baco_is_support)(struct smu_context *smu);
+
+/**
+ * @baco_get_state: Get the current BACO state.
+ *
+ * Return: Current BACO state.
+ */
 enum smu_baco_state (*baco_get_state)(struct smu_context *smu);
+
+/**
+ * @baco_set_state: Enter/exit BACO.
+ */
 int (*baco_set_state)(struct smu_context *smu, enum smu_baco_state state);
+
+/**
+ * @baco_enter: Enter BACO.
+ */
 int (*baco_enter)(struct smu_context *smu);
+
+/**
+ * @baco_exit: Exit Baco.
+ */
 int (*baco_exit)(struct smu_context *smu);
+
+/**
+ * @mode1_reset_is_support: Check if GPU supports mode1 reset.
+ */
 bool (*mode1_reset_is_support)(struct smu_context *smu);
+
+/**
+ * @mode1_reset: Perform mode1 reset.
+ *
+ * Complete GPU reset.
+ */
 int (*mode1_reset)(struct smu_context *smu);
+
+/**
+ * @mode2_reset: Perform mode2 reset.
+ *
+ * Mode2 reset generally does not reset as many IPs as mode1 reset. The
+ * IPs reset varies by asic.
+ */
 int (*mode2_reset)(struct smu_context *smu);
+
+/**
+ * @get_dpm_ultimate_freq: Get the hard frequency range of a clock
+ *                         domain in MHz.
+ */
 int (*get_dpm_ultimate_freq)(struct smu_context *smu, enum smu_clk_type clk_type, uint32_t *min, uint32_t *max);
+
+/**
+ * @set_soft_freq_limited_range: Set the soft frequency range of a clock
+ *                               domain in MHz.
+ */
 int (*set_soft_freq_limited_range)(struct smu_context *smu, enum smu_clk_type clk_type, uint32_t min, uint32_t max);
+
+/**
+ * @set_power_source: Notify the SMU of the current power source.
+ */
 int (*set_power_source)(struct smu_context *smu, enum smu_power_src_type power_src);
+
+/**
+ * @log_thermal_throttling_event: Print a thermal throttling warning to
+ *                                the system's log.
+ */
 void (*log_thermal_throttling_event)(struct smu_context *smu);
+
+/**
+ * @get_pp_feature_mask: Print a human readable table of enabled
+ *                       features to buffer.
+ */
 size_t (*get_pp_feature_mask)(struct smu_context *smu, char *buf);
+
+/**
+ * @set_pp_feature_mask: Request the SMU enable/disable features to
+ *                       match those enabled in &new_mask.
+ */
 int (*set_pp_feature_mask)(struct smu_context *smu, uint64_t new_mask);
+
+/**
+ * @get_gpu_metrics: Get a copy of the GPU metrics table from the SMU.
+ *
+ * Return: Size of &table
+ */
 ssize_t (*get_gpu_metrics)(struct smu_context *smu, void **table);
+
+/**
+ * @enable_mgpu_fan_boost: Enable multi-GPU fan boost.
+ */
 int (*enable_mgpu_fan_boost)(struct smu_context *smu);
+
+/**
+ * @gfx_ulv_control: Enable/disable ultra low voltage.
+ */
 int (*gfx_ulv_control)(struct smu_context *smu, bool enablement);
+
+/**
+ * @deep_sleep_control: Enable/disable deep sleep.
+ */
 int (*deep_sleep_control)(struct smu_context *smu, bool enablement);
+
+/**
+ * @get_fan_parameters: Get fan parameters.
+ *
+ * Get maximum fan speed from the power play table.
+ */
 int (*get_fan_parameters)(struct smu_context *smu);
+
+/**
+ * @post_init: Helper function for asic specific workarounds.
+ */
 int (*post_init)(struct smu_context *smu);
+
+/**
+ * @interrupt_work: Work task scheduled from SMU interrupt handler.
+ */
 void (*interrupt_work)(struct smu_context *smu);
+
+/**
+ * @gpo_control: Enable/disable graphics power optimization if supported.
+ */
 int (*gpo_control)(struct smu_context *smu, bool enablement);
+
+/**
+ * @gfx_state_change_set: Send the current graphics state to the SMU.
+ */
 int (*gfx_state_change_set)(struct smu_context *smu, uint32_t state);
+
+/**
+ * @set_fine_grain_gfx_freq_parameters: Set fine grain graphics clock
+ *                                      parameters to defaults.
+ */
 int (*set_fine_grain_gfx_freq_parameters)(struct smu_context *smu);
 };

--
2.29.2

_______________________________________________
amd-gfx mailing list
amd-gfx at lists.freedesktop.org
https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.freedesktop.org%2Fmailman%2Flistinfo%2Famd-gfx&data=04%7C01%7Cevan.quan%40amd.com%7C0bbf4131dabb4c0fd7c808d8a2bf6872%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637438290272670635%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&sdata=oTQTMp8%2FkTmH8LL%2FUhE%2B%2Bnm4Q1pM1wLKK%2FRLJX%2FSTA8%3D&reserved=0


More information about the amd-gfx mailing list