diff options
| author | Peter Ujfalusi <peter.ujfalusi@ti.com> | 2020-02-27 12:35:44 +0300 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2020-03-06 18:02:51 +0300 |
| commit | f9c23615c688270d2a383bd752f7a54a7137d596 (patch) | |
| tree | 3d7b2f0e610df65f15d18939f1888b81106365ac /sound/core/pcm_dmaengine.c | |
| parent | 6e02feb0d2663c1b7caa5e271c2a60e219f0ca07 (diff) | |
| download | linux-f9c23615c688270d2a383bd752f7a54a7137d596.tar.xz | |
ALSA: dmaengine_pcm: No need to take runtime reference twice in pcm_pointer
The runtime pointer has been taken in functional level so there is no need
to take it again under the if () case.
Fixes: 9d789dc047e3 ("ALSA: dmaengine_pcm: Consider DMA cache caused delay in pointer callback")
Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
Link: https://lore.kernel.org/r/20200227093544.27723-1-peter.ujfalusi@ti.com
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/core/pcm_dmaengine.c')
| -rw-r--r-- | sound/core/pcm_dmaengine.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/sound/core/pcm_dmaengine.c b/sound/core/pcm_dmaengine.c index 6852bb670b4e..9d4f48cfe47f 100644 --- a/sound/core/pcm_dmaengine.c +++ b/sound/core/pcm_dmaengine.c @@ -248,8 +248,6 @@ snd_pcm_uframes_t snd_dmaengine_pcm_pointer(struct snd_pcm_substream *substream) status = dmaengine_tx_status(prtd->dma_chan, prtd->cookie, &state); if (status == DMA_IN_PROGRESS || status == DMA_PAUSED) { - struct snd_pcm_runtime *runtime = substream->runtime; - buf_size = snd_pcm_lib_buffer_bytes(substream); if (state.residue > 0 && state.residue <= buf_size) pos = buf_size - state.residue; |