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

["Quan, Evan" <Evan.Quan@xxxxxxx>]

[AMD Official Use Only]

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

> -----Original Message-----
> From: amd-gfx <amd-gfx-bounces@xxxxxxxxxxxxxxxxxxxxx> On Behalf Of Alex
> Deucher
> Sent: Saturday, January 22, 2022 4:22 AM
> To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx
> Cc: Deucher, Alexander <Alexander.Deucher@xxxxxxx>
> 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.deucher@xxxxxxx>
> ---
>  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