diff options
author | Markus Elfring <elfring@users.sourceforge.net> | 2016-07-16 03:54:22 -0400 |
---|---|---|
committer | Daniel Vetter <daniel.vetter@ffwll.ch> | 2016-07-18 03:11:32 -0400 |
commit | f7813aa03768cea8c371534ddee4b8af026d1b10 (patch) | |
tree | 31b295fe5581d4ca7a21333b0fd0ed56d7bb7ea2 /drivers/gpu/drm/ast/ast_main.c | |
parent | 536cea6073381aa9ace9743cd94e77d80018e3ee (diff) |
drm/ast: Delete an unnecessary check before drm_gem_object_unreference_unlocked()
The drm_gem_object_unreference_unlocked() 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/f28c063f-ec56-e62b-9370-89ce833fa4e5@users.sourceforge.net
Diffstat (limited to 'drivers/gpu/drm/ast/ast_main.c')
-rw-r--r-- | drivers/gpu/drm/ast/ast_main.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/gpu/drm/ast/ast_main.c b/drivers/gpu/drm/ast/ast_main.c index 7bc3aa6dda8c..904beaa932d0 100644 --- a/drivers/gpu/drm/ast/ast_main.c +++ b/drivers/gpu/drm/ast/ast_main.c | |||
@@ -295,9 +295,8 @@ static int ast_get_dram_info(struct drm_device *dev) | |||
295 | static void ast_user_framebuffer_destroy(struct drm_framebuffer *fb) | 295 | static void ast_user_framebuffer_destroy(struct drm_framebuffer *fb) |
296 | { | 296 | { |
297 | struct ast_framebuffer *ast_fb = to_ast_framebuffer(fb); | 297 | struct ast_framebuffer *ast_fb = to_ast_framebuffer(fb); |
298 | if (ast_fb->obj) | ||
299 | drm_gem_object_unreference_unlocked(ast_fb->obj); | ||
300 | 298 | ||
299 | drm_gem_object_unreference_unlocked(ast_fb->obj); | ||
301 | drm_framebuffer_cleanup(fb); | 300 | drm_framebuffer_cleanup(fb); |
302 | kfree(fb); | 301 | kfree(fb); |
303 | } | 302 | } |