To simplify the code of amdgpu_ras_reset_error_status without logical
change.

Signed-off-by: Tao Zhou <tao.zh...@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c | 29 +++++++------------------
 1 file changed, 8 insertions(+), 21 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c
index 6dddb0423411..3698be299992 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_ras.c
@@ -1107,17 +1107,20 @@ int amdgpu_ras_reset_error_count(struct amdgpu_device 
*adev,
        struct amdgpu_ras_block_object *block_obj = 
amdgpu_ras_get_ras_block(adev, block, 0);
        const struct amdgpu_mca_smu_funcs *mca_funcs = adev->mca.mca_funcs;
 
-       if (!block_obj || !block_obj->hw_ops)
-               return 0;
+       if (!block_obj || !block_obj->hw_ops) {
+               dev_dbg_once(adev->dev, "%s doesn't config RAS function\n",
+                            ras_block_str(block));
+               return -EOPNOTSUPP;
+       }
 
        /* skip ras error reset in gpu reset */
        if (amdgpu_in_reset(adev) &&
            mca_funcs && mca_funcs->mca_set_debug_mode)
-               return 0;
+               return -EOPNOTSUPP;
 
        if (!amdgpu_ras_is_supported(adev, block) ||
            !amdgpu_ras_get_mca_debug_mode(adev))
-               return 0;
+               return -EOPNOTSUPP;
 
        if (block_obj->hw_ops->reset_ras_error_count)
                block_obj->hw_ops->reset_ras_error_count(adev);
@@ -1129,25 +1132,9 @@ int amdgpu_ras_reset_error_status(struct amdgpu_device 
*adev,
                enum amdgpu_ras_block block)
 {
        struct amdgpu_ras_block_object *block_obj = 
amdgpu_ras_get_ras_block(adev, block, 0);
-       const struct amdgpu_mca_smu_funcs *mca_funcs = adev->mca.mca_funcs;
 
-       if (!block_obj || !block_obj->hw_ops) {
-               dev_dbg_once(adev->dev, "%s doesn't config RAS function\n",
-                            ras_block_str(block));
+       if (amdgpu_ras_reset_error_count(adev, block) == -EOPNOTSUPP)
                return 0;
-       }
-
-       /* skip ras error reset in gpu reset */
-       if (amdgpu_in_reset(adev) &&
-           mca_funcs && mca_funcs->mca_set_debug_mode)
-               return 0;
-
-       if (!amdgpu_ras_is_supported(adev, block) ||
-           !amdgpu_ras_get_mca_debug_mode(adev))
-               return 0;
-
-       if (block_obj->hw_ops->reset_ras_error_count)
-               block_obj->hw_ops->reset_ras_error_count(adev);
 
        if ((block == AMDGPU_RAS_BLOCK__GFX) ||
            (block == AMDGPU_RAS_BLOCK__MMHUB)) {
-- 
2.35.1

Reply via email to