On Mon, Jun 10, 2024 at 05:45:52PM -0700, Lars-Peter Clausen wrote: > On 6/10/24 03:56, Jai Luthra wrote: > > Sometimes the stream may be stopped due to XRUN events, in which case > > the userspace can call snd_pcm_drop() and snd_pcm_prepare() to stop and > > start the stream again. > > In these cases, we must wait for the DMA channel to synchronize before > > marking the stream as prepared for playback, as the DMA channel gets > > stopped by snd_pcm_drop() without any synchronization. > We should really implement the sync_stop() PCM callback and let the ALSA > core let care of the sync. Good point, that's a better idea.
Attachment:
signature.asc
Description: PGP signature
- Follow-Ups:
- References:
- [PATCH v2 0/2] Fixes for McASP and dmaengine_pcm
- From: Jai Luthra
- [PATCH v2 1/2] ALSA: dmaengine: Synchronize dma channel in prepare()
- From: Jai Luthra
- Re: [PATCH v2 1/2] ALSA: dmaengine: Synchronize dma channel in prepare()
- From: Lars-Peter Clausen
- [PATCH v2 0/2] Fixes for McASP and dmaengine_pcm
- Prev by Date: Re: [Patch v2 2/2] ASoC: dt-bindings: lpc32xx: Add lpc32xx i2s DT binding
- Next by Date: Re: [Patch v2 2/2] ASoC: dt-bindings: lpc32xx: Add lpc32xx i2s DT binding
- Previous by thread: Re: [PATCH v2 1/2] ALSA: dmaengine: Synchronize dma channel in prepare()
- Next by thread: Re: Re: [PATCH v2 1/2] ALSA: dmaengine: Synchronize dma channel in prepare()
- Index(es):