There is no point to return DMA_ERROR if txstatus parameter is NULL. It's a valid case and should be handled correspondingly. Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> --- drivers/dma/at_hdmac.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/dma/at_hdmac.c b/drivers/dma/at_hdmac.c index a9d8dd990d6e..4035d5438530 100644 --- a/drivers/dma/at_hdmac.c +++ b/drivers/dma/at_hdmac.c @@ -1679,7 +1679,7 @@ atc_tx_status(struct dma_chan *chan, if (!txstate) { if (test_bit(ATC_IS_PAUSED, &atchan->status)) return DMA_PAUSED; - return DMA_ERROR; + return dma_status; } spin_lock_irqsave(&atchan->vc.lock, flags); -- 2.35.1