drm: Branch 'master'

[email protected] (GitLab Mirror) Wed, 15 Aug 2018 09:27:23 +0000 (UTC)
Newsgroups gmane.comp.video.dri.patches
Message-ID <[email protected]>
 amdgpu/amdgpu_bo.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 1e12c16d7697a1223630a507c1032d940794039a
Author: Michel Dänzer <[email protected]>
Date:   Tue Aug 14 11:41:29 2018 +0200

    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 <[email protected]>
    Reviewed-by: Junwei Zhang <[email protected]>

diff --git a/amdgpu/amdgpu_bo.c b/amdgpu/amdgpu_bo.c
index b790e9be..86d1c143 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)

------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
--
_______________________________________________
Dri-patches mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/dri-patches