RE: [PATCH] drm/amdgpu/powerplay: implement interface pp_power_profile_mode

["Quan, Evan" <Evan.Quan-5C7GfCeVMHo@public.gmane.org>]

Reviewed-by: Evan Quan <evan.quan@amd.com>

> -----Original Message-----
> From: Liang, Prike <Prike.Liang@amd.com>
> Sent: 2019年10月16日 16:24
> To: amd-gfx@lists.freedesktop.org
> Cc: Quan, Evan <Evan.Quan@amd.com>; Huang, Ray
> <Ray.Huang@amd.com>; Liang, Prike <Prike.Liang@amd.com>
> Subject: [PATCH] drm/amdgpu/powerplay: implement interface
> pp_power_profile_mode
> 
> implement get_power_profile_mode for getting power profile mode status.
> 
> Signed-off-by: Prike Liang <Prike.Liang@amd.com>
> ---
>  drivers/gpu/drm/amd/powerplay/renoir_ppt.c | 34
> ++++++++++++++++++++++++++++++
>  1 file changed, 34 insertions(+)
> 
> diff --git a/drivers/gpu/drm/amd/powerplay/renoir_ppt.c
> b/drivers/gpu/drm/amd/powerplay/renoir_ppt.c
> index fa314c2..953e347 100644
> --- a/drivers/gpu/drm/amd/powerplay/renoir_ppt.c
> +++ b/drivers/gpu/drm/amd/powerplay/renoir_ppt.c
> @@ -640,6 +640,39 @@ static int renoir_set_watermarks_table(
>  	return ret;
>  }
> 
> +static int renoir_get_power_profile_mode(struct smu_context *smu,
> +					   char *buf)
> +{
> +	static const char *profile_name[] = {
> +					"BOOTUP_DEFAULT",
> +					"3D_FULL_SCREEN",
> +					"POWER_SAVING",
> +					"VIDEO",
> +					"VR",
> +					"COMPUTE",
> +					"CUSTOM"};
> +	uint32_t i, size = 0;
> +	int16_t workload_type = 0;
> +
> +	if (!smu->pm_enabled || !buf)
> +		return -EINVAL;
> +
> +	for (i = 0; i <= PP_SMC_POWER_PROFILE_CUSTOM; i++) {
> +		/*
> +		 * Conv PP_SMC_POWER_PROFILE* to
> WORKLOAD_PPLIB_*_BIT
> +		 * Not all profile modes are supported on arcturus.
> +		 */
> +		workload_type = smu_workload_get_type(smu, i);
> +		if (workload_type < 0)
> +			continue;
> +
> +		size += sprintf(buf + size, "%2d %14s%s\n",
> +			i, profile_name[i], (i == smu->power_profile_mode) ?
> "*" : " ");
> +	}
> +
> +	return size;
> +}
> +
>  static const struct pptable_funcs renoir_ppt_funcs = {
>  	.get_smu_msg_index = renoir_get_smu_msg_index,
>  	.get_smu_table_index = renoir_get_smu_table_index, @@ -658,6
> +691,7 @@ static const struct pptable_funcs renoir_ppt_funcs = {
>  	.set_performance_level = renoir_set_performance_level,
>  	.get_dpm_clock_table = renoir_get_dpm_clock_table,
>  	.set_watermarks_table = renoir_set_watermarks_table,
> +	.get_power_profile_mode = renoir_get_power_profile_mode,
>  };
> 
>  void renoir_set_ppt_funcs(struct smu_context *smu)
> --
> 2.7.4

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx