[radeon-alex:drm-next-5.2-wip 230/300] drivers/gpu/drm/amd/amdgpu/../display/dc/dce/dce_aux.c:514 dce_aux_transfer_with_retries() error: uninitialized symbol 'operation_result'.

[Dan Carpenter]

tree:   git://people.freedesktop.org/~agd5f/linux.git drm-next-5.2-wip
head:   940d104240cd069a88f089f91299ac9bf4a154d1
commit: 2bec3b50c9c5be65ac4800d57ace6fc3b2d82136 [230/300] drm/amd/display: Respect aux return values

smatch warnings:
drivers/gpu/drm/amd/amdgpu/../display/dc/dce/dce_aux.c:514 dce_aux_transfer_with_retries() error: uninitialized symbol 'operation_result'.

git remote add radeon-alex git://people.freedesktop.org/~agd5f/linux.git
git remote update radeon-alex
git checkout 2bec3b50c9c5be65ac4800d57ace6fc3b2d82136
vim +/operation_result +514 drivers/gpu/drm/amd/amdgpu/../display/dc/dce/dce_aux.c

eae5ffa9 David Francis 2018-11-29  493  
eae5ffa9 David Francis 2018-11-29  494  bool dce_aux_transfer_with_retries(struct ddc_service *ddc,
eae5ffa9 David Francis 2018-11-29  495  		struct aux_payload *payload)
eae5ffa9 David Francis 2018-11-29  496  {
eae5ffa9 David Francis 2018-11-29  497  	int i, ret = 0;
eae5ffa9 David Francis 2018-11-29  498  	uint8_t reply;
eae5ffa9 David Francis 2018-11-29  499  	bool payload_reply = true;
2bec3b50 Thomas Lim    2019-01-16  500  	enum aux_channel_operation_result operation_result;
2bec3b50 Thomas Lim    2019-01-16  501  	int aux_ack_retries = 0,
2bec3b50 Thomas Lim    2019-01-16  502  		aux_defer_retries = 0,
2bec3b50 Thomas Lim    2019-01-16  503  		aux_i2c_defer_retries = 0,
2bec3b50 Thomas Lim    2019-01-16  504  		aux_timeout_retries = 0,
2bec3b50 Thomas Lim    2019-01-16  505  		aux_invalid_reply_retries = 0;
eae5ffa9 David Francis 2018-11-29  506  
eae5ffa9 David Francis 2018-11-29  507  	if (!payload->reply) {
eae5ffa9 David Francis 2018-11-29  508  		payload_reply = false;
eae5ffa9 David Francis 2018-11-29  509  		payload->reply = &reply;
eae5ffa9 David Francis 2018-11-29  510  	}
eae5ffa9 David Francis 2018-11-29  511  
2bec3b50 Thomas Lim    2019-01-16  512  	for (i = 0; i < AUX_MAX_RETRIES; i++) {
2bec3b50 Thomas Lim    2019-01-16  513  		ret = dce_aux_transfer_raw(ddc, payload, &operation_result);
                                                        ^^^^^^^^^^^^^^^^^^^^^^^^^^
My guess is that this function can fail, but I haven't looked at the
code.

2bec3b50 Thomas Lim    2019-01-16 @514  		switch (operation_result) {

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