> -----Original Message-----
> From: amd-gfx [mailto:amd-gfx-boun...@lists.freedesktop.org] On Behalf
> Of Rex Zhu
> Sent: Wednesday, August 02, 2017 7:05 AM
> To: amd-gfx@lists.freedesktop.org
> Cc: Deucher, Alexander; Zhu, Rex
> Subject: [PATCH 3/3] drm/amd/powerplay: add CZ profile support
> 
> From: Alex Deucher <alexander.deuc...@amd.com>
> 
> Support the profiling modes for sclk.
> 
> v2: delete profileing mode for mclk.
> 
> Change-Id: I07418e52c37f8b67fd02426ddf4e166eeb091c7d
> Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>
> Signed-off-by: Rex Zhu <rex....@amd.com>
> Tested-and-Reviewed-by Rex Zhu <rex....@amd.com>

Series is:
Reviewed-by: Alex Deucher <alexander.deuc...@amd.com>

> ---
>  drivers/gpu/drm/amd/powerplay/hwmgr/cz_hwmgr.c | 66
> +++++++++++++++++++++++++-
>  1 file changed, 64 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/cz_hwmgr.c
> b/drivers/gpu/drm/amd/powerplay/hwmgr/cz_hwmgr.c
> index 2f73c04..27b21d4 100644
> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/cz_hwmgr.c
> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/cz_hwmgr.c
> @@ -1322,33 +1322,95 @@ static int cz_phm_force_dpm_sclk(struct
> pp_hwmgr *hwmgr, uint32_t sclk)
>       return 0;
>  }
> 
> +static int cz_get_profiling_clk(struct pp_hwmgr *hwmgr, uint32_t *sclk)
> +{
> +     struct phm_clock_voltage_dependency_table *table =
> +             hwmgr->dyn_state.vddc_dependency_on_sclk;
> +     int32_t tmp_sclk;
> +     int32_t count;
> +
> +     tmp_sclk = table->entries[table->count-1].clk * 70 / 100;
> +
> +     for (count = table->count-1; count >= 0; count--) {
> +             if (tmp_sclk >= table->entries[count].clk) {
> +                     tmp_sclk = table->entries[count].clk;
> +                     *sclk = tmp_sclk;
> +                     break;
> +             }
> +     }
> +     if (count < 0)
> +             *sclk = table->entries[0].clk;
> +
> +     return 0;
> +}
> +
>  static int cz_dpm_force_dpm_level(struct pp_hwmgr *hwmgr,
>                               enum amd_dpm_forced_level level)
>  {
> +     uint32_t sclk = 0;
>       int ret = 0;
> +     uint32_t profile_mode_mask =
> AMD_DPM_FORCED_LEVEL_PROFILE_STANDARD |
> +
>       AMD_DPM_FORCED_LEVEL_PROFILE_MIN_SCLK |
> +
>       AMD_DPM_FORCED_LEVEL_PROFILE_PEAK;
> +
> +     if (level == hwmgr->dpm_level)
> +             return ret;
> +
> +     if (!(hwmgr->dpm_level & profile_mode_mask)) {
> +             /* enter profile mode, save current level, disable gfx cg*/
> +             if (level & profile_mode_mask) {
> +                     hwmgr->saved_dpm_level = hwmgr->dpm_level;
> +                     cgs_set_clockgating_state(hwmgr->device,
> +                                             AMD_IP_BLOCK_TYPE_GFX,
> +                                             AMD_CG_STATE_UNGATE);
> +             }
> +     } else {
> +             /* exit profile mode, restore level, enable gfx cg*/
> +             if (!(level & profile_mode_mask)) {
> +                     if (level ==
> AMD_DPM_FORCED_LEVEL_PROFILE_EXIT)
> +                             level = hwmgr->saved_dpm_level;
> +                     cgs_set_clockgating_state(hwmgr->device,
> +                                     AMD_IP_BLOCK_TYPE_GFX,
> +                                     AMD_CG_STATE_GATE);
> +             }
> +     }
> 
>       switch (level) {
>       case AMD_DPM_FORCED_LEVEL_HIGH:
> +     case AMD_DPM_FORCED_LEVEL_PROFILE_PEAK:
>               ret = cz_phm_force_dpm_highest(hwmgr);
>               if (ret)
>                       return ret;
> +             hwmgr->dpm_level = level;
>               break;
>       case AMD_DPM_FORCED_LEVEL_LOW:
> +     case AMD_DPM_FORCED_LEVEL_PROFILE_MIN_SCLK:
>               ret = cz_phm_force_dpm_lowest(hwmgr);
>               if (ret)
>                       return ret;
> +             hwmgr->dpm_level = level;
>               break;
>       case AMD_DPM_FORCED_LEVEL_AUTO:
>               ret = cz_phm_unforce_dpm_levels(hwmgr);
>               if (ret)
>                       return ret;
> +             hwmgr->dpm_level = level;
>               break;
> +     case AMD_DPM_FORCED_LEVEL_PROFILE_STANDARD:
> +             ret = cz_get_profiling_clk(hwmgr, &sclk);
> +             if (ret)
> +                     return ret;
> +             hwmgr->dpm_level = level;
> +             cz_phm_force_dpm_sclk(hwmgr, sclk);
> +             break;
> +     case AMD_DPM_FORCED_LEVEL_MANUAL:
> +             hwmgr->dpm_level = level;
> +             break;
> +     case AMD_DPM_FORCED_LEVEL_PROFILE_EXIT:
>       default:
>               break;
>       }
> 
> -     hwmgr->dpm_level = level;
> -
>       return ret;
>  }
> 
> --
> 1.9.1
> 
> _______________________________________________
> amd-gfx mailing list
> amd-gfx@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/amd-gfx
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to