We don't need to pad anything if the number of dwords
written to the ring already matches the requirements.

Fixes some "writting more dword to ring than expected"
warnings.

Signed-off-by: Christian König <deathsim...@vodafone.de>
Reviewed-by: Jerome Glisse <jgli...@redhat.com>
Reviewed-by: Michel Dänzer <michel.daen...@amd.com>
---
 drivers/gpu/drm/radeon/radeon_ring.c |    7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/drivers/gpu/drm/radeon/radeon_ring.c 
b/drivers/gpu/drm/radeon/radeon_ring.c
index 0826e77..674aaba 100644
--- a/drivers/gpu/drm/radeon/radeon_ring.c
+++ b/drivers/gpu/drm/radeon/radeon_ring.c
@@ -272,13 +272,8 @@ int radeon_ring_lock(struct radeon_device *rdev, struct 
radeon_ring *ring, unsig
 
 void radeon_ring_commit(struct radeon_device *rdev, struct radeon_ring *ring)
 {
-       unsigned count_dw_pad;
-       unsigned i;
-
        /* We pad to match fetch size */
-       count_dw_pad = (ring->align_mask + 1) -
-                      (ring->wptr & ring->align_mask);
-       for (i = 0; i < count_dw_pad; i++) {
+       while (ring->wptr & ring->align_mask) {
                radeon_ring_write(ring, ring->nop);
        }
        DRM_MEMORYBARRIER();
-- 
1.7.9.5

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to