Commit eb76c4cc authored Mar 25, 2019 by YueHaibing Committed by Alistair Delva Aug 19, 2019

UPSTREAM: drm/virtio: remove set but not used variable 'vgdev'



Fixes gcc '-Wunused-but-set-variable' warning:

drivers/gpu/drm/virtio/virtgpu_ttm.c: In function 'virtio_gpu_init_mem_type':
drivers/gpu/drm/virtio/virtgpu_ttm.c:117:28: warning:
 variable 'vgdev' set but not used [-Wunused-but-set-variable]

drivers/gpu/drm/virtio/virtgpu_ttm.c: In function 'virtio_gpu_bo_swap_notify':
drivers/gpu/drm/virtio/virtgpu_ttm.c:300:28: warning:
 variable 'vgdev' set but not used [-Wunused-but-set-variable]

It is never used since introduction in dc5698e8 ("Add virtio gpu driver.")

Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Reviewed-by: Mukesh Ojha <mojha@codeaurora.org>
Link: http://patchwork.freedesktop.org/patch/msgid/20190325092631.152060-1-yuehaibing@huawei.com


Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
(cherry picked from commit df16a224d2b59213385cf012ecdb68a72a003fc6)
Signed-off-by: Greg Hartman <ghartman@google.com>
BUG: 139386237
Change-Id: I7611c3ecfb028e0b2dd776b1f3234e91a5749a9a

parent a4f8cf10

drivers/gpu/drm/virtio/virtgpu_ttm.c

+0 −6

Original line number	Diff line number	Diff line
		@@ -172,10 +172,6 @@ static const struct ttm_mem_type_manager_func virtio_gpu_bo_manager_func = {
		static int virtio_gpu_init_mem_type(struct ttm_bo_device *bdev, uint32_t type,
		struct ttm_mem_type_manager *man)
		{
		struct virtio_gpu_device *vgdev;

		vgdev = virtio_gpu_get_vgdev(bdev);

		switch (type) {
		case TTM_PL_SYSTEM:
		/* System memory */
		@@ -355,10 +351,8 @@ static void virtio_gpu_bo_move_notify(struct ttm_buffer_object *tbo,
		static void virtio_gpu_bo_swap_notify(struct ttm_buffer_object *tbo)
		{
		struct virtio_gpu_object *bo;
		struct virtio_gpu_device *vgdev;

		bo = container_of(tbo, struct virtio_gpu_object, tbo);
		vgdev = (struct virtio_gpu_device *)bo->gem_base.dev->dev_private;

		if (bo->pages)
		virtio_gpu_object_free_sg_table(bo);