diff options
author | Michel Dänzer <michel.daenzer@amd.com> | 2018-08-14 11:41:29 +0200 |
---|---|---|
committer | Michel Dänzer <michel@daenzer.net> | 2018-08-15 10:34:59 +0200 |
commit | 1e12c16d7697a1223630a507c1032d940794039a (patch) | |
tree | 4df559c4e7885433bf108869c343b6ba90d3f29b | |
parent | 6e30031788f8fdb944d3cd8897acd45ca02433ba (diff) | |
download | external_libdrm-1e12c16d7697a1223630a507c1032d940794039a.tar.gz external_libdrm-1e12c16d7697a1223630a507c1032d940794039a.tar.bz2 external_libdrm-1e12c16d7697a1223630a507c1032d940794039a.zip |
amdgpu: Use uint32_t i in amdgpu_find_bo_by_cpu_mapping
The compiler points out that an int doesn't work as intended if
dev->bo_handles.max_key > INT_MAX:
../../amdgpu/amdgpu_bo.c: In function ‘amdgpu_find_bo_by_cpu_mapping’:
../../amdgpu/amdgpu_bo.c:550:16: warning: comparison of integer expressions of different signedness: ‘int’ and ‘uint32_t’ {aka ‘unsigned int’} [-Wsign-compare]
for (i = 0; i < dev->bo_handles.max_key; i++) {
^
../../amdgpu/amdgpu_bo.c:558:8: warning: comparison of integer expressions of different signedness: ‘int’ and ‘uint32_t’ {aka ‘unsigned int’} [-Wsign-compare]
if (i < dev->bo_handles.max_key) {
^
Fixes: 4d454424e1f2 ("amdgpu: add a function to find bo by cpu mapping
(v2)")
Reviewed-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Junwei Zhang <Jerry.Zhang@amd.com>
-rw-r--r-- | amdgpu/amdgpu_bo.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/amdgpu/amdgpu_bo.c b/amdgpu/amdgpu_bo.c index b790e9be2..86d1c1438 100644 --- a/amdgpu/amdgpu_bo.c +++ b/amdgpu/amdgpu_bo.c @@ -535,7 +535,7 @@ int amdgpu_find_bo_by_cpu_mapping(amdgpu_device_handle dev, amdgpu_bo_handle *buf_handle, uint64_t *offset_in_bo) { - int i; + uint32_t i; struct amdgpu_bo *bo; if (cpu == NULL || size == 0) |