drm/amdgpu: Use the ALIGN() macro
authorLuben Tuikov <luben.tuikov@amd.com>
Wed, 2 Oct 2019 04:02:18 +0000 (00:02 -0400)
committerAlex Deucher <alexander.deucher@amd.com>
Fri, 11 Oct 2019 00:35:20 +0000 (19:35 -0500)
Use the ALIGN() macro to set "num_dw" to a
multiple of 8, i.e. lower 3 bits cleared.

Signed-off-by: Luben Tuikov <luben.tuikov@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_ttm.c

index 74a9bd94f10c08456e17104e021dabf43b3fc204..9da6350a4ba2f04a650c36e38803171cc96e4a3b 100644 (file)
@@ -1921,10 +1921,7 @@ static int amdgpu_map_buffer(struct ttm_buffer_object *bo,
        *addr += (u64)window * AMDGPU_GTT_MAX_TRANSFER_SIZE *
                AMDGPU_GPU_PAGE_SIZE;
 
-       num_dw = adev->mman.buffer_funcs->copy_num_dw;
-       while (num_dw & 0x7)
-               num_dw++;
-
+       num_dw = ALIGN(adev->mman.buffer_funcs->copy_num_dw, 8);
        num_bytes = num_pages * 8;
 
        r = amdgpu_job_alloc_with_ib(adev, num_dw * 4 + num_bytes, &job);
@@ -1984,11 +1981,7 @@ int amdgpu_copy_buffer(struct amdgpu_ring *ring, uint64_t src_offset,
 
        max_bytes = adev->mman.buffer_funcs->copy_max_bytes;
        num_loops = DIV_ROUND_UP(byte_count, max_bytes);
-       num_dw = num_loops * adev->mman.buffer_funcs->copy_num_dw;
-
-       /* for IB padding */
-       while (num_dw & 0x7)
-               num_dw++;
+       num_dw = ALIGN(num_loops * adev->mman.buffer_funcs->copy_num_dw, 8);
 
        r = amdgpu_job_alloc_with_ib(adev, num_dw * 4, &job);
        if (r)