Thread (18 messages) 18 messages, 5 authors, 2021-04-22

Re: [PATCH] ASoC: fsl: imx-pcm-dma: Don't request dma channel in probe

From: Mark Brown <broonie@kernel.org>
Date: 2021-04-21 17:46:58
Also in: alsa-devel, dri-devel, linux-arm-kernel, linux-media, lkml

On Wed, Apr 21, 2021 at 07:43:18PM +0200, Lucas Stach wrote:
If your driver code drops the rpm refcount to 0 and starts the
autosuspend timer while a cyclic transfer is still in flight this is
clearly a bug. Autosuspend is not there to paper over driver bugs, but
to amortize cost of actually suspending and resuming the hardware. Your
driver code must still work even if the timeout is 0, i.e. the hardware
is immediately suspended after you drop the rpm refcount to 0.
If you still have transfers queued/in-flight the driver code must keep
a rpm reference.
Right, failing to do that is a clear bug.

Please delete unneeded context from mails when replying.  Doing this
makes it much easier to find your reply in the message, helping ensure
it won't be missed by people scrolling through the irrelevant quoted
material.

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help