On Fri, Nov 6, 2020 at 4:50 PM Lee Jones <lee.jones@xxxxxxxxxx> wrote: > > Also rid some unused ones. > > This patch solves 2000 warnings! > > Fixes the following W=1 kernel build warning(s): > > In file included from drivers/gpu/drm/amd/amdgpu/../display/dc/dc_types.h:33, > from drivers/gpu/drm/amd/amdgpu/../display/dc/dm_services_types.h:30, > from drivers/gpu/drm/amd/amdgpu/../display/dc/dm_services.h:35, > from drivers/gpu/drm/amd/amdgpu/../display/amdgpu_dm/amdgpu_dm_helpers.c:35: > At top level: > drivers/gpu/drm/amd/amdgpu/../display/include/fixed31_32.h:76:32: warning: ‘dc_fixpt_ln2_div_2’ defined but not used [-Wunused-const-variable=] > 76 | static const struct fixed31_32 dc_fixpt_ln2_div_2 = { 1488522236LL }; > | ^~~~~~~~~~~~~~~~~~ > drivers/gpu/drm/amd/amdgpu/../display/include/fixed31_32.h:75:32: warning: ‘dc_fixpt_ln2’ defined but not used [-Wunused-const-variable=] > 75 | static const struct fixed31_32 dc_fixpt_ln2 = { 2977044471LL }; > | ^~~~~~~~~~~~ > drivers/gpu/drm/amd/amdgpu/../display/include/fixed31_32.h:74:32: warning: ‘dc_fixpt_e’ defined but not used [-Wunused-const-variable=] > 74 | static const struct fixed31_32 dc_fixpt_e = { 11674931555LL }; > | ^~~~~~~~~~ > drivers/gpu/drm/amd/amdgpu/../display/include/fixed31_32.h:73:32: warning: ‘dc_fixpt_two_pi’ defined but not used [-Wunused-const-variable=] > 73 | static const struct fixed31_32 dc_fixpt_two_pi = { 26986075409LL }; > | ^~~~~~~~~~~~~~~ > drivers/gpu/drm/amd/amdgpu/../display/include/fixed31_32.h:72:32: warning: ‘dc_fixpt_pi’ defined but not used [-Wunused-const-variable=] > 72 | static const struct fixed31_32 dc_fixpt_pi = { 13493037705LL }; > | ^~~~~~~~~~~ > > Cc: Harry Wentland <harry.wentland@xxxxxxx> > Cc: Leo Li <sunpeng.li@xxxxxxx> > Cc: Alex Deucher <alexander.deucher@xxxxxxx> > Cc: "Christian König" <christian.koenig@xxxxxxx> > Cc: David Airlie <airlied@xxxxxxxx> > Cc: Daniel Vetter <daniel@xxxxxxxx> > Cc: amd-gfx@xxxxxxxxxxxxxxxxxxxxx > Cc: dri-devel@xxxxxxxxxxxxxxxxxxxxx > Signed-off-by: Lee Jones <lee.jones@xxxxxxxxxx> Applied. Thanks! Alex > --- > drivers/gpu/drm/amd/display/dc/basics/fixpt31_32.c | 5 +++++ > drivers/gpu/drm/amd/display/include/fixed31_32.h | 6 ------ > 2 files changed, 5 insertions(+), 6 deletions(-) > > diff --git a/drivers/gpu/drm/amd/display/dc/basics/fixpt31_32.c b/drivers/gpu/drm/amd/display/dc/basics/fixpt31_32.c > index 1e9a2d3520684..59f37563704ad 100644 > --- a/drivers/gpu/drm/amd/display/dc/basics/fixpt31_32.c > +++ b/drivers/gpu/drm/amd/display/dc/basics/fixpt31_32.c > @@ -26,6 +26,11 @@ > #include "dm_services.h" > #include "include/fixed31_32.h" > > +static const struct fixed31_32 dc_fixpt_pi = { 13493037705LL }; > +static const struct fixed31_32 dc_fixpt_two_pi = { 26986075409LL }; > +static const struct fixed31_32 dc_fixpt_ln2 = { 2977044471LL }; > +static const struct fixed31_32 dc_fixpt_ln2_div_2 = { 1488522236LL }; > + > static inline unsigned long long abs_i64( > long long arg) > { > diff --git a/drivers/gpu/drm/amd/display/include/fixed31_32.h b/drivers/gpu/drm/amd/display/include/fixed31_32.h > index 16df2a485dd0d..22053d7ea6ced 100644 > --- a/drivers/gpu/drm/amd/display/include/fixed31_32.h > +++ b/drivers/gpu/drm/amd/display/include/fixed31_32.h > @@ -69,12 +69,6 @@ static const struct fixed31_32 dc_fixpt_epsilon = { 1LL }; > static const struct fixed31_32 dc_fixpt_half = { 0x80000000LL }; > static const struct fixed31_32 dc_fixpt_one = { 0x100000000LL }; > > -static const struct fixed31_32 dc_fixpt_pi = { 13493037705LL }; > -static const struct fixed31_32 dc_fixpt_two_pi = { 26986075409LL }; > -static const struct fixed31_32 dc_fixpt_e = { 11674931555LL }; > -static const struct fixed31_32 dc_fixpt_ln2 = { 2977044471LL }; > -static const struct fixed31_32 dc_fixpt_ln2_div_2 = { 1488522236LL }; > - > /* > * @brief > * Initialization routines > -- > 2.25.1 > > _______________________________________________ > dri-devel mailing list > dri-devel@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/dri-devel _______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx