dmaengine: fix is_slave_direction() return false when DMA_DEV_TO_DEV

nxpfrankli · gregkh · commit 8c859156f510 · 2024-02-14T15:22:32.000+01:00
[ Upstream commit a22fe1d ] 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> Signed-off-by: Sasha Levin <sashal@kernel.org>
diff --git a/include/linux/dmaengine.h b/include/linux/dmaengine.h
@@ -955,7 +955,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
`@@ -955,7 +955,8 @@ static inline int dmaengine_slave_config(struct dma_chan *chan,`
`955`	`955`
`956`	`956`	`static inline bool is_slave_direction(enum dma_transfer_direction direction)`
`957`	`957`	`{`
`958`		`- return (direction == DMA_MEM_TO_DEV) \|\| (direction == DMA_DEV_TO_MEM);`
	`958`	`+ return (direction == DMA_MEM_TO_DEV) \|\| (direction == DMA_DEV_TO_MEM) \|\|`
	`959`	`+ (direction == DMA_DEV_TO_DEV);`
`959`	`960`	`}`
`960`	`961`
`961`	`962`	`static inline struct dma_async_tx_descriptor *dmaengine_prep_slave_single(`