[radeon-alex:amd-staging-drm-next 155/158] drivers/gpu/drm/amd/amdgpu/../display/dc/dce/dce_link_encoder.c:1066:2: warning: this 'if' clause does not guard...

[kbuild test robot <fengguang.wu@intel.com>]

[-- Attachment #1: Type: text/plain, Size: 3516 bytes --]

tree:   git://people.freedesktop.org/~agd5f/linux.git amd-staging-drm-next
head:   5d8e75c019bf76889bd606a67afabf7d0b32d3fa
commit: 008cc768ebd34f1c205cd2066f6ab450d90a9022 [155/158] drm/amd/display: set HBR3 and TPS4 capable flags
config: alpha-allyesconfig (attached as .config)
compiler: alpha-linux-gnu-gcc (Debian 6.1.1-9) 6.1.1 20160705
reproduce:
        wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        git checkout 008cc768ebd34f1c205cd2066f6ab450d90a9022
        # save the attached .config to linux build tree
        make.cross ARCH=alpha 

All warnings (new ones prefixed by >>):

   drivers/gpu/drm/amd/amdgpu/../display/dc/dce/dce_link_encoder.c: In function 'dce110_link_encoder_construct':
>> drivers/gpu/drm/amd/amdgpu/../display/dc/dce/dce_link_encoder.c:1066:2: warning: this 'if' clause does not guard... [-Wmisleading-indentation]
     if (BP_RESULT_OK == bp_funcs->get_encoder_cap_info(
     ^~
   drivers/gpu/drm/amd/amdgpu/../display/dc/dce/dce_link_encoder.c:1071:3: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the 'if'
      enc110->base.features.flags.bits.IS_HBR3_CAPABLE =
      ^~~~~~

vim +/if +1066 drivers/gpu/drm/amd/amdgpu/../display/dc/dce/dce_link_encoder.c

5a0a7096 Harry Wentland 2017-01-03  1050  	break;
5a0a7096 Harry Wentland 2017-01-03  1051  	default:
5a0a7096 Harry Wentland 2017-01-03  1052  		ASSERT_CRITICAL(false);
5a0a7096 Harry Wentland 2017-01-03  1053  		enc110->base.preferred_engine = ENGINE_ID_UNKNOWN;
5a0a7096 Harry Wentland 2017-01-03  1054  	}
5a0a7096 Harry Wentland 2017-01-03  1055  
5a0a7096 Harry Wentland 2017-01-03  1056  	dm_logger_write(init_data->ctx->logger, LOG_I2C_AUX,
5a0a7096 Harry Wentland 2017-01-03  1057  			"Using channel: %s [%d]\n",
5a0a7096 Harry Wentland 2017-01-03  1058  			DECODE_CHANNEL_ID(init_data->channel),
5a0a7096 Harry Wentland 2017-01-03  1059  			init_data->channel);
5a0a7096 Harry Wentland 2017-01-03  1060  
5a0a7096 Harry Wentland 2017-01-03  1061  	/* Override features with DCE-specific values */
5a0a7096 Harry Wentland 2017-01-03  1062  	{
5a0a7096 Harry Wentland 2017-01-03  1063  	struct bp_encoder_cap_info bp_cap_info = {0};
5a0a7096 Harry Wentland 2017-01-03  1064  	const struct dc_vbios_funcs *bp_funcs = enc110->base.ctx->dc_bios->funcs;
5a0a7096 Harry Wentland 2017-01-03  1065  
5a0a7096 Harry Wentland 2017-01-03 @1066  	if (BP_RESULT_OK == bp_funcs->get_encoder_cap_info(
5a0a7096 Harry Wentland 2017-01-03  1067  			enc110->base.ctx->dc_bios, enc110->base.id,
5a0a7096 Harry Wentland 2017-01-03  1068  			&bp_cap_info))
5a0a7096 Harry Wentland 2017-01-03  1069  		enc110->base.features.flags.bits.IS_HBR2_CAPABLE =
5a0a7096 Harry Wentland 2017-01-03  1070  				bp_cap_info.DP_HBR2_CAP;
008cc768 Hersen Wu      2016-12-23  1071  		enc110->base.features.flags.bits.IS_HBR3_CAPABLE =
008cc768 Hersen Wu      2016-12-23  1072  				bp_cap_info.DP_HBR3_EN;
008cc768 Hersen Wu      2016-12-23  1073  
5a0a7096 Harry Wentland 2017-01-03  1074  	}

:::::: The code at line 1066 was first introduced by commit
:::::: 5a0a709652ceaca0a36ef2d0954b8d9d3f690560 drm/amd/dc: Add dc display driver

:::::: TO: Harry Wentland <harry.wentland@amd.com>
:::::: CC: Alex Deucher <alexander.deucher@amd.com>

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 47963 bytes --]

[-- Attachment #3: Type: text/plain, Size: 160 bytes --]

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