diff options
author | David Herrmann <dh.herrmann@gmail.com> | 2013-07-01 20:32:58 +0200 |
---|---|---|
committer | Dave Airlie <airlied@redhat.com> | 2013-07-02 13:34:41 +1000 |
commit | 77ef8bbc87be7ad10b410247efc6d0f10676b401 (patch) | |
tree | 9cf69a4bbf55b95eeb25a1c4b8a700db138486c7 /drivers/gpu/drm/ttm/ttm_bo_manager.c | |
parent | 6ef92fbea2b5680204da5b8796e8972109d01bd3 (diff) | |
download | blackbird-obmc-linux-77ef8bbc87be7ad10b410247efc6d0f10676b401.tar.gz blackbird-obmc-linux-77ef8bbc87be7ad10b410247efc6d0f10676b401.zip |
drm: make drm_mm_init() return void
There is no reason to return "int" as this function never fails.
Furthermore, several drivers (ast, sis) already depend on this.
Signed-off-by: David Herrmann <dh.herrmann@gmail.com>
Reviewed-by: Daniel Vetter <daniel.vetter@ffwll.ch>
Signed-off-by: Dave Airlie <airlied@redhat.com>
Diffstat (limited to 'drivers/gpu/drm/ttm/ttm_bo_manager.c')
-rw-r--r-- | drivers/gpu/drm/ttm/ttm_bo_manager.c | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/drivers/gpu/drm/ttm/ttm_bo_manager.c b/drivers/gpu/drm/ttm/ttm_bo_manager.c index 9212494e9072..e4367f91472a 100644 --- a/drivers/gpu/drm/ttm/ttm_bo_manager.c +++ b/drivers/gpu/drm/ttm/ttm_bo_manager.c @@ -103,18 +103,12 @@ static int ttm_bo_man_init(struct ttm_mem_type_manager *man, unsigned long p_size) { struct ttm_range_manager *rman; - int ret; rman = kzalloc(sizeof(*rman), GFP_KERNEL); if (!rman) return -ENOMEM; - ret = drm_mm_init(&rman->mm, 0, p_size); - if (ret) { - kfree(rman); - return ret; - } - + drm_mm_init(&rman->mm, 0, p_size); spin_lock_init(&rman->lock); man->priv = rman; return 0; |