Message ID | 20150610192751.GB10145@mwanda (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On Mi, 2015-06-10 at 22:27 +0300, Dan Carpenter wrote: > The goto is correct, and we never reach the return statement so just > delete the dead code. > > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> Reviewed-by: Gerd Hoffmann <kraxel@redhat.com>
diff --git a/drivers/gpu/drm/virtio/virtgpu_ttm.c b/drivers/gpu/drm/virtio/virtgpu_ttm.c index e0e74c6..b092d7b 100644 --- a/drivers/gpu/drm/virtio/virtgpu_ttm.c +++ b/drivers/gpu/drm/virtio/virtgpu_ttm.c @@ -443,14 +443,12 @@ int virtio_gpu_ttm_init(struct virtio_gpu_device *vgdev) if (r) { DRM_ERROR("failed initializing buffer object driver(%d).\n", r); goto err_dev_init; - return r; } r = ttm_bo_init_mm(&vgdev->mman.bdev, TTM_PL_TT, 0); if (r) { DRM_ERROR("Failed initializing GTT heap.\n"); goto err_mm_init; - return r; } return 0;
The goto is correct, and we never reach the return statement so just delete the dead code. Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>