On Thu, Apr 7, 2022 at 4:49 PM Sascha Hauer <s.hauer@xxxxxxxxxxxxxx> wrote:
> The micfil hardware provides the microphone data on multiple successive
> FIFO registers, one register per stereo pair. Also to work properly the
> SDMA_DONE0_CONFIG_DONE_SEL bit in the SDMA engines SDMA_DONE0_CONFIG
> register must be set. This patch provides the necessary information to
> the SDMA engine driver.
>
> Signed-off-by: Sascha Hauer <s.hauer@xxxxxxxxxxxxxx>
>
Acked-by: Shengjiu Wang <shengjiu.wang@xxxxxxxxx>
Best regards
Wang Shengjiu
> ---
>
> Notes:
> Changes since v3:
> - Fix include name
>
> Changes since v2:
> - Add forgotten commit message
>
> sound/soc/fsl/fsl_micfil.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
> diff --git a/sound/soc/fsl/fsl_micfil.c b/sound/soc/fsl/fsl_micfil.c
> index 4656a18a03e45..56df916ad55f2 100644
> --- a/sound/soc/fsl/fsl_micfil.c
> +++ b/sound/soc/fsl/fsl_micfil.c
> @@ -16,6 +16,7 @@
> #include <linux/regmap.h>
> #include <linux/sysfs.h>
> #include <linux/types.h>
> +#include <linux/dma/imx-dma.h>
> #include <sound/dmaengine_pcm.h>
> #include <sound/pcm.h>
> #include <sound/soc.h>
> @@ -35,6 +36,7 @@ struct fsl_micfil {
> struct clk *busclk;
> struct clk *mclk;
> struct snd_dmaengine_dai_dma_data dma_params_rx;
> + struct sdma_peripheral_config sdmacfg;
> unsigned int dataline;
> char name[32];
> int irq[MICFIL_IRQ_LINES];
> @@ -324,6 +326,10 @@ static int fsl_micfil_hw_params(struct
> snd_pcm_substream *substream,
> return ret;
> }
>
> + micfil->dma_params_rx.peripheral_config = &micfil->sdmacfg;
> + micfil->dma_params_rx.peripheral_size = sizeof(micfil->sdmacfg);
> + micfil->sdmacfg.n_fifos_src = channels;
> + micfil->sdmacfg.sw_done = true;
> micfil->dma_params_rx.maxburst = channels * MICFIL_DMA_MAXBURST_RX;
>
> return 0;
> --
> 2.30.2
>
>
