Commit 43ebfe61 authored by Lee Jones's avatar Lee Jones Committed by Zack Rusin
Browse files

drm/vmwgfx/vmwgfx_cmdbuf_res: Remove unused variable 'ret' 



Fixes the following W=1 kernel build warning(s):

 drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf_res.c: In function ‘vmw_cmdbuf_res_revert’:
 drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf_res.c:162:6: warning: variable ‘ret’ set but not used [-Wunused-but-set-variable]

Cc: VMware Graphics <linux-graphics-maintainer@vmware.com>
Cc: Roland Scheidegger <sroland@vmware.com>
Cc: Zack Rusin <zackr@vmware.com>
Cc: David Airlie <airlied@linux.ie>
Cc: Daniel Vetter <daniel@ffwll.ch>
Cc: dri-devel@lists.freedesktop.org
Signed-off-by: default avatarLee Jones <lee.jones@linaro.org>
Signed-off-by: default avatarZack Rusin <zackr@vmware.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20210115181313.3431493-40-lee.jones@linaro.org
parent 17ef20f1

drivers/gpu/drm/vmwgfx/vmwgfx_cmdbuf_res.c

+1 −3
Original line number Diff line number Diff line
@@ -159,7 +159,6 @@ void vmw_cmdbuf_res_commit(struct list_head *list) 
void vmw_cmdbuf_res_revert(struct list_head *list)

{

	struct vmw_cmdbuf_res *entry, *next;

	int ret;



	list_for_each_entry_safe(entry, next, list, head) {

		switch (entry->state) {
@@ -167,8 +166,7 @@ void vmw_cmdbuf_res_revert(struct list_head *list) 
			vmw_cmdbuf_res_free(entry->man, entry);

			break;

		case VMW_CMDBUF_RES_DEL:

			ret = drm_ht_insert_item(&entry->man->resources,

						 &entry->hash);

			drm_ht_insert_item(&entry->man->resources, &entry->hash);

			list_del(&entry->head);

			list_add_tail(&entry->head, &entry->man->list);

			entry->state = VMW_CMDBUF_RES_COMMITTED;