The gfx_v8_0_cp_test_all_rings() function should return success
only when all ring tests were successful.

Signed-off-by: Timur Kristóf <[email protected]>
---
 drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c 
b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
index 9fe430e3ae65..6d69d49539ae 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gfx_v8_0.c
@@ -4703,12 +4703,14 @@ static int gfx_v8_0_cp_test_all_rings(struct 
amdgpu_device *adev)
        if (r)
                return r;
 
+       r = 0;
+
        for (i = 0; i < adev->gfx.num_compute_rings; i++) {
                ring = &adev->gfx.compute_ring[i];
-               amdgpu_ring_test_helper(ring);
+               r |= amdgpu_ring_test_helper(ring);
        }
 
-       return 0;
+       return r;
 }
 
 static int gfx_v8_0_cp_resume(struct amdgpu_device *adev)
-- 
2.54.0

Reply via email to