Skip to content

Commit

Permalink
drm: Use the driver's gem_object_free function from CMA helpers.
Browse files Browse the repository at this point in the history
VC4 wraps the CMA objects in its own structures, so it needs to do its
own teardown (waiting for GPU to finish, updating bo_stats tracking).
The other CMA drivers are using drm_gem_cma_free_object as their
gem_free_object, so this should be a no-op for them.

Signed-off-by: Eric Anholt <[email protected]>
  • Loading branch information
anholt authored and popcornmix committed Dec 21, 2015
1 parent 5f726bd commit a881210
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
6 changes: 3 additions & 3 deletions drivers/gpu/drm/drm_fb_cma_helper.c
Original file line number Diff line number Diff line change
Expand Up @@ -266,7 +266,7 @@ static int drm_fbdev_cma_create(struct drm_fb_helper *helper,
fbi = drm_fb_helper_alloc_fbi(helper);
if (IS_ERR(fbi)) {
ret = PTR_ERR(fbi);
goto err_drm_gem_cma_free_object;
goto err_gem_free_object;
}

fbdev_cma->fb = drm_fb_cma_alloc(dev, &mode_cmd, &obj, 1);
Expand Down Expand Up @@ -299,8 +299,8 @@ static int drm_fbdev_cma_create(struct drm_fb_helper *helper,

err_fb_info_destroy:
drm_fb_helper_release_fbi(helper);
err_drm_gem_cma_free_object:
drm_gem_cma_free_object(&obj->base);
err_gem_free_object:
dev->driver->gem_free_object(&obj->base);
return ret;
}

Expand Down
4 changes: 2 additions & 2 deletions drivers/gpu/drm/drm_gem_cma_helper.c
Original file line number Diff line number Diff line change
Expand Up @@ -121,7 +121,7 @@ struct drm_gem_cma_object *drm_gem_cma_create(struct drm_device *drm,
return cma_obj;

error:
drm_gem_cma_free_object(&cma_obj->base);
drm->driver->gem_free_object(&cma_obj->base);
return ERR_PTR(ret);
}
EXPORT_SYMBOL_GPL(drm_gem_cma_create);
Expand Down Expand Up @@ -171,7 +171,7 @@ drm_gem_cma_create_with_handle(struct drm_file *file_priv,
return cma_obj;

err_handle_create:
drm_gem_cma_free_object(gem_obj);
drm->driver->gem_free_object(gem_obj);

return ERR_PTR(ret);
}
Expand Down

0 comments on commit a881210

Please sign in to comment.