diff options
| author | Colin Ian King <colin.king@canonical.com> | 2018-10-04 13:49:53 -0400 |
|---|---|---|
| committer | Thomas Hellstrom <thellstrom@vmware.com> | 2018-12-05 04:09:16 -0500 |
| commit | ec9a5b611e2fde765372885220555a5b38e13c09 (patch) | |
| tree | ab8e55e95751fab71d9de14ae20ea7185ab6ae18 /drivers/gpu/drm | |
| parent | 9da6e26c0aae3fda6017c1ecf5c8881f8dbc37df (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>
Signed-off-by: Thomas Hellstrom <thellstrom@vmware.com>
Diffstat (limited to 'drivers/gpu/drm')
| -rw-r--r-- | drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c b/drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c index 5a6b70ba137a..260650bb5560 100644 --- a/drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_execbuf.c | |||
| @@ -1738,7 +1738,6 @@ static int vmw_cmd_check_define_gmrfb(struct vmw_private *dev_priv, | |||
| 1738 | void *buf) | 1738 | void *buf) |
| 1739 | { | 1739 | { |
| 1740 | struct vmw_buffer_object *vmw_bo; | 1740 | struct vmw_buffer_object *vmw_bo; |
| 1741 | int ret; | ||
| 1742 | 1741 | ||
| 1743 | struct { | 1742 | struct { |
| 1744 | uint32_t header; | 1743 | uint32_t header; |
| @@ -1748,7 +1747,6 @@ static int vmw_cmd_check_define_gmrfb(struct vmw_private *dev_priv, | |||
| 1748 | return vmw_translate_guest_ptr(dev_priv, sw_context, | 1747 | return vmw_translate_guest_ptr(dev_priv, sw_context, |
| 1749 | &cmd->body.ptr, | 1748 | &cmd->body.ptr, |
| 1750 | &vmw_bo); | 1749 | &vmw_bo); |
| 1751 | return ret; | ||
| 1752 | } | 1750 | } |
| 1753 | 1751 | ||
| 1754 | 1752 | ||
