diff options
author | Christian König <deathsimple@vodafone.de> | 2012-07-05 05:55:34 -0400 |
---|---|---|
committer | Christian König <deathsimple@vodafone.de> | 2012-07-17 04:32:41 -0400 |
commit | 2898c348e5caa50b31b9113726dfa45b2dfc3124 (patch) | |
tree | bc5ba4341cdc609899f28279834b3a65c9d98ecd /drivers/gpu/drm/radeon/r420.c | |
parent | b90ca986a41b233c2fcc8c809ee4f970d2de075e (diff) |
drm/radeon: remove ip_pool start/suspend
The IB pool is in gart memory, so it is completely
superfluous to unpin / repin it on suspend / resume.
Signed-off-by: Christian König <deathsimple@vodafone.de>
Reviewed-by: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/radeon/r420.c')
-rw-r--r-- | drivers/gpu/drm/radeon/r420.c | 17 |
1 files changed, 6 insertions, 11 deletions
diff --git a/drivers/gpu/drm/radeon/r420.c b/drivers/gpu/drm/radeon/r420.c index 99137be7a300..0062938a589c 100644 --- a/drivers/gpu/drm/radeon/r420.c +++ b/drivers/gpu/drm/radeon/r420.c | |||
@@ -275,9 +275,11 @@ static int r420_startup(struct radeon_device *rdev) | |||
275 | } | 275 | } |
276 | r420_cp_errata_init(rdev); | 276 | r420_cp_errata_init(rdev); |
277 | 277 | ||
278 | r = radeon_ib_pool_start(rdev); | 278 | r = radeon_ib_pool_init(rdev); |
279 | if (r) | 279 | if (r) { |
280 | dev_err(rdev->dev, "IB initialization failed (%d).\n", r); | ||
280 | return r; | 281 | return r; |
282 | } | ||
281 | 283 | ||
282 | r = radeon_ib_ring_tests(rdev); | 284 | r = radeon_ib_ring_tests(rdev); |
283 | if (r) | 285 | if (r) |
@@ -324,7 +326,6 @@ int r420_resume(struct radeon_device *rdev) | |||
324 | 326 | ||
325 | int r420_suspend(struct radeon_device *rdev) | 327 | int r420_suspend(struct radeon_device *rdev) |
326 | { | 328 | { |
327 | radeon_ib_pool_suspend(rdev); | ||
328 | r420_cp_errata_fini(rdev); | 329 | r420_cp_errata_fini(rdev); |
329 | r100_cp_disable(rdev); | 330 | r100_cp_disable(rdev); |
330 | radeon_wb_disable(rdev); | 331 | radeon_wb_disable(rdev); |
@@ -340,7 +341,7 @@ void r420_fini(struct radeon_device *rdev) | |||
340 | { | 341 | { |
341 | r100_cp_fini(rdev); | 342 | r100_cp_fini(rdev); |
342 | radeon_wb_fini(rdev); | 343 | radeon_wb_fini(rdev); |
343 | r100_ib_fini(rdev); | 344 | radeon_ib_pool_fini(rdev); |
344 | radeon_gem_fini(rdev); | 345 | radeon_gem_fini(rdev); |
345 | if (rdev->flags & RADEON_IS_PCIE) | 346 | if (rdev->flags & RADEON_IS_PCIE) |
346 | rv370_pcie_gart_fini(rdev); | 347 | rv370_pcie_gart_fini(rdev); |
@@ -438,20 +439,14 @@ int r420_init(struct radeon_device *rdev) | |||
438 | } | 439 | } |
439 | r420_set_reg_safe(rdev); | 440 | r420_set_reg_safe(rdev); |
440 | 441 | ||
441 | r = radeon_ib_pool_init(rdev); | ||
442 | rdev->accel_working = true; | 442 | rdev->accel_working = true; |
443 | if (r) { | ||
444 | dev_err(rdev->dev, "IB initialization failed (%d).\n", r); | ||
445 | rdev->accel_working = false; | ||
446 | } | ||
447 | |||
448 | r = r420_startup(rdev); | 443 | r = r420_startup(rdev); |
449 | if (r) { | 444 | if (r) { |
450 | /* Somethings want wront with the accel init stop accel */ | 445 | /* Somethings want wront with the accel init stop accel */ |
451 | dev_err(rdev->dev, "Disabling GPU acceleration\n"); | 446 | dev_err(rdev->dev, "Disabling GPU acceleration\n"); |
452 | r100_cp_fini(rdev); | 447 | r100_cp_fini(rdev); |
453 | radeon_wb_fini(rdev); | 448 | radeon_wb_fini(rdev); |
454 | r100_ib_fini(rdev); | 449 | radeon_ib_pool_fini(rdev); |
455 | radeon_irq_kms_fini(rdev); | 450 | radeon_irq_kms_fini(rdev); |
456 | if (rdev->flags & RADEON_IS_PCIE) | 451 | if (rdev->flags & RADEON_IS_PCIE) |
457 | rv370_pcie_gart_fini(rdev); | 452 | rv370_pcie_gart_fini(rdev); |