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/r300.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/r300.c')
-rw-r--r-- | drivers/gpu/drm/radeon/r300.c | 17 |
1 files changed, 6 insertions, 11 deletions
diff --git a/drivers/gpu/drm/radeon/r300.c b/drivers/gpu/drm/radeon/r300.c index 97722a33e513..b396e341a7b8 100644 --- a/drivers/gpu/drm/radeon/r300.c +++ b/drivers/gpu/drm/radeon/r300.c | |||
@@ -1391,9 +1391,11 @@ static int r300_startup(struct radeon_device *rdev) | |||
1391 | return r; | 1391 | return r; |
1392 | } | 1392 | } |
1393 | 1393 | ||
1394 | r = radeon_ib_pool_start(rdev); | 1394 | r = radeon_ib_pool_init(rdev); |
1395 | if (r) | 1395 | if (r) { |
1396 | dev_err(rdev->dev, "IB initialization failed (%d).\n", r); | ||
1396 | return r; | 1397 | return r; |
1398 | } | ||
1397 | 1399 | ||
1398 | r = radeon_ib_ring_tests(rdev); | 1400 | r = radeon_ib_ring_tests(rdev); |
1399 | if (r) | 1401 | if (r) |
@@ -1436,7 +1438,6 @@ int r300_resume(struct radeon_device *rdev) | |||
1436 | 1438 | ||
1437 | int r300_suspend(struct radeon_device *rdev) | 1439 | int r300_suspend(struct radeon_device *rdev) |
1438 | { | 1440 | { |
1439 | radeon_ib_pool_suspend(rdev); | ||
1440 | r100_cp_disable(rdev); | 1441 | r100_cp_disable(rdev); |
1441 | radeon_wb_disable(rdev); | 1442 | radeon_wb_disable(rdev); |
1442 | r100_irq_disable(rdev); | 1443 | r100_irq_disable(rdev); |
@@ -1451,7 +1452,7 @@ void r300_fini(struct radeon_device *rdev) | |||
1451 | { | 1452 | { |
1452 | r100_cp_fini(rdev); | 1453 | r100_cp_fini(rdev); |
1453 | radeon_wb_fini(rdev); | 1454 | radeon_wb_fini(rdev); |
1454 | r100_ib_fini(rdev); | 1455 | radeon_ib_pool_fini(rdev); |
1455 | radeon_gem_fini(rdev); | 1456 | radeon_gem_fini(rdev); |
1456 | if (rdev->flags & RADEON_IS_PCIE) | 1457 | if (rdev->flags & RADEON_IS_PCIE) |
1457 | rv370_pcie_gart_fini(rdev); | 1458 | rv370_pcie_gart_fini(rdev); |
@@ -1538,20 +1539,14 @@ int r300_init(struct radeon_device *rdev) | |||
1538 | } | 1539 | } |
1539 | r300_set_reg_safe(rdev); | 1540 | r300_set_reg_safe(rdev); |
1540 | 1541 | ||
1541 | r = radeon_ib_pool_init(rdev); | ||
1542 | rdev->accel_working = true; | 1542 | rdev->accel_working = true; |
1543 | if (r) { | ||
1544 | dev_err(rdev->dev, "IB initialization failed (%d).\n", r); | ||
1545 | rdev->accel_working = false; | ||
1546 | } | ||
1547 | |||
1548 | r = r300_startup(rdev); | 1543 | r = r300_startup(rdev); |
1549 | if (r) { | 1544 | if (r) { |
1550 | /* Somethings want wront with the accel init stop accel */ | 1545 | /* Somethings want wront with the accel init stop accel */ |
1551 | dev_err(rdev->dev, "Disabling GPU acceleration\n"); | 1546 | dev_err(rdev->dev, "Disabling GPU acceleration\n"); |
1552 | r100_cp_fini(rdev); | 1547 | r100_cp_fini(rdev); |
1553 | radeon_wb_fini(rdev); | 1548 | radeon_wb_fini(rdev); |
1554 | r100_ib_fini(rdev); | 1549 | radeon_ib_pool_fini(rdev); |
1555 | radeon_irq_kms_fini(rdev); | 1550 | radeon_irq_kms_fini(rdev); |
1556 | if (rdev->flags & RADEON_IS_PCIE) | 1551 | if (rdev->flags & RADEON_IS_PCIE) |
1557 | rv370_pcie_gart_fini(rdev); | 1552 | rv370_pcie_gart_fini(rdev); |