dmaengine: fix is_slave_direction() return false when DMA_DEV_TO_DEV

nxpfrankli · vinodkoul · commit a22fe1d6dec7 · 2024-01-30T22:11:13.000+05:30
is_slave_direction() should return true when direction is DMA_DEV_TO_DEV. Fixes: 49920bc ("dmaengine: add new enum dma_transfer_direction") Signed-off-by: Frank Li <Frank.Li@nxp.com> Link: https://lore.kernel.org/r/20240123172842.3764529-1-Frank.Li@nxp.com Signed-off-by: Vinod Koul <vkoul@kernel.org>
diff --git a/include/linux/dmaengine.h b/include/linux/dmaengine.h
@@ -953,7 +953,8 @@ static inline int dmaengine_slave_config(struct dma_chan *chan,
 
 static inline bool is_slave_direction(enum dma_transfer_direction direction)
 {
-	return (direction == DMA_MEM_TO_DEV) || (direction == DMA_DEV_TO_MEM);
+	return (direction == DMA_MEM_TO_DEV) || (direction == DMA_DEV_TO_MEM) ||
+	       (direction == DMA_DEV_TO_DEV);
 }
 
 static inline struct dma_async_tx_descriptor *dmaengine_prep_slave_single(

Original file line number	Diff line number	Diff line change
`@@ -953,7 +953,8 @@ static inline int dmaengine_slave_config(struct dma_chan *chan,`
`953`	`953`
`954`	`954`	`static inline bool is_slave_direction(enum dma_transfer_direction direction)`
`955`	`955`	`{`
`956`		`- return (direction == DMA_MEM_TO_DEV) \|\| (direction == DMA_DEV_TO_MEM);`
	`956`	`+ return (direction == DMA_MEM_TO_DEV) \|\| (direction == DMA_DEV_TO_MEM) \|\|`
	`957`	`+ (direction == DMA_DEV_TO_DEV);`
`957`	`958`	`}`
`958`	`959`
`959`	`960`	`static inline struct dma_async_tx_descriptor *dmaengine_prep_slave_single(`