[AMD Official Use Only - General] Hi Jesse,
> -----Original Message----- > From: amd-gfx <[email protected]> On Behalf Of Jesse > Zhang > Sent: Friday, May 10, 2024 10:50 AM > To: [email protected] > Cc: Deucher, Alexander <[email protected]>; Koenig, Christian > <[email protected]>; Huang, Tim <[email protected]>; Zhang, > Jesse(Jie) <[email protected]>; Zhang, Jesse(Jie) > <[email protected]> > Subject: [PATCH 09/22] drm/amd/pm: check specific index for smu13 > > Check for specific indexes that may be invalid values. > > Signed-off-by: Jesse Zhang <[email protected]> > --- > drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_6_ppt.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_6_ppt.c > b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_6_ppt.c > index 051092f1b1b4..7c343dd12a7f 100644 > --- a/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_6_ppt.c > +++ b/drivers/gpu/drm/amd/pm/swsmu/smu13/smu_v13_0_6_ppt.c > @@ -2336,6 +2336,8 @@ static int smu_v13_0_6_mode2_reset(struct > smu_context *smu) > > index = smu_cmn_to_asic_specific_index(smu, > CMN2ASIC_MAPPING_MSG, > SMU_MSG_GfxDeviceDriverReset); > + if (index < 0) > + ret = -EINVAL; We should need to return the index here? Tim > > mutex_lock(&smu->message_lock); > > -- > 2.25.1
