mirror of
https://github.com/torvalds/linux.git
synced 2026-06-01 02:53:36 +02:00
ASoC: fsl_dma: Use managed buffer allocation
This patch simplifies the buffer pre-allocation code of fsl_dma driver with the standard managed buffer helper. It uses the newly introduced fixed-size buffer allocation helper. Cc: Nicolin Chen <nicoleotsuka@gmail.com> Cc: Xiubo Li <Xiubo.Lee@gmail.com> Cc: Fabio Estevam <festevam@gmail.com> Cc: Shengjiu Wang <shengjiu.wang@gmail.com> Acked-by: Mark Brown <broonie@kernel.org> Link: https://lore.kernel.org/r/20210802072815.13551-12-tiwai@suse.de Signed-off-by: Takashi Iwai <tiwai@suse.de>
This commit is contained in:
parent
1855ce6293
commit
e159704f79
|
|
@ -290,32 +290,9 @@ static int fsl_dma_new(struct snd_soc_component *component,
|
||||||
if (ret)
|
if (ret)
|
||||||
return ret;
|
return ret;
|
||||||
|
|
||||||
/* Some codecs have separate DAIs for playback and capture, so we
|
return snd_pcm_set_fixed_buffer_all(pcm, SNDRV_DMA_TYPE_DEV,
|
||||||
* should allocate a DMA buffer only for the streams that are valid.
|
card->dev,
|
||||||
*/
|
fsl_dma_hardware.buffer_bytes_max);
|
||||||
|
|
||||||
if (pcm->streams[SNDRV_PCM_STREAM_PLAYBACK].substream) {
|
|
||||||
ret = snd_dma_alloc_pages(SNDRV_DMA_TYPE_DEV, card->dev,
|
|
||||||
fsl_dma_hardware.buffer_bytes_max,
|
|
||||||
&pcm->streams[SNDRV_PCM_STREAM_PLAYBACK].substream->dma_buffer);
|
|
||||||
if (ret) {
|
|
||||||
dev_err(card->dev, "can't alloc playback dma buffer\n");
|
|
||||||
return ret;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
if (pcm->streams[SNDRV_PCM_STREAM_CAPTURE].substream) {
|
|
||||||
ret = snd_dma_alloc_pages(SNDRV_DMA_TYPE_DEV, card->dev,
|
|
||||||
fsl_dma_hardware.buffer_bytes_max,
|
|
||||||
&pcm->streams[SNDRV_PCM_STREAM_CAPTURE].substream->dma_buffer);
|
|
||||||
if (ret) {
|
|
||||||
dev_err(card->dev, "can't alloc capture dma buffer\n");
|
|
||||||
snd_dma_free_pages(&pcm->streams[SNDRV_PCM_STREAM_PLAYBACK].substream->dma_buffer);
|
|
||||||
return ret;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
return 0;
|
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
|
|
@ -442,7 +419,6 @@ static int fsl_dma_open(struct snd_soc_component *component,
|
||||||
|
|
||||||
dma->assigned = true;
|
dma->assigned = true;
|
||||||
|
|
||||||
snd_pcm_set_runtime_buffer(substream, &substream->dma_buffer);
|
|
||||||
snd_soc_set_runtime_hwparams(substream, &fsl_dma_hardware);
|
snd_soc_set_runtime_hwparams(substream, &fsl_dma_hardware);
|
||||||
runtime->private_data = dma_private;
|
runtime->private_data = dma_private;
|
||||||
|
|
||||||
|
|
@ -815,25 +791,6 @@ static int fsl_dma_close(struct snd_soc_component *component,
|
||||||
return 0;
|
return 0;
|
||||||
}
|
}
|
||||||
|
|
||||||
/*
|
|
||||||
* Remove this PCM driver.
|
|
||||||
*/
|
|
||||||
static void fsl_dma_free_dma_buffers(struct snd_soc_component *component,
|
|
||||||
struct snd_pcm *pcm)
|
|
||||||
{
|
|
||||||
struct snd_pcm_substream *substream;
|
|
||||||
unsigned int i;
|
|
||||||
|
|
||||||
for (i = 0; i < ARRAY_SIZE(pcm->streams); i++) {
|
|
||||||
substream = pcm->streams[i].substream;
|
|
||||||
if (substream) {
|
|
||||||
snd_dma_free_pages(&substream->dma_buffer);
|
|
||||||
substream->dma_buffer.area = NULL;
|
|
||||||
substream->dma_buffer.addr = 0;
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* find_ssi_node -- returns the SSI node that points to its DMA channel node
|
* find_ssi_node -- returns the SSI node that points to its DMA channel node
|
||||||
*
|
*
|
||||||
|
|
@ -904,7 +861,6 @@ static int fsl_soc_dma_probe(struct platform_device *pdev)
|
||||||
dma->dai.hw_free = fsl_dma_hw_free;
|
dma->dai.hw_free = fsl_dma_hw_free;
|
||||||
dma->dai.pointer = fsl_dma_pointer;
|
dma->dai.pointer = fsl_dma_pointer;
|
||||||
dma->dai.pcm_construct = fsl_dma_new;
|
dma->dai.pcm_construct = fsl_dma_new;
|
||||||
dma->dai.pcm_destruct = fsl_dma_free_dma_buffers;
|
|
||||||
|
|
||||||
/* Store the SSI-specific information that we need */
|
/* Store the SSI-specific information that we need */
|
||||||
dma->ssi_stx_phys = res.start + REG_SSI_STX0;
|
dma->ssi_stx_phys = res.start + REG_SSI_STX0;
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue
Block a user