[Public] This patch is, Reviewed-by: Tim Huang <Tim.Huang@xxxxxxx> > -----Original Message----- > From: Jesse Zhang <jesse.zhang@xxxxxxx> > Sent: Monday, June 3, 2024 4:48 PM > To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx > Cc: Deucher, Alexander <Alexander.Deucher@xxxxxxx>; Koenig, Christian > <Christian.Koenig@xxxxxxx>; Kuehling, Felix <Felix.Kuehling@xxxxxxx>; > Huang, Tim <Tim.Huang@xxxxxxx>; Zhang, Jesse(Jie) > <Jesse.Zhang@xxxxxxx>; Zhang, Jesse(Jie) <Jesse.Zhang@xxxxxxx> > Subject: [PATCH 05/12] drm/amd/pm: remove dead code in > navi10_emit_clk_levels and navi10_print_clk_levels > > Since the range of the varibable i is 0 - 3. > So execution cannot reach this statement: default. > > Signed-off-by: Jesse Zhang <Jesse.Zhang@xxxxxxx> > --- > drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c > b/drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c > index cf556f1b5ed1..076620fa3ef5 100644 > --- a/drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c > +++ b/drivers/gpu/drm/amd/pm/swsmu/smu11/navi10_ppt.c > @@ -1389,8 +1389,6 @@ static int navi10_emit_clk_levels(struct > smu_context *smu, > case 2: > curve_settings = &od_table->GfxclkFreq3; > break; > - default: > - break; > } > *offset += sysfs_emit_at(buf, *offset, > "%d: %uMHz %umV\n", > i, curve_settings[0], > @@ -1594,8 +1592,6 @@ static int navi10_print_clk_levels(struct > smu_context *smu, > case 2: > curve_settings = &od_table->GfxclkFreq3; > break; > - default: > - break; > } > size += sysfs_emit_at(buf, size, > "%d: %uMHz %umV\n", > i, curve_settings[0], > -- > 2.25.1