[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...

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@xxxxxxxxx>

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@xxxxxxx>
:::::: CC: Alex Deucher <alexander.deucher@xxxxxxx>

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@xxxxxxxxxxxx

Attachment: .config.gz
Description: application/gzip

_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel

[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux