[PATCH] drm/amd/powerplay: optimize the mclk dpm policy settings

[Alex Deucher]

Ping?

On Thu, Jul 30, 2020 at 2:16 PM Alex Deucher <alexdeucher at gmail.com> wrote:
>
> From: Evan Quan <evan.quan at amd.com>
>
> Different mclk dpm policy will be applied based on the VRAM
> width.
>
> Signed-off-by: Evan Quan <evan.quan at amd.com>
> Signed-off-by: Alex Deucher <alexander.deucher at amd.com>
> ---
>  drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c | 16 +++++++++++++---
>  1 file changed, 13 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c b/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c
> index ffe05b7cc1f0..b81719433017 100644
> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c
> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c
> @@ -1585,9 +1585,19 @@ static void smu7_init_dpm_defaults(struct pp_hwmgr *hwmgr)
>         data->current_profile_setting.sclk_down_hyst = 100;
>         data->current_profile_setting.sclk_activity = SMU7_SCLK_TARGETACTIVITY_DFLT;
>         data->current_profile_setting.bupdate_mclk = 1;
> -       data->current_profile_setting.mclk_up_hyst = 0;
> -       data->current_profile_setting.mclk_down_hyst = 100;
> -       data->current_profile_setting.mclk_activity = SMU7_MCLK_TARGETACTIVITY_DFLT;
> +       if (adev->gmc.vram_width == 256) {
> +               data->current_profile_setting.mclk_up_hyst = 10;
> +               data->current_profile_setting.mclk_down_hyst = 60;
> +               data->current_profile_setting.mclk_activity = 25;
> +       } else if (adev->gmc.vram_width == 128) {
> +               data->current_profile_setting.mclk_up_hyst = 5;
> +               data->current_profile_setting.mclk_down_hyst = 16;
> +               data->current_profile_setting.mclk_activity = 20;
> +       } else if (adev->gmc.vram_width == 64) {
> +               data->current_profile_setting.mclk_up_hyst = 3;
> +               data->current_profile_setting.mclk_down_hyst = 16;
> +               data->current_profile_setting.mclk_activity = 20;
> +       }
>         hwmgr->workload_mask = 1 << hwmgr->workload_prority[PP_SMC_POWER_PROFILE_FULLSCREEN3D];
>         hwmgr->power_profile_mode = PP_SMC_POWER_PROFILE_FULLSCREEN3D;
>         hwmgr->default_power_profile_mode = PP_SMC_POWER_PROFILE_FULLSCREEN3D;
> --
> 2.25.4
>