Commit d40fd3a3 authored by Christian König's avatar Christian König

drm/radeon: remove FIXME comment from chipset suspend

For a normal suspend/resume we allready wait for
the rings to be empty, and for a suspend/reasume
in case of a lockup we REALLY don't want to wait
for anything.
Signed-off-by: default avatarChristian König <deathsimple@vodafone.de>
Reviewed-by: default avatarJerome Glisse <jglisse@redhat.com>
Reviewed-by: default avatarAlex Deucher <alexander.deucher@amd.com>
parent 31be6183
...@@ -3137,7 +3137,6 @@ int evergreen_suspend(struct radeon_device *rdev) ...@@ -3137,7 +3137,6 @@ int evergreen_suspend(struct radeon_device *rdev)
struct radeon_ring *ring = &rdev->ring[RADEON_RING_TYPE_GFX_INDEX]; struct radeon_ring *ring = &rdev->ring[RADEON_RING_TYPE_GFX_INDEX];
r600_audio_fini(rdev); r600_audio_fini(rdev);
/* FIXME: we should wait for ring to be empty */
radeon_ib_pool_suspend(rdev); radeon_ib_pool_suspend(rdev);
r600_blit_suspend(rdev); r600_blit_suspend(rdev);
r700_cp_stop(rdev); r700_cp_stop(rdev);
......
...@@ -1334,7 +1334,6 @@ int cayman_resume(struct radeon_device *rdev) ...@@ -1334,7 +1334,6 @@ int cayman_resume(struct radeon_device *rdev)
int cayman_suspend(struct radeon_device *rdev) int cayman_suspend(struct radeon_device *rdev)
{ {
r600_audio_fini(rdev); r600_audio_fini(rdev);
/* FIXME: we should wait for ring to be empty */
radeon_ib_pool_suspend(rdev); radeon_ib_pool_suspend(rdev);
radeon_vm_manager_suspend(rdev); radeon_vm_manager_suspend(rdev);
r600_blit_suspend(rdev); r600_blit_suspend(rdev);
......
...@@ -2461,7 +2461,6 @@ int r600_suspend(struct radeon_device *rdev) ...@@ -2461,7 +2461,6 @@ int r600_suspend(struct radeon_device *rdev)
r600_audio_fini(rdev); r600_audio_fini(rdev);
radeon_ib_pool_suspend(rdev); radeon_ib_pool_suspend(rdev);
r600_blit_suspend(rdev); r600_blit_suspend(rdev);
/* FIXME: we should wait for ring to be empty */
r600_cp_stop(rdev); r600_cp_stop(rdev);
rdev->ring[RADEON_RING_TYPE_GFX_INDEX].ready = false; rdev->ring[RADEON_RING_TYPE_GFX_INDEX].ready = false;
r600_irq_suspend(rdev); r600_irq_suspend(rdev);
......
...@@ -996,7 +996,6 @@ int rv770_suspend(struct radeon_device *rdev) ...@@ -996,7 +996,6 @@ int rv770_suspend(struct radeon_device *rdev)
r600_audio_fini(rdev); r600_audio_fini(rdev);
radeon_ib_pool_suspend(rdev); radeon_ib_pool_suspend(rdev);
r600_blit_suspend(rdev); r600_blit_suspend(rdev);
/* FIXME: we should wait for ring to be empty */
r700_cp_stop(rdev); r700_cp_stop(rdev);
rdev->ring[RADEON_RING_TYPE_GFX_INDEX].ready = false; rdev->ring[RADEON_RING_TYPE_GFX_INDEX].ready = false;
r600_irq_suspend(rdev); r600_irq_suspend(rdev);
......
...@@ -3807,7 +3807,6 @@ int si_resume(struct radeon_device *rdev) ...@@ -3807,7 +3807,6 @@ int si_resume(struct radeon_device *rdev)
int si_suspend(struct radeon_device *rdev) int si_suspend(struct radeon_device *rdev)
{ {
/* FIXME: we should wait for ring to be empty */
radeon_ib_pool_suspend(rdev); radeon_ib_pool_suspend(rdev);
radeon_vm_manager_suspend(rdev); radeon_vm_manager_suspend(rdev);
#if 0 #if 0
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment