[LINUX PATCH 3/4] dmaengine: xilinx_dma: Fix compilation warning

Appana Durga Kedareswara Rao appanad at xilinx.com
Tue Jan 2 21:13:29 PST 2018


Hi Vinod,

	Thanks for the review... 

>
>On Thu, Dec 21, 2017 at 03:41:37PM +0530, Kedareswara rao Appana wrote:
>
>Fix title here too

Sure will fix in v2... 

>
>BTW whats with LINUX tag in patches, pls drop them

Ok will mention the Linux tag info in the cover letter patch from the next patch series on wards...

Regards,
Kedar.

>
>> This patch fixes the below sparse warning in the driver
>> drivers/dma/xilinx/xilinx_dma.c: In function
>‘xilinx_vdma_dma_prep_interleaved’:
>> drivers/dma/xilinx/xilinx_dma.c:1614:43: warning: variable ‘prev’ set but not
>used [-Wunused-but-set-variable]
>>   struct xilinx_vdma_tx_segment *segment, *prev = NULL;
>>
>> Signed-off-by: Kedareswara rao Appana <appanad at xilinx.com>
>> ---
>>  drivers/dma/xilinx/xilinx_dma.c | 4 +---
>>  1 file changed, 1 insertion(+), 3 deletions(-)
>>
>> diff --git a/drivers/dma/xilinx/xilinx_dma.c
>> b/drivers/dma/xilinx/xilinx_dma.c index 8467671..845e638 100644
>> --- a/drivers/dma/xilinx/xilinx_dma.c
>> +++ b/drivers/dma/xilinx/xilinx_dma.c
>> @@ -1611,7 +1611,7 @@ xilinx_vdma_dma_prep_interleaved(struct dma_chan
>> *dchan,  {
>>  	struct xilinx_dma_chan *chan = to_xilinx_chan(dchan);
>>  	struct xilinx_dma_tx_descriptor *desc;
>> -	struct xilinx_vdma_tx_segment *segment, *prev = NULL;
>> +	struct xilinx_vdma_tx_segment *segment;
>>  	struct xilinx_vdma_desc_hw *hw;
>>
>>  	if (!is_slave_direction(xt->dir))
>> @@ -1665,8 +1665,6 @@ xilinx_vdma_dma_prep_interleaved(struct dma_chan
>*dchan,
>>  	/* Insert the segment into the descriptor segments list. */
>>  	list_add_tail(&segment->node, &desc->segments);
>>
>> -	prev = segment;
>> -
>>  	/* Link the last hardware descriptor with the first. */
>>  	segment = list_first_entry(&desc->segments,
>>  				   struct xilinx_vdma_tx_segment, node);
>> --
>> 2.7.4
>>
>
>--
>~Vinod


More information about the linux-arm-kernel mailing list