On 18/10/2019 16:48, Ben Dooks wrote: > We see odd FIFO overruns with this, we assume the best thing to do is > to disable the RX I2S frontend first, and then disable the FIFO that > is using it. > > This also fixes an issue where using multi-word frames (TDM) have > partial samples stuck in the FIFO which then get read out when the > next capture is started. > > Signed-off-by: Ben Dooks <ben.dooks@xxxxxxxxxxxxxxx> > --- > sound/soc/tegra/tegra30_i2s.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/sound/soc/tegra/tegra30_i2s.c b/sound/soc/tegra/tegra30_i2s.c > index fc77e65a3646..3839e3d955a8 100644 > --- a/sound/soc/tegra/tegra30_i2s.c > +++ b/sound/soc/tegra/tegra30_i2s.c > @@ -236,9 +236,9 @@ static void tegra30_i2s_start_capture(struct tegra30_i2s *i2s) > > static void tegra30_i2s_stop_capture(struct tegra30_i2s *i2s) > { > - tegra30_ahub_disable_rx_fifo(i2s->capture_fifo_cif); > regmap_update_bits(i2s->regmap, TEGRA30_I2S_CTRL, > TEGRA30_I2S_CTRL_XFER_EN_RX, 0); > + tegra30_ahub_disable_rx_fifo(i2s->capture_fifo_cif); > } > > static int tegra30_i2s_trigger(struct snd_pcm_substream *substream, int cmd, Acked-by: Jon Hunter <jonathanh@xxxxxxxxxx> Thanks! Jon -- nvpublic _______________________________________________ Alsa-devel mailing list Alsa-devel@xxxxxxxxxxxxxxxx https://mailman.alsa-project.org/mailman/listinfo/alsa-devel