drm/amdgpu: no need to call pci_ignore_hotplug for _PR3
authorAlex Deucher <alexander.deucher@amd.com>
Wed, 9 Dec 2020 18:20:03 +0000 (13:20 -0500)
committerAlex Deucher <alexander.deucher@amd.com>
Tue, 15 Dec 2020 16:35:19 +0000 (11:35 -0500)
The platform knows it's doing d3cold.

Acked-by: Evan Quan <evan.quan@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c

index 90b8ce762aee25d96cff746fa85bdb5f685f552f..4f97d493eedac9e5567086e9feb4465f1207baaf 100644 (file)
@@ -1352,20 +1352,13 @@ static int amdgpu_pmops_runtime_suspend(struct device *dev)
                /* Only need to handle PCI state in the driver for ATPX
                 * PCI core handles it for _PR3.
                 */
-               if (amdgpu_is_atpx_hybrid()) {
-                       pci_ignore_hotplug(pdev);
-               } else {
+               if (!amdgpu_is_atpx_hybrid()) {
                        amdgpu_device_cache_pci_state(pdev);
                        pci_disable_device(pdev);
                        pci_ignore_hotplug(pdev);
                        pci_set_power_state(pdev, PCI_D3cold);
                }
                drm_dev->switch_power_state = DRM_SWITCH_POWER_DYNAMIC_OFF;
-       } else if (amdgpu_device_supports_boco(drm_dev)) {
-               /* Only need to handle PCI state in the driver for ATPX
-                * PCI core handles it for _PR3.
-                */
-               pci_ignore_hotplug(pdev);
        } else if (amdgpu_device_supports_baco(drm_dev)) {
                amdgpu_device_baco_enter(drm_dev);
        }