ASoC: fsl-ssi: imx-pcm-fiq bugfix
authorMichael Grzeschik <m.grzeschik@pengutronix.de>
Mon, 19 Aug 2013 15:06:00 +0000 (17:06 +0200)
committerMark Brown <broonie@linaro.org>
Thu, 22 Aug 2013 10:10:25 +0000 (11:10 +0100)
imx-pcm-fiq is checking for TE RE bits, so enable them only if
necessary.

Signed-off-by: Michael Grzeschik <m.grzeschik@pengutronix.de>
Signed-off-by: Mark Brown <broonie@linaro.org>
sound/soc/fsl/fsl_ssi.c

index 6daeb5fbdc9bab314f6650dac71d713706e02ea8..198656fd171dfb764168caf12da903d7f924d916 100644 (file)
@@ -382,31 +382,12 @@ static int fsl_ssi_setup(struct fsl_ssi_private *ssi_private)
                CCSR_SSI_SFCSR_TFWM1(wm) | CCSR_SSI_SFCSR_RFWM1(wm),
                &ssi->sfcsr);
 
-       /*
-        * For non-ac97 setups, we keep the SSI disabled because if we enable
-        * it, then the DMA controller will start. It's not supposed to start
-        * until the SCR.TE (or SCR.RE) bit is set, but it does anyway. The DMA
-        * controller will transfer one "BWC" of data (i.e. the amount of data
-        * that the MR.BWC bits are set to). The reason this is bad is because
-        * at this point, the PCM driver has not finished initializing the DMA
-        * controller.
-        */
-
-
        /*
         * For ac97 interrupts are enabled with the startup of the substream
         * because it is also running without an active substream. Normally SSI
         * is only enabled when there is a substream.
         */
-       if (!ssi_private->imx_ac97) {
-               /* Enable the interrupts and DMA requests */
-               if (ssi_private->use_dma)
-                       write_ssi(SIER_FLAGS, &ssi->sier);
-               else
-                       write_ssi(CCSR_SSI_SIER_TIE | CCSR_SSI_SIER_TFE0_EN |
-                                       CCSR_SSI_SIER_RIE |
-                                       CCSR_SSI_SIER_RFF0_EN, &ssi->sier);
-       } else {
+       if (ssi_private->imx_ac97) {
                /*
                 * Setup the clock control register
                 */
@@ -574,6 +555,27 @@ static int fsl_ssi_trigger(struct snd_pcm_substream *substream, int cmd,
        struct snd_soc_pcm_runtime *rtd = substream->private_data;
        struct fsl_ssi_private *ssi_private = snd_soc_dai_get_drvdata(rtd->cpu_dai);
        struct ccsr_ssi __iomem *ssi = ssi_private->ssi;
+       unsigned int sier_bits;
+
+       /*
+        *  Enable only the interrupts and DMA requests
+        *  that are needed for the channel. As the fiq
+        *  is polling for this bits, we have to ensure
+        *  that this are aligned with the preallocated
+        *  buffers
+        */
+
+       if (substream->stream == SNDRV_PCM_STREAM_PLAYBACK) {
+               if (ssi_private->use_dma)
+                       sier_bits = SIER_FLAGS;
+               else
+                       sier_bits = CCSR_SSI_SIER_TIE | CCSR_SSI_SIER_TFE0_EN;
+       } else {
+               if (ssi_private->use_dma)
+                       sier_bits = SIER_FLAGS;
+               else
+                       sier_bits = CCSR_SSI_SIER_RIE | CCSR_SSI_SIER_RFF0_EN;
+       }
 
        switch (cmd) {
        case SNDRV_PCM_TRIGGER_START:
@@ -602,6 +604,8 @@ static int fsl_ssi_trigger(struct snd_pcm_substream *substream, int cmd,
                return -EINVAL;
        }
 
+       write_ssi(sier_bits, &ssi->sier);
+
        return 0;
 }