drm/radeon: make use of radeon_gem_init() consistent
authorAlex Deucher <alexander.deucher@amd.com>
Thu, 10 May 2012 17:00:06 +0000 (13:00 -0400)
committerDave Airlie <airlied@redhat.com>
Thu, 17 May 2012 10:46:32 +0000 (11:46 +0100)
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>
drivers/gpu/drm/radeon/evergreen.c
drivers/gpu/drm/radeon/ni.c
drivers/gpu/drm/radeon/r600.c
drivers/gpu/drm/radeon/radeon_device.c
drivers/gpu/drm/radeon/rv770.c
drivers/gpu/drm/radeon/si.c

index 7e7ac3d6e76e072a67fe0c6900f459e2a748e67d..58991af90502dea8b98d9aac157d182a5feca21e 100644 (file)
@@ -3435,10 +3435,6 @@ int evergreen_init(struct radeon_device *rdev)
 {
        int r;
 
-       /* This don't do much */
-       r = radeon_gem_init(rdev);
-       if (r)
-               return r;
        /* Read BIOS */
        if (!radeon_get_bios(rdev)) {
                if (ASIC_IS_AVIVO(rdev))
index 107b2177e6c5a44bcbc390f989c98e1565d7aecc..b01c2dd627b0e9a345e2dc8964d415ba37deffff 100644 (file)
@@ -1629,10 +1629,6 @@ int cayman_init(struct radeon_device *rdev)
        struct radeon_ring *ring = &rdev->ring[RADEON_RING_TYPE_GFX_INDEX];
        int r;
 
-       /* This don't do much */
-       r = radeon_gem_init(rdev);
-       if (r)
-               return r;
        /* Read BIOS */
        if (!radeon_get_bios(rdev)) {
                if (ASIC_IS_AVIVO(rdev))
index d2a536d788e9223e6c700058aaef2894ae29dc78..ab5d6f2a06c207be993a17de3961d2338ab88850 100644 (file)
@@ -2560,10 +2560,6 @@ int r600_init(struct radeon_device *rdev)
        if (r600_debugfs_mc_info_init(rdev)) {
                DRM_ERROR("Failed to register debugfs file for mc !\n");
        }
-       /* This don't do much */
-       r = radeon_gem_init(rdev);
-       if (r)
-               return r;
        /* Read BIOS */
        if (!radeon_get_bios(rdev)) {
                if (ASIC_IS_AVIVO(rdev))
index 3d41525c1bcf4a3a03846606a2f3c74d899a0429..57637017eed628f7161e4d374f76fd236b42609d 100644 (file)
@@ -736,9 +736,11 @@ int radeon_device_init(struct radeon_device *rdev,
        mutex_init(&rdev->gem.mutex);
        mutex_init(&rdev->pm.mutex);
        mutex_init(&rdev->vram_mutex);
-       INIT_LIST_HEAD(&rdev->gem.objects);
        init_waitqueue_head(&rdev->irq.vblank_queue);
        init_waitqueue_head(&rdev->irq.idle_queue);
+       r = radeon_gem_init(rdev);
+       if (r)
+               return r;
        /* initialize vm here */
        rdev->vm_manager.use_bitmap = 1;
        rdev->vm_manager.max_pfn = 1 << 20;
index 40f82e22a621325fa26d76d4aa635b27cdd7b0bb..c2f473bc13b85bf189b44d823d0ef193b94fd3c2 100644 (file)
@@ -1175,10 +1175,6 @@ int rv770_init(struct radeon_device *rdev)
 {
        int r;
 
-       /* This don't do much */
-       r = radeon_gem_init(rdev);
-       if (r)
-               return r;
        /* Read BIOS */
        if (!radeon_get_bios(rdev)) {
                if (ASIC_IS_AVIVO(rdev))
index 554ec141770bba0ada06511b232454a142406b46..63ae41dac5ca5907439d11e1c59d350d7d8b9e30 100644 (file)
@@ -3987,10 +3987,6 @@ int si_init(struct radeon_device *rdev)
        struct radeon_ring *ring = &rdev->ring[RADEON_RING_TYPE_GFX_INDEX];
        int r;
 
-       /* This don't do much */
-       r = radeon_gem_init(rdev);
-       if (r)
-               return r;
        /* Read BIOS */
        if (!radeon_get_bios(rdev)) {
                if (ASIC_IS_AVIVO(rdev))