Commit 3c3f7624 authored Oct 12, 2018 by YueHaibing Committed by Greg Kroah-Hartman Oct 15, 2018

misc: mic: scif: remove set but not used variables 'src_dma_addr, dst_dma_addr'



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

drivers/misc/mic/scif/scif_dma.c: In function 'scif_rma_list_dma_copy_wrapper':
drivers/misc/mic/scif/scif_dma.c:1558:27: warning:
 variable 'dst_dma_addr' set but not used [-Wunused-but-set-variable]

drivers/misc/mic/scif/scif_dma.c:1558:13: warning:
 variable 'src_dma_addr' set but not used [-Wunused-but-set-variable]

They never used since introduction in
commit 7cc31cd2 ("misc: mic: SCIF DMA and CPU copy interface")

Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

parent 6b995f4e

drivers/misc/mic/scif/scif_dma.c

+0 −3

Original line number	Diff line number	Diff line
		@@ -1553,7 +1553,6 @@ static int scif_rma_list_dma_copy_wrapper(struct scif_endpt *epd,
		u8 *temp = NULL;
		bool src_local = true;
		struct scif_dma_comp_cb *comp_cb;
		dma_addr_t src_dma_addr, dst_dma_addr;
		int err;

		if (is_dma_copy_aligned(chan->device, 1, 1, 1))
		@@ -1567,8 +1566,6 @@ static int scif_rma_list_dma_copy_wrapper(struct scif_endpt *epd,

		if (work->loopback)
		return scif_rma_list_cpu_copy(work);
		src_dma_addr = __scif_off_to_dma_addr(work->src_window, src_offset);
		dst_dma_addr = __scif_off_to_dma_addr(work->dst_window, dst_offset);
		src_local = work->src_window->type == SCIF_WINDOW_SELF;

		/* Allocate dma_completion cb */

Admin message