]> git.proxmox.com Git - mirror_ubuntu-kernels.git/commitdiff
ASoC: imx-pcm-dma: simplify pcm_config
authorLucas Stach <l.stach@pengutronix.de>
Wed, 8 Mar 2017 09:17:02 +0000 (10:17 +0100)
committerMark Brown <broonie@kernel.org>
Wed, 8 Mar 2017 09:58:42 +0000 (10:58 +0100)
The generic snd_dmaengine_pcm is able to retrieve all the needed
information from the attached dmaengine and is in fact able to
provide much more accurate flags to userspace, like the SDMA engine
being only able to operate in batch mode.

To avoid any future inconsistencies between the dmaengine and the
pcm_config, rip out the fixed config and rely on the core to fill
in the right flags derived from the dmaengine information.

Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/fsl/imx-pcm-dma.c

index f3d3d1ffa84e4257029495785ca88e72cb013533..314814ddd2b0a406799058df39611c1dafd8935e 100644 (file)
@@ -33,48 +33,20 @@ static bool filter(struct dma_chan *chan, void *param)
        return true;
 }
 
-static const struct snd_pcm_hardware imx_pcm_hardware = {
-       .info = SNDRV_PCM_INFO_INTERLEAVED |
-               SNDRV_PCM_INFO_BLOCK_TRANSFER |
-               SNDRV_PCM_INFO_MMAP |
-               SNDRV_PCM_INFO_MMAP_VALID |
-               SNDRV_PCM_INFO_PAUSE |
-               SNDRV_PCM_INFO_RESUME,
-       .buffer_bytes_max = IMX_DEFAULT_DMABUF_SIZE,
-       .period_bytes_min = 128,
-       .period_bytes_max = 65535, /* Limited by SDMA engine */
-       .periods_min = 2,
-       .periods_max = 255,
-       .fifo_size = 0,
-};
-
 static const struct snd_dmaengine_pcm_config imx_dmaengine_pcm_config = {
-       .pcm_hardware = &imx_pcm_hardware,
        .prepare_slave_config = snd_dmaengine_pcm_prepare_slave_config,
        .compat_filter_fn = filter,
-       .prealloc_buffer_size = IMX_DEFAULT_DMABUF_SIZE,
 };
 
 int imx_pcm_dma_init(struct platform_device *pdev, size_t size)
 {
        struct snd_dmaengine_pcm_config *config;
-       struct snd_pcm_hardware *pcm_hardware;
 
        config = devm_kzalloc(&pdev->dev,
                        sizeof(struct snd_dmaengine_pcm_config), GFP_KERNEL);
        if (!config)
                return -ENOMEM;
        *config = imx_dmaengine_pcm_config;
-       if (size)
-               config->prealloc_buffer_size = size;
-
-       pcm_hardware = devm_kzalloc(&pdev->dev,
-                       sizeof(struct snd_pcm_hardware), GFP_KERNEL);
-       *pcm_hardware = imx_pcm_hardware;
-       if (size)
-               pcm_hardware->buffer_bytes_max = size;
-
-       config->pcm_hardware = pcm_hardware;
 
        return devm_snd_dmaengine_pcm_register(&pdev->dev,
                config,