diff options
author | Alex Deucher <alexander.deucher@amd.com> | 2012-05-10 13:00:06 -0400 |
---|---|---|
committer | Dave Airlie <airlied@redhat.com> | 2012-05-17 06:46:32 -0400 |
commit | 1b9c3dd0220600391611637c2f04371dbc896e21 (patch) | |
tree | 72933b5525507c358e5ea9e2e61482394d4a6adf | |
parent | c1d2dbd2ad39b7870c741404220fd251dc69c678 (diff) |
drm/radeon: make use of radeon_gem_init() consistent
All radeon_gem_init() does is initialize the gem objects
list. radeon_device.c does this explicitly. r600+ calls
radeon_gem_init() so the list gets initialized twice. Older
asics don't call it at all and rely on the the init in
radeon_device.c. Just call radeon_gem_init() in radeon_device.c
and remove the explicit calls from all the newer asics.
All asics call radeon_gem_fini() in their fini pathes. That
could possibly be cleaned up too.
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Reviewed-by: Michel Dänzer <michel.daenzer@amd.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
-rw-r--r-- | drivers/gpu/drm/radeon/evergreen.c | 4 | ||||
-rw-r--r-- | drivers/gpu/drm/radeon/ni.c | 4 | ||||
-rw-r--r-- | drivers/gpu/drm/radeon/r600.c | 4 | ||||
-rw-r--r-- | drivers/gpu/drm/radeon/radeon_device.c | 4 | ||||
-rw-r--r-- | drivers/gpu/drm/radeon/rv770.c | 4 | ||||
-rw-r--r-- | drivers/gpu/drm/radeon/si.c | 4 |
6 files changed, 3 insertions, 21 deletions
diff --git a/drivers/gpu/drm/radeon/evergreen.c b/drivers/gpu/drm/radeon/evergreen.c index 7e7ac3d6e76e..58991af90502 100644 --- a/drivers/gpu/drm/radeon/evergreen.c +++ b/drivers/gpu/drm/radeon/evergreen.c | |||
@@ -3435,10 +3435,6 @@ int evergreen_init(struct radeon_device *rdev) | |||
3435 | { | 3435 | { |
3436 | int r; | 3436 | int r; |
3437 | 3437 | ||
3438 | /* This don't do much */ | ||
3439 | r = radeon_gem_init(rdev); | ||
3440 | if (r) | ||
3441 | return r; | ||
3442 | /* Read BIOS */ | 3438 | /* Read BIOS */ |
3443 | if (!radeon_get_bios(rdev)) { | 3439 | if (!radeon_get_bios(rdev)) { |
3444 | if (ASIC_IS_AVIVO(rdev)) | 3440 | if (ASIC_IS_AVIVO(rdev)) |
diff --git a/drivers/gpu/drm/radeon/ni.c b/drivers/gpu/drm/radeon/ni.c index 107b2177e6c5..b01c2dd627b0 100644 --- a/drivers/gpu/drm/radeon/ni.c +++ b/drivers/gpu/drm/radeon/ni.c | |||
@@ -1629,10 +1629,6 @@ int cayman_init(struct radeon_device *rdev) | |||
1629 | struct radeon_ring *ring = &rdev->ring[RADEON_RING_TYPE_GFX_INDEX]; | 1629 | struct radeon_ring *ring = &rdev->ring[RADEON_RING_TYPE_GFX_INDEX]; |
1630 | int r; | 1630 | int r; |
1631 | 1631 | ||
1632 | /* This don't do much */ | ||
1633 | r = radeon_gem_init(rdev); | ||
1634 | if (r) | ||
1635 | return r; | ||
1636 | /* Read BIOS */ | 1632 | /* Read BIOS */ |
1637 | if (!radeon_get_bios(rdev)) { | 1633 | if (!radeon_get_bios(rdev)) { |
1638 | if (ASIC_IS_AVIVO(rdev)) | 1634 | if (ASIC_IS_AVIVO(rdev)) |
diff --git a/drivers/gpu/drm/radeon/r600.c b/drivers/gpu/drm/radeon/r600.c index d2a536d788e9..ab5d6f2a06c2 100644 --- a/drivers/gpu/drm/radeon/r600.c +++ b/drivers/gpu/drm/radeon/r600.c | |||
@@ -2560,10 +2560,6 @@ int r600_init(struct radeon_device *rdev) | |||
2560 | if (r600_debugfs_mc_info_init(rdev)) { | 2560 | if (r600_debugfs_mc_info_init(rdev)) { |
2561 | DRM_ERROR("Failed to register debugfs file for mc !\n"); | 2561 | DRM_ERROR("Failed to register debugfs file for mc !\n"); |
2562 | } | 2562 | } |
2563 | /* This don't do much */ | ||
2564 | r = radeon_gem_init(rdev); | ||
2565 | if (r) | ||
2566 | return r; | ||
2567 | /* Read BIOS */ | 2563 | /* Read BIOS */ |
2568 | if (!radeon_get_bios(rdev)) { | 2564 | if (!radeon_get_bios(rdev)) { |
2569 | if (ASIC_IS_AVIVO(rdev)) | 2565 | if (ASIC_IS_AVIVO(rdev)) |
diff --git a/drivers/gpu/drm/radeon/radeon_device.c b/drivers/gpu/drm/radeon/radeon_device.c index 3d41525c1bcf..57637017eed6 100644 --- a/drivers/gpu/drm/radeon/radeon_device.c +++ b/drivers/gpu/drm/radeon/radeon_device.c | |||
@@ -736,9 +736,11 @@ int radeon_device_init(struct radeon_device *rdev, | |||
736 | mutex_init(&rdev->gem.mutex); | 736 | mutex_init(&rdev->gem.mutex); |
737 | mutex_init(&rdev->pm.mutex); | 737 | mutex_init(&rdev->pm.mutex); |
738 | mutex_init(&rdev->vram_mutex); | 738 | mutex_init(&rdev->vram_mutex); |
739 | INIT_LIST_HEAD(&rdev->gem.objects); | ||
740 | init_waitqueue_head(&rdev->irq.vblank_queue); | 739 | init_waitqueue_head(&rdev->irq.vblank_queue); |
741 | init_waitqueue_head(&rdev->irq.idle_queue); | 740 | init_waitqueue_head(&rdev->irq.idle_queue); |
741 | r = radeon_gem_init(rdev); | ||
742 | if (r) | ||
743 | return r; | ||
742 | /* initialize vm here */ | 744 | /* initialize vm here */ |
743 | rdev->vm_manager.use_bitmap = 1; | 745 | rdev->vm_manager.use_bitmap = 1; |
744 | rdev->vm_manager.max_pfn = 1 << 20; | 746 | rdev->vm_manager.max_pfn = 1 << 20; |
diff --git a/drivers/gpu/drm/radeon/rv770.c b/drivers/gpu/drm/radeon/rv770.c index 40f82e22a621..c2f473bc13b8 100644 --- a/drivers/gpu/drm/radeon/rv770.c +++ b/drivers/gpu/drm/radeon/rv770.c | |||
@@ -1175,10 +1175,6 @@ int rv770_init(struct radeon_device *rdev) | |||
1175 | { | 1175 | { |
1176 | int r; | 1176 | int r; |
1177 | 1177 | ||
1178 | /* This don't do much */ | ||
1179 | r = radeon_gem_init(rdev); | ||
1180 | if (r) | ||
1181 | return r; | ||
1182 | /* Read BIOS */ | 1178 | /* Read BIOS */ |
1183 | if (!radeon_get_bios(rdev)) { | 1179 | if (!radeon_get_bios(rdev)) { |
1184 | if (ASIC_IS_AVIVO(rdev)) | 1180 | if (ASIC_IS_AVIVO(rdev)) |
diff --git a/drivers/gpu/drm/radeon/si.c b/drivers/gpu/drm/radeon/si.c index 554ec141770b..63ae41dac5ca 100644 --- a/drivers/gpu/drm/radeon/si.c +++ b/drivers/gpu/drm/radeon/si.c | |||
@@ -3987,10 +3987,6 @@ int si_init(struct radeon_device *rdev) | |||
3987 | struct radeon_ring *ring = &rdev->ring[RADEON_RING_TYPE_GFX_INDEX]; | 3987 | struct radeon_ring *ring = &rdev->ring[RADEON_RING_TYPE_GFX_INDEX]; |
3988 | int r; | 3988 | int r; |
3989 | 3989 | ||
3990 | /* This don't do much */ | ||
3991 | r = radeon_gem_init(rdev); | ||
3992 | if (r) | ||
3993 | return r; | ||
3994 | /* Read BIOS */ | 3990 | /* Read BIOS */ |
3995 | if (!radeon_get_bios(rdev)) { | 3991 | if (!radeon_get_bios(rdev)) { |
3996 | if (ASIC_IS_AVIVO(rdev)) | 3992 | if (ASIC_IS_AVIVO(rdev)) |