[PATCH 09/19] drm/amdgpu: add additional boco checks to runtime suspend/resume

[Alex Deucher]

We will take slightly different paths for boco and baco.

Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 46 ++++++++++++++-----------
 1 file changed, 26 insertions(+), 20 deletions(-)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
index bd22f0ce4d47..199b240811da 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
@@ -1159,18 +1159,21 @@ static int amdgpu_pmops_runtime_suspend(struct device 
*dev)
                return -EBUSY;
        }
 
-       drm_dev->switch_power_state = DRM_SWITCH_POWER_CHANGING;
+       if (amdgpu_device_supports_boco(drm_dev))
+               drm_dev->switch_power_state = DRM_SWITCH_POWER_CHANGING;
        drm_kms_helper_poll_disable(drm_dev);
 
        ret = amdgpu_device_suspend(drm_dev, false, false);
-       pci_save_state(pdev);
-       pci_disable_device(pdev);
-       pci_ignore_hotplug(pdev);
-       if (amdgpu_is_atpx_hybrid())
-               pci_set_power_state(pdev, PCI_D3cold);
-       else if (!amdgpu_has_atpx_dgpu_power_cntl())
-               pci_set_power_state(pdev, PCI_D3hot);
-       drm_dev->switch_power_state = DRM_SWITCH_POWER_DYNAMIC_OFF;
+       if (amdgpu_device_supports_boco(drm_dev)) {
+               pci_save_state(pdev);
+               pci_disable_device(pdev);
+               pci_ignore_hotplug(pdev);
+               if (amdgpu_is_atpx_hybrid())
+                       pci_set_power_state(pdev, PCI_D3cold);
+               else if (!amdgpu_has_atpx_dgpu_power_cntl())
+                       pci_set_power_state(pdev, PCI_D3hot);
+               drm_dev->switch_power_state = DRM_SWITCH_POWER_DYNAMIC_OFF;
+       }
 
        return 0;
 }
@@ -1184,20 +1187,23 @@ static int amdgpu_pmops_runtime_resume(struct device 
*dev)
        if (!amdgpu_device_supports_boco(drm_dev))
                return -EINVAL;
 
-       drm_dev->switch_power_state = DRM_SWITCH_POWER_CHANGING;
-
-       if (amdgpu_is_atpx_hybrid() ||
-           !amdgpu_has_atpx_dgpu_power_cntl())
-               pci_set_power_state(pdev, PCI_D0);
-       pci_restore_state(pdev);
-       ret = pci_enable_device(pdev);
-       if (ret)
-               return ret;
-       pci_set_master(pdev);
+       if (amdgpu_device_supports_boco(drm_dev))
+               drm_dev->switch_power_state = DRM_SWITCH_POWER_CHANGING;
 
+       if (amdgpu_device_supports_boco(drm_dev)) {
+               if (amdgpu_is_atpx_hybrid() ||
+                   !amdgpu_has_atpx_dgpu_power_cntl())
+                       pci_set_power_state(pdev, PCI_D0);
+               pci_restore_state(pdev);
+               ret = pci_enable_device(pdev);
+               if (ret)
+                       return ret;
+               pci_set_master(pdev);
+       }
        ret = amdgpu_device_resume(drm_dev, false, false);
        drm_kms_helper_poll_enable(drm_dev);
-       drm_dev->switch_power_state = DRM_SWITCH_POWER_ON;
+       if (amdgpu_device_supports_boco(drm_dev))
+               drm_dev->switch_power_state = DRM_SWITCH_POWER_ON;
        return 0;
 }
 
-- 
2.20.1

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