| Message ID | 20220407084936.223075-13-s.hauer@pengutronix.de (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | ASoC: fsl_micfil: Driver updates | expand |
On Thu, Apr 7, 2022 at 4:49 PM Sascha Hauer <s.hauer@pengutronix.de> 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@pengutronix.de> > Acked-by: Shengjiu Wang <shengjiu.wang@gmail.com> 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 > >
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;
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@pengutronix.de> --- 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(+)