[AMD Official Use Only - General]
> -----Original Message-----
> From: kunliu13 <Kun.Liu2@xxxxxxx>
> Sent: Tuesday, February 14, 2023 4:03 PM
> To: Limonciello, Mario <Mario.Limonciello@xxxxxxx>; Liang, Richard qi
> <Richardqi.Liang@xxxxxxx>; Yuan, Perry <Perry.Yuan@xxxxxxx>; amd-
> gfx@xxxxxxxxxxxxxxxxxxxxx
> Cc: Deucher, Alexander <Alexander.Deucher@xxxxxxx>; Du, Xiaojian
> <Xiaojian.Du@xxxxxxx>; Quan, Evan <Evan.Quan@xxxxxxx>; Liu, Kun
> <Kun.Liu2@xxxxxxx>
> Subject: [PATCH 2/2] drm/amdgpu: added a sysfs interface for thermal
> throttling
>
> implement apu_thermal_cap r/w callback for vangogh
>
> Jira ID: SWDEV-354511
[Quan, Evan] Please drop this internal link.
> Signed-off-by: Kun Liu <Kun.Liu2@xxxxxxx>
> ---
> .../gpu/drm/amd/pm/swsmu/smu11/vangogh_ppt.c | 23
> +++++++++++++++++++
> 1 file changed, 23 insertions(+)
>
> diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu11/vangogh_ppt.c
> b/drivers/gpu/drm/amd/pm/swsmu/smu11/vangogh_ppt.c
> index cb10c7e31..d211b1dfe 100644
> --- a/drivers/gpu/drm/amd/pm/swsmu/smu11/vangogh_ppt.c
> +++ b/drivers/gpu/drm/amd/pm/swsmu/smu11/vangogh_ppt.c
> @@ -1590,6 +1590,27 @@ static int vangogh_read_sensor(struct
> smu_context *smu,
> return ret;
> }
>
> +static int vangogh_get_apu_thermal_limit(struct smu_context *smu,
> uint32_t *limit)
> +{
> + int ret = -EINVAL;
> +
> + ret = smu_cmn_send_smc_msg_with_param(smu,
> + SMU_MSG_GetThermalLimit,
> + 0, limit);
> + return ret;
[Quan, Evan] "ret" here and below in vangogh_set_apu_thermal_limit seem unnecessary.
Other that those, the patch is reviewed-by: Evan Quan <evan.quan@xxxxxxx>
Evan
> +}
> +
> +int vangogh_set_apu_thermal_limit(struct smu_context *smu, uint32_t
> limit)
> +{
> + int ret = -EINVAL;
> +
> + ret = smu_cmn_send_smc_msg_with_param(smu,
> +
> SMU_MSG_SetReducedThermalLimit,
> + limit, NULL);
> + return ret;
> +}
> +
> +
> static int vangogh_set_watermarks_table(struct smu_context *smu,
> struct pp_smu_wm_range_sets
> *clock_ranges)
> {
> @@ -2425,6 +2446,8 @@ static const struct pptable_funcs
> vangogh_ppt_funcs = {
> .dpm_set_jpeg_enable = vangogh_dpm_set_jpeg_enable,
> .is_dpm_running = vangogh_is_dpm_running,
> .read_sensor = vangogh_read_sensor,
> + .get_apu_thermal_limit = vangogh_get_apu_thermal_limit,
> + .set_apu_thermal_limit = vangogh_set_apu_thermal_limit,
> .get_enabled_mask = smu_cmn_get_enabled_mask,
> .get_pp_feature_mask = smu_cmn_get_pp_feature_mask,
> .set_watermarks_table = vangogh_set_watermarks_table,
> --
> 2.25.1
