On Thu, Jul 22, 2021 at 10:57:14AM +0530, Pratik Vishwakarma wrote: > [Why] > User might set mem_sleep as deep and it will result > in amdgpu resume errors. > > [How] > Check with pm for default suspend state > > Signed-off-by: Pratik Vishwakarma <Pratik.Vishwakarma@xxxxxxx> > --- > drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c > index af1710971ff3..d92196429741 100644 > --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c > +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_drv.c > @@ -1468,7 +1468,8 @@ static int amdgpu_pmops_suspend(struct device *dev) > struct amdgpu_device *adev = drm_to_adev(drm_dev); > int r; > > - if (amdgpu_acpi_is_s0ix_supported(adev)) > + if (amdgpu_acpi_is_s0ix_supported(adev) > + && pm_suspend_default_s2idle()) Why pm_suspend_default_s2idle instead of pm_suspend_via_firmware? > adev->in_s0ix = true; > adev->in_s3 = true; > r = amdgpu_device_suspend(drm_dev, true); _______________________________________________ amd-gfx mailing list amd-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/amd-gfx
