[radeon-alex:amd-staging-drm-next 263/272] drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_link_ddc.c:285:2: warning: this 'if' clause does not guard...
kernel test robot
lkp at intel.com
Tue Oct 27 01:33:29 UTC 2020
tree: git://people.freedesktop.org/~agd5f/linux.git amd-staging-drm-next
head: 3458ef3969082647e64825452b8bfd06ee7c3351
commit: 254a8d162af6360334e20ee6601c0b38f31dfa2d [263/272] drm/amd/display: Using udelay for specific dongle while edid return defer
config: alpha-allyesconfig (attached as .config)
compiler: alpha-linux-gcc (GCC) 9.3.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
git remote add radeon-alex git://people.freedesktop.org/~agd5f/linux.git
git fetch --no-tags radeon-alex amd-staging-drm-next
git checkout 254a8d162af6360334e20ee6601c0b38f31dfa2d
# save the attached .config to linux build tree
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-9.3.0 make.cross ARCH=alpha
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp at intel.com>
All warnings (new ones prefixed by >>):
drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_link_ddc.c: In function 'defer_delay_converter_wa':
>> drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_link_ddc.c:285:2: warning: this 'if' clause does not guard... [-Wmisleading-indentation]
285 | if (link->dpcd_caps.branch_dev_id == DP_BRANCH_DEVICE_ID_0080E1 &&
| ^~
drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_link_ddc.c:291:3: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'if'
291 | if (link->dpcd_caps.branch_dev_id == DP_BRANCH_DEVICE_ID_006037 &&
| ^~
In file included from drivers/gpu/drm/amd/amdgpu/../display/dc/inc/core_types.h:86,
from drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_link_ddc.c:36:
At top level:
drivers/gpu/drm/amd/amdgpu/../display/dc/inc/hw/dpp.h:50:42: warning: 'dpp_input_csc_matrix' defined but not used [-Wunused-const-variable=]
50 | static const struct dpp_input_csc_matrix dpp_input_csc_matrix[] = {
| ^~~~~~~~~~~~~~~~~~~~
In file included from drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_link_ddc.c:31:
drivers/gpu/drm/amd/amdgpu/../display/include/ddc_service_types.h:123:22: warning: 'DP_VGA_LVDS_CONVERTER_ID_3' defined but not used [-Wunused-const-variable=]
123 | static const uint8_t DP_VGA_LVDS_CONVERTER_ID_3[] = "dnomlA";
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
drivers/gpu/drm/amd/amdgpu/../display/include/ddc_service_types.h:121:22: warning: 'DP_VGA_LVDS_CONVERTER_ID_2' defined but not used [-Wunused-const-variable=]
121 | static const uint8_t DP_VGA_LVDS_CONVERTER_ID_2[] = "sivarT";
| ^~~~~~~~~~~~~~~~~~~~~~~~~~
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/dc/core/dc_link_ddc.c:28:
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 };
| ^~~~~~~~~~~
vim +/if +285 drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_link_ddc.c
4562236b3bc0a28 Harry Wentland 2017-09-12 278
4562236b3bc0a28 Harry Wentland 2017-09-12 279 static uint32_t defer_delay_converter_wa(
4562236b3bc0a28 Harry Wentland 2017-09-12 280 struct ddc_service *ddc,
4562236b3bc0a28 Harry Wentland 2017-09-12 281 uint32_t defer_delay)
4562236b3bc0a28 Harry Wentland 2017-09-12 282 {
d0778ebfd58f565 Harry Wentland 2017-07-22 283 struct dc_link *link = ddc->link;
4562236b3bc0a28 Harry Wentland 2017-09-12 284
df3b7e32ed459a5 Qingqing Zhuo 2019-07-31 @285 if (link->dpcd_caps.branch_dev_id == DP_BRANCH_DEVICE_ID_0080E1 &&
4562236b3bc0a28 Harry Wentland 2017-09-12 286 !memcmp(link->dpcd_caps.branch_dev_name,
4562236b3bc0a28 Harry Wentland 2017-09-12 287 DP_DVI_CONVERTER_ID_4,
4562236b3bc0a28 Harry Wentland 2017-09-12 288 sizeof(link->dpcd_caps.branch_dev_name)))
4562236b3bc0a28 Harry Wentland 2017-09-12 289 return defer_delay > I2C_OVER_AUX_DEFER_WA_DELAY ?
4562236b3bc0a28 Harry Wentland 2017-09-12 290 defer_delay : I2C_OVER_AUX_DEFER_WA_DELAY;
254a8d162af6360 jinlong zhang 2020-09-25 291 if (link->dpcd_caps.branch_dev_id == DP_BRANCH_DEVICE_ID_006037 &&
254a8d162af6360 jinlong zhang 2020-09-25 292 !memcmp(link->dpcd_caps.branch_dev_name,
254a8d162af6360 jinlong zhang 2020-09-25 293 DP_DVI_CONVERTER_ID_5,
254a8d162af6360 jinlong zhang 2020-09-25 294 sizeof(link->dpcd_caps.branch_dev_name)))
254a8d162af6360 jinlong zhang 2020-09-25 295 return defer_delay > I2C_OVER_AUX_DEFER_WA_DELAY_1MS ?
254a8d162af6360 jinlong zhang 2020-09-25 296 I2C_OVER_AUX_DEFER_WA_DELAY_1MS : defer_delay;
4562236b3bc0a28 Harry Wentland 2017-09-12 297
4562236b3bc0a28 Harry Wentland 2017-09-12 298 return defer_delay;
4562236b3bc0a28 Harry Wentland 2017-09-12 299 }
4562236b3bc0a28 Harry Wentland 2017-09-12 300
:::::: The code at line 285 was first introduced by commit
:::::: df3b7e32ed459a5348f7b408a9b8142b7358fde8 drm/amd/display: refactor Device ID for external chips
:::::: TO: Qingqing Zhuo <qingqing.zhuo at amd.com>
:::::: CC: Alex Deucher <alexander.deucher at amd.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: .config.gz
Type: application/gzip
Size: 65746 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20201027/5fbd7847/attachment-0001.gz>
More information about the dri-devel
mailing list