RE: [PATCH] drm/amdgpu/pm/smu7: drop message about VI performance levels

[Quan, Evan]

[AMD Official Use Only]

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

> -----Original Message-----
> From: amd-gfx <amd-gfx-boun...@lists.freedesktop.org> On Behalf Of Alex
> Deucher
> Sent: Saturday, January 22, 2022 4:22 AM
> To: amd-gfx@lists.freedesktop.org
> Cc: Deucher, Alexander <alexander.deuc...@amd.com>
> Subject: [PATCH] drm/amdgpu/pm/smu7: drop message about VI
> performance levels
> 
> Earlier chips only had two performance levels, but newer
> ones potentially had more.  The message is harmless.  Drop the
> message to avoid spamming the log.
> 
> Bug:
> https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgitla
> b.freedesktop.org%2Fdrm%2Famd%2F-
> %2Fissues%2F1874&amp;data=04%7C01%7Cevan.quan%40amd.com%7C008
> 496779cf84e3eb9bc08d9dd1bb44e%7C3dd8961fe4884e608e11a82d994e183d
> %7C0%7C0%7C637783933331038923%7CUnknown%7CTWFpbGZsb3d8eyJWIj
> oiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3
> 000&amp;sdata=e53W24tcHU8hRut26458QE5v%2BrTk0xRd4eTMxaFn72E%3
> D&amp;reserved=0
> Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>
> ---
>  drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu7_hwmgr.c | 4 ----
>  1 file changed, 4 deletions(-)
> 
> diff --git a/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu7_hwmgr.c
> b/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu7_hwmgr.c
> index cd99db0dc2be..a1e11037831a 100644
> --- a/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu7_hwmgr.c
> +++ b/drivers/gpu/drm/amd/pm/powerplay/hwmgr/smu7_hwmgr.c
> @@ -3295,10 +3295,6 @@ static int smu7_apply_state_adjust_rules(struct
> pp_hwmgr *hwmgr,
>                       request_ps->classification.ui_label);
>       data->mclk_ignore_signal = false;
> 
> -     PP_ASSERT_WITH_CODE(smu7_ps->performance_level_count == 2,
> -                              "VI should always have 2 performance
> levels",
> -                             );
> -
>       max_limits = adev->pm.ac_power ?
>                       &(hwmgr->dyn_state.max_clock_voltage_on_ac) :
>                       &(hwmgr->dyn_state.max_clock_voltage_on_dc);
> --
> 2.34.1