From mboxrd@z Thu Jan 1 00:00:00 1970 From: Robert Jarzmik Subject: Re: [PATCH] spi: pxa2xx: Use newer more explicit DMAengine terminate API Date: Thu, 03 Mar 2016 19:25:01 +0100 Message-ID: <87twknpgg2.fsf@belgarion.home> References: <1457002934-9671-1-git-send-email-jarkko.nikula@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain Cc: linux-spi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Mark Brown , Daniel Mack , Haojian Zhuang , Mika Westerberg To: Jarkko Nikula Return-path: In-Reply-To: <1457002934-9671-1-git-send-email-jarkko.nikula-VuQAYsv1563Yd54FQh9/CA@public.gmane.org> (Jarkko Nikula's message of "Thu, 3 Mar 2016 13:02:14 +0200") Sender: linux-spi-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-ID: Jarkko Nikula writes: > Commit b36f09c3c441 ("dmaengine: Add transfer termination > synchronization support") marked dmaengine_terminate_all() as > deprecated and is being replaced by explicit synchronous and asynchronous > terminate functions. > > Here DMA termination are done in two cases: FIFO overrun and module > removal. > > FIFO overrun is handled in interrupt context and converting > dmaengine_terminate_all() to dmaengine_terminate_async() does the same than > before. > > Using synchronous termination in module removal however adds a bit more > robustness as it waits all completion callbacks have finished. Although it > looks all known DMA engines used with spi-pxa2xx don't implement > device_synchronize() callback so this too appears to be a no-op in > practice. > > Signed-off-by: Jarkko Nikula Acked-by: Robert Jarzmik Cheers. -- Robert -- To unsubscribe from this list: send the line "unsubscribe linux-spi" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html