Commit 4cb8802e authored by Daniel Vetter's avatar Daniel Vetter

drm/mgag200: Remove unecessary NULL check in gem_free

The ->gem_free_object never gets called with a NULL pointer, the check
is redundant. Also checking after the upcast allows compilers to elide
it anyway.

Spotted by coverity.
Reviewed-by: default avatarDavid Herrmann <dh.herrmann@gmail.com>
Reviewed-by: default avatarIan Romanick <ian.d.romanick@intel.com>
Signed-off-by: default avatarDaniel Vetter <daniel.vetter@ffwll.ch>
parent 36b347fb
...@@ -329,8 +329,6 @@ void mgag200_gem_free_object(struct drm_gem_object *obj) ...@@ -329,8 +329,6 @@ void mgag200_gem_free_object(struct drm_gem_object *obj)
{ {
struct mgag200_bo *mgag200_bo = gem_to_mga_bo(obj); struct mgag200_bo *mgag200_bo = gem_to_mga_bo(obj);
if (!mgag200_bo)
return;
mgag200_bo_unref(&mgag200_bo); mgag200_bo_unref(&mgag200_bo);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment