Commit a2ab7045 authored by Paul Cercueil's avatar Paul Cercueil Committed by Vinod Koul
Browse files

dmaengine: axi-dmac: Small code cleanup 



Use a for() loop instead of a while() loop in axi_dmac_fill_linear_sg().
This makes the code leaner and cleaner overall, and does not introduce
any functional change.

Signed-off-by: default avatarPaul Cercueil <paul@crapouillou.net>
Link: https://lore.kernel.org/r/20231215131313.23840-2-paul@crapouillou.net


Signed-off-by: default avatarVinod Koul <vkoul@kernel.org>
parent 71e7d3cb

drivers/dma/dma-axi-dmac.c

+1 −4
Original line number Diff line number Diff line
@@ -508,16 +508,13 @@ static struct axi_dmac_sg *axi_dmac_fill_linear_sg(struct axi_dmac_chan *chan, 
	segment_size = ((segment_size - 1) | chan->length_align_mask) + 1;



	for (i = 0; i < num_periods; i++) {

		len = period_len;



		while (len > segment_size) {

		for (len = period_len; len > segment_size; sg++) {

			if (direction == DMA_DEV_TO_MEM)

				sg->dest_addr = addr;

			else

				sg->src_addr = addr;

			sg->x_len = segment_size;

			sg->y_len = 1;

			sg++;

			addr += segment_size;

			len -= segment_size;

		}