On 2/21/2024 03:35, Ma Jun wrote:
Don't set power state flag when system enter runtime suspend,
or it may cause runtime resume failure issue.

Signed-off-by: Ma Jun <jun....@amd.com>

Reviewed-by: Mario Limonciello <mario.limoncie...@amd.com>

Please add a fixes tag for 3a9626c816db901def438dc2513622e281186d39 when committing.

---
  drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c | 3 +++
  1 file changed, 3 insertions(+)

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c 
b/drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c
index f754e50e7b43..4f1a4435cee4 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_acpi.c
@@ -1531,6 +1531,9 @@ bool amdgpu_acpi_is_s0ix_active(struct amdgpu_device 
*adev)
   */
  void amdgpu_choose_low_power_state(struct amdgpu_device *adev)
  {
+       if (adev->in_runpm)
+               return;
+
        if (amdgpu_acpi_is_s0ix_active(adev))
                adev->in_s0ix = true;
        else if (amdgpu_acpi_is_s3_active(adev))

Reply via email to