Use generic PCI reset for GPU reset if the user specifies
PCI reset as the reset mechanism.  This should in general
only be used for validation.

Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/si.c | 37 ++++++++++++++++++++-------------
 1 file changed, 23 insertions(+), 14 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/si.c b/drivers/gpu/drm/amd/amdgpu/si.c
index 668dd6dfe6a3..7cbc2bb03bc6 100644
--- a/drivers/gpu/drm/amd/amdgpu/si.c
+++ b/drivers/gpu/drm/amd/amdgpu/si.c
@@ -1409,17 +1409,6 @@ static int si_gpu_pci_config_reset(struct amdgpu_device 
*adev)
        return r;
 }
 
-static int si_asic_reset(struct amdgpu_device *adev)
-{
-       int r;
-
-       dev_info(adev->dev, "PCI CONFIG reset\n");
-
-       r = si_gpu_pci_config_reset(adev);
-
-       return r;
-}
-
 static bool si_asic_supports_baco(struct amdgpu_device *adev)
 {
        return false;
@@ -1428,14 +1417,34 @@ static bool si_asic_supports_baco(struct amdgpu_device 
*adev)
 static enum amd_reset_method
 si_asic_reset_method(struct amdgpu_device *adev)
 {
-       if (amdgpu_reset_method != AMD_RESET_METHOD_LEGACY &&
-           amdgpu_reset_method != -1)
+       if (amdgpu_reset_method == AMD_RESET_METHOD_PCI)
+               return amdgpu_reset_method;
+       else if (amdgpu_reset_method != AMD_RESET_METHOD_LEGACY &&
+                amdgpu_reset_method != -1)
                dev_warn(adev->dev, "Specified reset method:%d isn't supported, 
using AUTO instead.\n",
-                                 amdgpu_reset_method);
+                        amdgpu_reset_method);
 
        return AMD_RESET_METHOD_LEGACY;
 }
 
+static int si_asic_reset(struct amdgpu_device *adev)
+{
+       int r;
+
+       switch (si_asic_reset_method(adev)) {
+       case AMD_RESET_METHOD_PCI:
+               dev_info(adev->dev, "PCI reset\n");
+               r = amdgpu_device_pci_reset(adev);
+               break;
+       default:
+               dev_info(adev->dev, "PCI CONFIG reset\n");
+               r = si_gpu_pci_config_reset(adev);
+               break;
+       }
+
+       return r;
+}
+
 static u32 si_get_config_memsize(struct amdgpu_device *adev)
 {
        return RREG32(mmCONFIG_MEMSIZE);
-- 
2.29.2

_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx

Reply via email to