On Tue, Oct 14, 2025 at 05:05:22PM +0800, Zhen Ni wrote:

When fsl_edma_alloc_chan_resources() fails after clk_prepare_enable(), the error paths only free IRQs and destroy the TCD pool, but forget to call clk_disable_unprepare(). This causes the channel clock to remain enabled, leaking power and resources.

Fix it by disabling the channel clock in the error unwind path.

Fixes: d8d4355861d8 ("dmaengine: fsl-edma: add i.MX8ULP edma support") Cc: stable@vger.kernel.org Suggested-by: Frank Li Frank.Li@nxp.com Signed-off-by: Zhen Ni zhen.ni@easystack.cn

Reviewed-by: Frank Li Frank.Li@nxp.com

---

Changes in v2:

• Remove FSL_EDMA_DRV_HAS_CHCLK check

Changes in v3:

• Remove cleanup

Changes in v4:

• Re-send as a new thread

---

drivers/dma/fsl-edma-common.c | 1 + 1 file changed, 1 insertion(+)

diff --git a/drivers/dma/fsl-edma-common.c b/drivers/dma/fsl-edma-common.c index 4976d7dde080..11655dcc4d6c 100644 --- a/drivers/dma/fsl-edma-common.c +++ b/drivers/dma/fsl-edma-common.c @@ -852,6 +852,7 @@ int fsl_edma_alloc_chan_resources(struct dma_chan *chan) free_irq(fsl_chan->txirq, fsl_chan); err_txirq: dma_pool_destroy(fsl_chan->tcd_pool);

• clk_disable_unprepare(fsl_chan->clk);

  return ret;

}

2.20.1