[Public] Reviewed-by: Tim Huang <tim.hu...@amd.com>
Best Regards, Tim Huang > -----Original Message----- > From: Jesse Zhang <jesse.zh...@amd.com> > Sent: Monday, April 29, 2024 3:29 PM > To: amd-gfx@lists.freedesktop.org > Cc: Deucher, Alexander <alexander.deuc...@amd.com>; Koenig, Christian > <christian.koe...@amd.com>; Huang, Tim <tim.hu...@amd.com>; Zhang, > Jesse(Jie) <jesse.zh...@amd.com>; Zhang, Jesse(Jie) <jesse.zh...@amd.com> > Subject: [PATCH V2] drm/amd/pm: fix warning using uninitialized value of > max_vid_step > > Check the return of pp_atomfwctrl_get_Voltage_table_v4 > as it may fail to initialize max_vid_step > V2: change the check condition (Tim Huang) > > Signed-off-by: Jesse Zhang <jesse.zh...@amd.com> > --- > drivers/gpu/drm/amd/pm/powerplay/hwmgr/vega10_hwmgr.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/pm/powerplay/hwmgr/vega10_hwmgr.c > b/drivers/gpu/drm/amd/pm/powerplay/hwmgr/vega10_hwmgr.c > index b602059436a8..d004cdbe97b4 100644 > --- a/drivers/gpu/drm/amd/pm/powerplay/hwmgr/vega10_hwmgr.c > +++ b/drivers/gpu/drm/amd/pm/powerplay/hwmgr/vega10_hwmgr.c > @@ -2573,8 +2573,11 @@ static int vega10_init_smc_table(struct pp_hwmgr > *hwmgr) > } > } > > - pp_atomfwctrl_get_voltage_table_v4(hwmgr, VOLTAGE_TYPE_VDDC, > + result = pp_atomfwctrl_get_voltage_table_v4(hwmgr, > VOLTAGE_TYPE_VDDC, > VOLTAGE_OBJ_SVID2, &voltage_table); > + PP_ASSERT_WITH_CODE(!result, > + "Failed to get voltage table!", > + return result); > pp_table->MaxVidStep = voltage_table.max_vid_step; > > pp_table->GfxDpmVoltageMode = > -- > 2.25.1