Thanks clean up the duplicate code. The patch is Reviewed-by: Prike Liang <Prike.Liang@xxxxxxx> Thanks, Prike > -----Original Message----- > From: Wang, Kevin(Yang) <Kevin1.Wang@xxxxxxx> > Sent: Monday, September 23, 2019 3:50 PM > To: amd-gfx@xxxxxxxxxxxxxxxxxxxxx > Cc: Huang, Ray <Ray.Huang@xxxxxxx>; Feng, Kenneth > <Kenneth.Feng@xxxxxxx>; Liang, Prike <Prike.Liang@xxxxxxx>; Wang, > Kevin(Yang) <Kevin1.Wang@xxxxxxx> > Subject: [PATCH] drm/amd/powerplay: remove duplicate macro of > smu_get_uclk_dpm_states > > remove duplicate macro of smu_get_uclk_dpm_states > > fix commit: > drm/amd/powerplay: add the interface for getting ultimate frequency v3 > > Signed-off-by: Kevin Wang <kevin1.wang@xxxxxxx> > --- > drivers/gpu/drm/amd/powerplay/inc/amdgpu_smu.h | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/gpu/drm/amd/powerplay/inc/amdgpu_smu.h > b/drivers/gpu/drm/amd/powerplay/inc/amdgpu_smu.h > index 45da21dc2356..5a0059b9aec9 100644 > --- a/drivers/gpu/drm/amd/powerplay/inc/amdgpu_smu.h > +++ b/drivers/gpu/drm/amd/powerplay/inc/amdgpu_smu.h > @@ -752,8 +752,6 @@ struct smu_funcs > ((smu)->ppt_funcs->get_uclk_dpm_states ? (smu)->ppt_funcs- > >get_uclk_dpm_states((smu), (clocks_in_khz), (num_states)) : 0) #define > smu_get_max_sustainable_clocks_by_dc(smu, max_clocks) \ > ((smu)->funcs->get_max_sustainable_clocks_by_dc ? (smu)->funcs- > >get_max_sustainable_clocks_by_dc((smu), (max_clocks)) : 0) -#define > smu_get_uclk_dpm_states(smu, clocks_in_khz, num_states) \ > - ((smu)->ppt_funcs->get_uclk_dpm_states ? (smu)->ppt_funcs- > >get_uclk_dpm_states((smu), (clocks_in_khz), (num_states)) : 0) > #define smu_baco_is_support(smu) \ > ((smu)->funcs->baco_is_support? (smu)->funcs- > >baco_is_support((smu)) : false) #define smu_baco_get_state(smu, state) \ > -- > 2.17.1 _______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx