ASoC: cirrus: i2s: Stop enabling I2S2 and I2S3 FIFOs
authorAlexander Sverdlin <alexander.sverdlin@gmail.com>
Sat, 28 Apr 2018 20:51:40 +0000 (22:51 +0200)
committerMark Brown <broonie@kernel.org>
Fri, 11 May 2018 02:27:14 +0000 (11:27 +0900)
The driver never supported more than 2 channels because of
ep93xx_i2s_dma_data[] supporting only 1 DMA channel in each
direction.
Stop enabling two unused I2S controller FIFOs, this will simplify
future interrupt support.

Signed-off-by: Alexander Sverdlin <alexander.sverdlin@gmail.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/cirrus/ep93xx-i2s.c

index 0dc3852c46219f1aca029a08b7a46681d0b0a389..42cc6d22baac09c29680f7a17518eaaddd43a3b0 100644 (file)
@@ -98,7 +98,6 @@ static inline unsigned ep93xx_i2s_read_reg(struct ep93xx_i2s_info *info,
 static void ep93xx_i2s_enable(struct ep93xx_i2s_info *info, int stream)
 {
        unsigned base_reg;
-       int i;
 
        if ((ep93xx_i2s_read_reg(info, EP93XX_I2S_TX0EN) & 0x1) == 0 &&
            (ep93xx_i2s_read_reg(info, EP93XX_I2S_RX0EN) & 0x1) == 0) {
@@ -111,27 +110,24 @@ static void ep93xx_i2s_enable(struct ep93xx_i2s_info *info, int stream)
                ep93xx_i2s_write_reg(info, EP93XX_I2S_GLCTRL, 1);
        }
 
-       /* Enable fifos */
+       /* Enable fifo */
        if (stream == SNDRV_PCM_STREAM_PLAYBACK)
                base_reg = EP93XX_I2S_TX0EN;
        else
                base_reg = EP93XX_I2S_RX0EN;
-       for (i = 0; i < 3; i++)
-               ep93xx_i2s_write_reg(info, base_reg + (i * 4), 1);
+       ep93xx_i2s_write_reg(info, base_reg, 1);
 }
 
 static void ep93xx_i2s_disable(struct ep93xx_i2s_info *info, int stream)
 {
        unsigned base_reg;
-       int i;
 
-       /* Disable fifos */
+       /* Disable fifo */
        if (stream == SNDRV_PCM_STREAM_PLAYBACK)
                base_reg = EP93XX_I2S_TX0EN;
        else
                base_reg = EP93XX_I2S_RX0EN;
-       for (i = 0; i < 3; i++)
-               ep93xx_i2s_write_reg(info, base_reg + (i * 4), 0);
+       ep93xx_i2s_write_reg(info, base_reg, 0);
 
        if ((ep93xx_i2s_read_reg(info, EP93XX_I2S_TX0EN) & 0x1) == 0 &&
            (ep93xx_i2s_read_reg(info, EP93XX_I2S_RX0EN) & 0x1) == 0) {