Use memset32 instead of open coding it, just because it is
that bit nicer.

Signed-off-by: Tvrtko Ursulin <tvrtko.ursu...@igalia.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c | 12 ++++++++++--
 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c
index 6379bb25bf5c..0918d832da94 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ring.c
@@ -159,8 +159,16 @@ void amdgpu_ring_insert_nop(struct amdgpu_ring *ring, 
uint32_t count)
  */
 void amdgpu_ring_generic_pad_ib(struct amdgpu_ring *ring, struct amdgpu_ib *ib)
 {
-       while (ib->length_dw & ring->funcs->align_mask)
-               ib->ptr[ib->length_dw++] = ring->funcs->nop;
+       u32 align_mask = ring->funcs->align_mask;
+       u32 count = ib->length_dw & align_mask;
+
+       if (count) {
+               count = align_mask + 1 - count;
+
+               memset32(&ib->ptr[ib->length_dw], ring->funcs->nop, count);
+
+               ib->length_dw += count;
+       }
 }
 
 /**
-- 
2.48.0

Reply via email to