drm/amdgpu: don't enable baco on boco platforms in runpm

alexdeucher · alexdeucher · commit 11e612a093ab · 2021-08-05T21:18:00.000-04:00
If the platform uses BOCO, don't use BACO in runtime suspend. We could end up executing the BACO path if the platform supports both. Bug: https://gitlab.freedesktop.org/drm/amd/-/issues/1669 Reviewed-by: Evan Quan <evan.quan@amd.com> Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c
@@ -1573,6 +1573,8 @@ static int amdgpu_pmops_runtime_suspend(struct device *dev)
 		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)) {
+		/* nothing to do */
 	} else if (amdgpu_device_supports_baco(drm_dev)) {
 		amdgpu_device_baco_enter(drm_dev);
 	}

Original file line number	Diff line number	Diff line change
`@@ -1573,6 +1573,8 @@ static int amdgpu_pmops_runtime_suspend(struct device *dev)`
`1573`	`1573`	`pci_ignore_hotplug(pdev);`
`1574`	`1574`	`pci_set_power_state(pdev, PCI_D3cold);`
`1575`	`1575`	`drm_dev->switch_power_state = DRM_SWITCH_POWER_DYNAMIC_OFF;`
	`1576`	`+ } else if (amdgpu_device_supports_boco(drm_dev)) {`
	`1577`	`+ /* nothing to do */`
`1576`	`1578`	`} else if (amdgpu_device_supports_baco(drm_dev)) {`
`1577`	`1579`	`amdgpu_device_baco_enter(drm_dev);`
`1578`	`1580`	`}`