diff options
author | Colin Ian King <colin.king@canonical.com> | 2018-10-04 18:49:53 +0100 |
---|---|---|
committer | Sinclair Yeh <syeh@vmware.com> | 2018-10-05 14:58:10 -0700 |
commit | a876c7c9d4e8c4d0705756ded9b93245ec05c783 (patch) | |
tree | 058252ccda72709888a8ad3a0a11500583f57906 /vmwgfx_execbuf.c | |
parent | 84fc4945b92228289057ac89d3f2c8eb25858956 (diff) |
drm/vmwgfx: remove redundant return ret statement
The return statement is redundant as there is a return statement
immediately before it so we have dead code that can be removed.
Also remove the unused declaration of ret.
Detected by CoverityScan, CID#1473793 ("Structurally dead code")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Reviewed-by: Sinclair Yeh <syeh@vmware.com>
Diffstat (limited to 'vmwgfx_execbuf.c')
-rw-r--r-- | vmwgfx_execbuf.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/vmwgfx_execbuf.c b/vmwgfx_execbuf.c index 191950e..fa9bd83 100644 --- a/vmwgfx_execbuf.c +++ b/vmwgfx_execbuf.c @@ -1738,7 +1738,6 @@ static int vmw_cmd_check_define_gmrfb(struct vmw_private *dev_priv, void *buf) { struct vmw_buffer_object *vmw_bo; - int ret; struct { uint32_t header; @@ -1748,7 +1747,6 @@ static int vmw_cmd_check_define_gmrfb(struct vmw_private *dev_priv, return vmw_translate_guest_ptr(dev_priv, sw_context, &cmd->body.ptr, &vmw_bo); - return ret; } |