summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMarkus Elfring <elfring@users.sourceforge.net>2016-07-18 16:06:18 +0200
committerDaniel Vetter <daniel.vetter@ffwll.ch>2016-07-19 08:31:49 +0200
commita1bf09e69f8e938fef85f18c117ac390a48f7763 (patch)
treeafd3683cfdfb10e3b1c1bfdce14666ec7a2443e0
parentbf0901731ddf68e8c6d52d7fc0841fe3d4f66847 (diff)
downloadtalos-op-linux-a1bf09e69f8e938fef85f18c117ac390a48f7763.tar.gz
talos-op-linux-a1bf09e69f8e938fef85f18c117ac390a48f7763.zip
drm/ttm: Delete an unnecessary check before the function call "ttm_tt_destroy"
The ttm_tt_destroy() function tests whether its argument is NULL and then returns immediately. Thus the test around the call is not needed. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring <elfring@users.sourceforge.net> Signed-off-by: Daniel Vetter <daniel.vetter@ffwll.ch> Link: http://patchwork.freedesktop.org/patch/msgid/54338f58-830c-a8b4-4554-5d4459bcd321@users.sourceforge.net
-rw-r--r--drivers/gpu/drm/ttm/ttm_bo.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/gpu/drm/ttm/ttm_bo.c b/drivers/gpu/drm/ttm/ttm_bo.c
index 39386f50af87..4e55863e836b 100644
--- a/drivers/gpu/drm/ttm/ttm_bo.c
+++ b/drivers/gpu/drm/ttm/ttm_bo.c
@@ -146,9 +146,7 @@ static void ttm_bo_release_list(struct kref *list_kref)
BUG_ON(bo->mem.mm_node != NULL);
BUG_ON(!list_empty(&bo->lru));
BUG_ON(!list_empty(&bo->ddestroy));
-
- if (bo->ttm)
- ttm_tt_destroy(bo->ttm);
+ ttm_tt_destroy(bo->ttm);
atomic_dec(&bo->glob->bo_count);
if (bo->resv == &bo->ttm_resv)
reservation_object_fini(&bo->ttm_resv);
OpenPOWER on IntegriCloud