From: Tao Zhou <tao.zh...@amd.com>

So GFX RAS injection could use default function if it doesn't define its
own injection interface.

Signed-off-by: Tao Zhou <tao.zh...@amd.com>
Reviewed-by: Hawking Zhang <hawking.zh...@amd.com>
Reviewed-by: Stanley.Yang <stanley.y...@amd.com>
Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c
index 7ae08f168f99..b7d8250a9281 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c
@@ -1123,16 +1123,15 @@ int amdgpu_ras_error_inject(struct amdgpu_device *adev,
                                                          block_info.address);
        }
 
-       if (info->head.block == AMDGPU_RAS_BLOCK__GFX) {
-               if (block_obj->hw_ops->ras_error_inject)
+       if (block_obj->hw_ops->ras_error_inject) {
+               if (info->head.block == AMDGPU_RAS_BLOCK__GFX)
                        ret = block_obj->hw_ops->ras_error_inject(adev, info, 
info->instance_mask);
-       } else {
-               /* If defined special ras_error_inject(e.g: xgmi), implement 
special ras_error_inject */
-               if (block_obj->hw_ops->ras_error_inject)
+               else /* Special ras_error_inject is defined (e.g: xgmi) */
                        ret = block_obj->hw_ops->ras_error_inject(adev, 
&block_info,
                                                info->instance_mask);
-               else  /*If not defined .ras_error_inject, use default 
ras_error_inject*/
-                       ret = psp_ras_trigger_error(&adev->psp, &block_info, 
info->instance_mask);
+       } else {
+               /* default path */
+               ret = psp_ras_trigger_error(&adev->psp, &block_info, 
info->instance_mask);
        }
 
        if (ret)
-- 
2.40.1

Reply via email to