From: kernel test robot <lkp@intel.com>
To: kbuild@lists.01.org
Subject: drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_link_dpcd.c:195 core_link_read_dpcd() error: uninitialized symbol 'status'.
Date: Mon, 20 Dec 2021 00:53:19 +0800 [thread overview]
Message-ID: <202112200044.QYrs6tAB-lkp@intel.com> (raw)
[-- Attachment #1: Type: text/plain, Size: 5871 bytes --]
CC: kbuild-all(a)lists.01.org
CC: linux-kernel(a)vger.kernel.org
TO: Wesley Chalmers <Wesley.Chalmers@amd.com>
CC: Alex Deucher <alexander.deucher@amd.com>
CC: Jun Lei <Jun.Lei@amd.com>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 3f667b5d4053ad54aee13dab5c94f04ff75ddfdf
commit: 9cf9498f668d4c78616ebd2fe2e5f3850b189c5b drm/amd/display: Partition DPCD address space and break up transactions
date: 6 months ago
:::::: branch date: 19 hours ago
:::::: commit date: 6 months ago
config: x86_64-randconfig-m001-20211207 (https://download.01.org/0day-ci/archive/20211220/202112200044.QYrs6tAB-lkp(a)intel.com/config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
smatch warnings:
drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_link_dpcd.c:195 core_link_read_dpcd() error: uninitialized symbol 'status'.
drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_link_dpcd.c:217 core_link_write_dpcd() error: uninitialized symbol 'status'.
vim +/status +195 drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_link_dpcd.c
30adeee52d1eba Wesley Chalmers 2021-04-01 164
30adeee52d1eba Wesley Chalmers 2021-04-01 165 enum dc_status core_link_read_dpcd(
30adeee52d1eba Wesley Chalmers 2021-04-01 166 struct dc_link *link,
30adeee52d1eba Wesley Chalmers 2021-04-01 167 uint32_t address,
30adeee52d1eba Wesley Chalmers 2021-04-01 168 uint8_t *data,
30adeee52d1eba Wesley Chalmers 2021-04-01 169 uint32_t size)
30adeee52d1eba Wesley Chalmers 2021-04-01 170 {
30adeee52d1eba Wesley Chalmers 2021-04-01 171 uint32_t extended_address;
9cf9498f668d4c Wesley Chalmers 2021-04-08 172 uint32_t partitioned_address;
30adeee52d1eba Wesley Chalmers 2021-04-01 173 uint8_t *extended_data;
30adeee52d1eba Wesley Chalmers 2021-04-01 174 uint32_t extended_size;
9cf9498f668d4c Wesley Chalmers 2021-04-08 175 /* size of the remaining partitioned address space */
9cf9498f668d4c Wesley Chalmers 2021-04-08 176 uint32_t size_left_to_read;
30adeee52d1eba Wesley Chalmers 2021-04-01 177 enum dc_status status;
9cf9498f668d4c Wesley Chalmers 2021-04-08 178 /* size of the next partition to be read from */
9cf9498f668d4c Wesley Chalmers 2021-04-08 179 uint32_t partition_size;
9cf9498f668d4c Wesley Chalmers 2021-04-08 180 uint32_t data_index = 0;
30adeee52d1eba Wesley Chalmers 2021-04-01 181
30adeee52d1eba Wesley Chalmers 2021-04-01 182 dpcd_extend_address_range(address, data, size, &extended_address, &extended_data, &extended_size);
9cf9498f668d4c Wesley Chalmers 2021-04-08 183 partitioned_address = extended_address;
9cf9498f668d4c Wesley Chalmers 2021-04-08 184 size_left_to_read = extended_size;
9cf9498f668d4c Wesley Chalmers 2021-04-08 185 while (size_left_to_read) {
9cf9498f668d4c Wesley Chalmers 2021-04-08 186 partition_size = dpcd_get_next_partition_size(partitioned_address, size_left_to_read);
9cf9498f668d4c Wesley Chalmers 2021-04-08 187 status = internal_link_read_dpcd(link, partitioned_address, &extended_data[data_index], partition_size);
9cf9498f668d4c Wesley Chalmers 2021-04-08 188 if (status != DC_OK)
9cf9498f668d4c Wesley Chalmers 2021-04-08 189 break;
9cf9498f668d4c Wesley Chalmers 2021-04-08 190 partitioned_address += partition_size;
9cf9498f668d4c Wesley Chalmers 2021-04-08 191 data_index += partition_size;
9cf9498f668d4c Wesley Chalmers 2021-04-08 192 size_left_to_read -= partition_size;
9cf9498f668d4c Wesley Chalmers 2021-04-08 193 }
30adeee52d1eba Wesley Chalmers 2021-04-01 194 dpcd_reduce_address_range(extended_address, extended_data, extended_size, address, data, size);
30adeee52d1eba Wesley Chalmers 2021-04-01 @195 return status;
30adeee52d1eba Wesley Chalmers 2021-04-01 196 }
30adeee52d1eba Wesley Chalmers 2021-04-01 197
30adeee52d1eba Wesley Chalmers 2021-04-01 198 enum dc_status core_link_write_dpcd(
30adeee52d1eba Wesley Chalmers 2021-04-01 199 struct dc_link *link,
30adeee52d1eba Wesley Chalmers 2021-04-01 200 uint32_t address,
30adeee52d1eba Wesley Chalmers 2021-04-01 201 const uint8_t *data,
30adeee52d1eba Wesley Chalmers 2021-04-01 202 uint32_t size)
30adeee52d1eba Wesley Chalmers 2021-04-01 203 {
9cf9498f668d4c Wesley Chalmers 2021-04-08 204 uint32_t partition_size;
9cf9498f668d4c Wesley Chalmers 2021-04-08 205 uint32_t data_index = 0;
9cf9498f668d4c Wesley Chalmers 2021-04-08 206 enum dc_status status;
9cf9498f668d4c Wesley Chalmers 2021-04-08 207
9cf9498f668d4c Wesley Chalmers 2021-04-08 208 while (size) {
9cf9498f668d4c Wesley Chalmers 2021-04-08 209 partition_size = dpcd_get_next_partition_size(address, size);
9cf9498f668d4c Wesley Chalmers 2021-04-08 210 status = internal_link_write_dpcd(link, address, &data[data_index], partition_size);
9cf9498f668d4c Wesley Chalmers 2021-04-08 211 if (status != DC_OK)
9cf9498f668d4c Wesley Chalmers 2021-04-08 212 break;
9cf9498f668d4c Wesley Chalmers 2021-04-08 213 address += partition_size;
9cf9498f668d4c Wesley Chalmers 2021-04-08 214 data_index += partition_size;
9cf9498f668d4c Wesley Chalmers 2021-04-08 215 size -= partition_size;
9cf9498f668d4c Wesley Chalmers 2021-04-08 216 }
9cf9498f668d4c Wesley Chalmers 2021-04-08 @217 return status;
:::::: The code at line 195 was first introduced by commit
:::::: 30adeee52d1ebadd8e4e594a54c7cf77250b91db drm/amd/display: Enforce DPCD Address ranges
:::::: TO: Wesley Chalmers <Wesley.Chalmers@amd.com>
:::::: CC: Alex Deucher <alexander.deucher@amd.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
next reply other threads:[~2021-12-19 16:53 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-12-19 16:53 kernel test robot [this message]
2022-03-02 2:51 drivers/gpu/drm/amd/amdgpu/../display/dc/core/dc_link_dpcd.c:195 core_link_read_dpcd() error: uninitialized symbol 'status' kernel test robot
2022-03-04 2:47 kernel test robot
2022-03-09 20:25 kernel test robot
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202112200044.QYrs6tAB-lkp@intel.com \
--to=lkp@intel.com \
--cc=kbuild@lists.01.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.