Commit 99b20148 authored by Sakari Ailus's avatar Sakari Ailus Committed by Vinod Koul
Browse files

dmaengine: at_xdmac: Use pm_runtime_put_noidle() with many usage_counts 



We're holding more than one Runtime PM usage_counts in
at_xdmac_device_terminate_all(). This makes pm_runtime_mark_last_busy()
redundant and pm_runtime_put_autosuspend() misleading. Drop
pm_runtime_mark_last_busy() and use pm_runtime_put_noidle() to decrement
the usage_count, except in the case it may be the last.

Signed-off-by: default avatarSakari Ailus <sakari.ailus@linux.intel.com>
Link: https://lore.kernel.org/r/20250410065804.3676582-1-sakari.ailus@linux.intel.com


Signed-off-by: default avatarVinod Koul <vkoul@kernel.org>
parent 862f3c49

drivers/dma/at_xdmac.c

+2 −4
Original line number Diff line number Diff line
@@ -2033,10 +2033,8 @@ static int at_xdmac_device_terminate_all(struct dma_chan *chan) 
		 * at_xdmac_start_xfer() for this descriptor. Now it's time

		 * to release it.

		 */

		if (desc->active_xfer) {

			pm_runtime_put_autosuspend(atxdmac->dev);

			pm_runtime_mark_last_busy(atxdmac->dev);

		}

		if (desc->active_xfer)

			pm_runtime_put_noidle(atxdmac->dev);

	}



	clear_bit(AT_XDMAC_CHAN_IS_PAUSED, &atchan->status);