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
_______________________________________________
dri-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to