]> git.baikalelectronics.ru Git - kernel.git/commit
drm/amd/display: Fix wrongly passed static prefix
authorTakashi Iwai <tiwai@suse.de>
Thu, 28 Nov 2019 08:27:14 +0000 (09:27 +0100)
committerAlex Deucher <alexander.deucher@amd.com>
Thu, 6 Feb 2020 20:04:38 +0000 (15:04 -0500)
commit7c0bfaeabd122dce78afdb29e4b63cffc7cfa939
treeec5e2929189a8bad5f05872947cd24dcc19f876f
parenta8170398d765dbc4e7a40908c06e40536fa4c7d9
drm/amd/display: Fix wrongly passed static prefix

Currently, gcc spews a warning as:
  drivers/gpu/drm/amd/amdgpu/../display/dc/dcn10/dcn10_hubbub.c: In function ‘hubbub1_verify_allow_pstate_change_high’:
  ./include/drm/drm_print.h:316:2: warning: ‘debug_data’ may be used uninitialized in this function [-Wmaybe-uninitialized]

This is because the code checks against a static value although it's
basically a constant and guaranteed to be set.

This patch changes the type prefix from static to const for addressing
the compile warning above and also for letting the compiler optimize
better.

Fixes: e1969e4d2f09 ("drm/amd/display: create new files for hubbub functions")
Reviewed-by: Harry Wentland <harry.wentland@amd.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/dc/dcn10/dcn10_hubbub.c