serial: 8250_dma: terminate correct DMA in tx_dma_flush()

johnkeeping · vijay-suman · commit 0c99f5ee397b · 2025-04-16T03:06:16.000-07:00
commit a26503092c75abba70a0be2aa01145ecf90c2a22 upstream. When flushing transmit side DMA, it is the transmit channel that should be terminated, not the receive channel. Fixes: 9e512ea ("serial: 8250: Fix fifo underflow on flush") Cc: stable <stable@kernel.org> Reported-by: Wentao Guan <guanwentao@uniontech.com> Signed-off-by: John Keeping <jkeeping@inmusicbrands.com> Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com> Link: https://lore.kernel.org/r/20250224121831.1429323-1-jkeeping@inmusicbrands.com Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> (cherry picked from commit 03c4c633a0226341067698975f52a55fe6d3abeb) FOF: 0525 Signed-off-by: Vijayendra Suman <vijayendra.suman@oracle.com>
diff --git a/drivers/tty/serial/8250/8250_dma.c b/drivers/tty/serial/8250/8250_dma.c
@@ -150,7 +150,7 @@ void serial8250_tx_dma_flush(struct uart_8250_port *p)
 	 */
 	dma->tx_size = 0;
 
-	dmaengine_terminate_async(dma->rxchan);
+	dmaengine_terminate_async(dma->txchan);
 }
 
 int serial8250_rx_dma(struct uart_8250_port *p)

Original file line number	Diff line number	Diff line change
`@@ -150,7 +150,7 @@ void serial8250_tx_dma_flush(struct uart_8250_port *p)`
`150`	`150`	`*/`
`151`	`151`	`dma->tx_size = 0;`
`152`	`152`
`153`		`- dmaengine_terminate_async(dma->rxchan);`
	`153`	`+ dmaengine_terminate_async(dma->txchan);`
`154`	`154`	`}`
`155`	`155`
`156`	`156`	`int serial8250_rx_dma(struct uart_8250_port *p)`