aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/gpu/drm/radeon
diff options
context:
space:
mode:
authorMarkus Elfring <elfring@users.sourceforge.net>2016-07-12 16:00:55 -0400
committerAlex Deucher <alexander.deucher@amd.com>2016-07-29 14:36:51 -0400
commit00f687292568c8fb1609a63ac56f3c88b91c1084 (patch)
tree89c96f90221833c80639378d331162912b7b91ab /drivers/gpu/drm/radeon
parent538f1ef34c71aa5c6a7bd1b65d5e603080ff7b74 (diff)
GPU-DRM-Radeon: 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: Alex Deucher <alexander.deucher@amd.com>
Diffstat (limited to 'drivers/gpu/drm/radeon')
-rw-r--r--drivers/gpu/drm/radeon/radeon_display.c4
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/gpu/drm/radeon/radeon_display.c b/drivers/gpu/drm/radeon/radeon_display.c
index 5f1cd695c965..1efab5fbf332 100644
--- a/drivers/gpu/drm/radeon/radeon_display.c
+++ b/drivers/gpu/drm/radeon/radeon_display.c
@@ -1324,9 +1324,7 @@ static void radeon_user_framebuffer_destroy(struct drm_framebuffer *fb)
1324{ 1324{
1325 struct radeon_framebuffer *radeon_fb = to_radeon_framebuffer(fb); 1325 struct radeon_framebuffer *radeon_fb = to_radeon_framebuffer(fb);
1326 1326
1327 if (radeon_fb->obj) { 1327 drm_gem_object_unreference_unlocked(radeon_fb->obj);
1328 drm_gem_object_unreference_unlocked(radeon_fb->obj);
1329 }
1330 drm_framebuffer_cleanup(fb); 1328 drm_framebuffer_cleanup(fb);
1331 kfree(radeon_fb); 1329 kfree(radeon_fb);
1332} 1330}