ASoC: mediatek: Add MTK_STREAM_NUM to mtk-base-afe.h

Add MTK_STREAM_NUM to common header and modify related stuff so that
the other SoCs can reuse it.

Signed-off-by: Ryder Lee <ryder.lee@mediatek.com>
Reviewed-by: Garlic Tseng <garlic.tseng@mediatek.com>
Signed-off-by: Mark Brown <broonie@kernel.org>

diff --git a/sound/soc/mediatek/common/mtk-base-afe.h b/sound/soc/mediatek/common/mtk-base-afe.h
index 3a78f6f171958f911e43f73db934c76d6a5449e6..e50adc9116cf107eee81df8b5cf4e3dce19ee5b8 100644 (file)
--- a/sound/soc/mediatek/common/mtk-base-afe.h
+++ b/sound/soc/mediatek/common/mtk-base-afe.h
@@ -17,6 +17,8 @@
 #ifndef _MTK_BASE_AFE_H_
 #define _MTK_BASE_AFE_H_
 
+#define MTK_STREAM_NUM (SNDRV_PCM_STREAM_LAST + 1)
+
 struct mtk_base_memif_data {
        int id;
        const char *name;

diff --git a/sound/soc/mediatek/mt2701/mt2701-afe-clock-ctrl.c b/sound/soc/mediatek/mt2701/mt2701-afe-clock-ctrl.c
index 565005f821d03c6559aaf99f0accd9bde4518b6e..d4e6a5ea63f4ef8a73e2a6e21f1a62a5272d002e 100644 (file)
--- a/sound/soc/mediatek/mt2701/mt2701-afe-clock-ctrl.c
+++ b/sound/soc/mediatek/mt2701/mt2701-afe-clock-ctrl.c
@@ -46,6 +46,7 @@ int mt2701_init_clock(struct mtk_base_afe *afe)
        /* Get I2S related clocks */
        for (i = 0; i < MT2701_I2S_NUM; i++) {
                struct mt2701_i2s_path *i2s_path = &afe_priv->i2s_path[i];
+               struct clk *i2s_ck;
                char name[13];
 
                snprintf(name, sizeof(name), "i2s%d_src_sel", i);
@@ -70,18 +71,20 @@ int mt2701_init_clock(struct mtk_base_afe *afe)
                }
 
                snprintf(name, sizeof(name), "i2so%d_hop_ck", i);
-               i2s_path->hop_ck[I2S_OUT] = devm_clk_get(afe->dev, name);
-               if (IS_ERR(i2s_path->hop_ck[I2S_OUT])) {
+               i2s_ck = devm_clk_get(afe->dev, name);
+               if (IS_ERR(i2s_ck)) {
                        dev_err(afe->dev, "failed to get %s\n", name);
-                       return PTR_ERR(i2s_path->hop_ck[I2S_OUT]);
+                       return PTR_ERR(i2s_ck);
                }
+               i2s_path->hop_ck[SNDRV_PCM_STREAM_PLAYBACK] = i2s_ck;
 
                snprintf(name, sizeof(name), "i2si%d_hop_ck", i);
-               i2s_path->hop_ck[I2S_IN] = devm_clk_get(afe->dev, name);
-               if (IS_ERR(i2s_path->hop_ck[I2S_IN])) {
+               i2s_ck = devm_clk_get(afe->dev, name);
+               if (IS_ERR(i2s_ck)) {
                        dev_err(afe->dev, "failed to get %s\n", name);
-                       return PTR_ERR(i2s_path->hop_ck[I2S_IN]);
+                       return PTR_ERR(i2s_ck);
                }
+               i2s_path->hop_ck[SNDRV_PCM_STREAM_CAPTURE] = i2s_ck;
 
                snprintf(name, sizeof(name), "asrc%d_out_ck", i);
                i2s_path->asrco_ck = devm_clk_get(afe->dev, name);

diff --git a/sound/soc/mediatek/mt2701/mt2701-afe-common.h b/sound/soc/mediatek/mt2701/mt2701-afe-common.h
index ae8ddeacfbfe25584c2e9c6a2e6133a959f754bf..8dabf1913533295d1497b41a03bcfe3b86cd6480 100644 (file)
--- a/sound/soc/mediatek/mt2701/mt2701-afe-common.h
+++ b/sound/soc/mediatek/mt2701/mt2701-afe-common.h
@@ -23,7 +23,6 @@
 #include "mt2701-reg.h"
 #include "../common/mtk-base-afe.h"
 
-#define MT2701_STREAM_DIR_NUM (SNDRV_PCM_STREAM_LAST + 1)
 #define MT2701_PLL_DOMAIN_0_RATE       98304000
 #define MT2701_PLL_DOMAIN_1_RATE       90316800
 #define MT2701_I2S_NUM 4
@@ -100,19 +99,13 @@ struct mt2701_i2s_data {
        int i2s_asrc_fs_mask;
 };
 
-enum mt2701_i2s_dir {
-       I2S_OUT,
-       I2S_IN,
-       I2S_DIR_NUM,
-};
-
 struct mt2701_i2s_path {
        int dai_id;
        int mclk_rate;
-       int on[I2S_DIR_NUM];
-       int occupied[I2S_DIR_NUM];
-       const struct mt2701_i2s_data *i2s_data[I2S_DIR_NUM];
-       struct clk *hop_ck[I2S_DIR_NUM];
+       int on[MTK_STREAM_NUM];
+       int occupied[MTK_STREAM_NUM];
+       const struct mt2701_i2s_data *i2s_data[MTK_STREAM_NUM];
+       struct clk *hop_ck[MTK_STREAM_NUM];
        struct clk *sel_ck;
        struct clk *div_ck;
        struct clk *mclk_ck;
@@ -123,7 +116,7 @@ struct mt2701_afe_private {
        struct mt2701_i2s_path i2s_path[MT2701_I2S_NUM];
        struct clk *base_ck[MT2701_BASE_CLK_NUM];
        struct clk *mrgif_ck;
-       bool mrg_enable[MT2701_STREAM_DIR_NUM];
+       bool mrg_enable[MTK_STREAM_NUM];
 };
 
 #endif

diff --git a/sound/soc/mediatek/mt2701/mt2701-afe-pcm.c b/sound/soc/mediatek/mt2701/mt2701-afe-pcm.c
index 2a161f4d01f6f33b16b05ff11dca642361060079..99094a574213b3d4edabdba0d52f71dbd9392648 100644 (file)
--- a/sound/soc/mediatek/mt2701/mt2701-afe-pcm.c
+++ b/sound/soc/mediatek/mt2701/mt2701-afe-pcm.c
@@ -1256,63 +1256,24 @@ static const struct mtk_base_irq_data irq_data[MT2701_IRQ_ASYS_END] = {
        }
 };
 
-static const struct mt2701_i2s_data mt2701_i2s_data[MT2701_I2S_NUM][2] = {
+static const struct mt2701_i2s_data mt2701_i2s_data[][2] = {
        {
-               {
-                       .i2s_ctrl_reg = ASYS_I2SO1_CON,
-                       .i2s_asrc_fs_shift = 0,
-                       .i2s_asrc_fs_mask = 0x1f,
-
-               },
-               {
-                       .i2s_ctrl_reg = ASYS_I2SIN1_CON,
-                       .i2s_asrc_fs_shift = 0,
-                       .i2s_asrc_fs_mask = 0x1f,
-
-               },
+               { ASYS_I2SO1_CON, 0, 0x1f },
+               { ASYS_I2SIN1_CON, 0, 0x1f },
        },
        {
-               {
-                       .i2s_ctrl_reg = ASYS_I2SO2_CON,
-                       .i2s_asrc_fs_shift = 5,
-                       .i2s_asrc_fs_mask = 0x1f,
-
-               },
-               {
-                       .i2s_ctrl_reg = ASYS_I2SIN2_CON,
-                       .i2s_asrc_fs_shift = 5,
-                       .i2s_asrc_fs_mask = 0x1f,
-
-               },
+               { ASYS_I2SO2_CON, 5, 0x1f },
+               { ASYS_I2SIN2_CON, 5, 0x1f },
        },
        {
-               {
-                       .i2s_ctrl_reg = ASYS_I2SO3_CON,
-                       .i2s_asrc_fs_shift = 10,
-                       .i2s_asrc_fs_mask = 0x1f,
-
-               },
-               {
-                       .i2s_ctrl_reg = ASYS_I2SIN3_CON,
-                       .i2s_asrc_fs_shift = 10,
-                       .i2s_asrc_fs_mask = 0x1f,
-
-               },
+               { ASYS_I2SO3_CON, 10, 0x1f },
+               { ASYS_I2SIN3_CON, 10, 0x1f },
        },
        {
-               {
-                       .i2s_ctrl_reg = ASYS_I2SO4_CON,
-                       .i2s_asrc_fs_shift = 15,
-                       .i2s_asrc_fs_mask = 0x1f,
-
-               },
-               {
-                       .i2s_ctrl_reg = ASYS_I2SIN4_CON,
-                       .i2s_asrc_fs_shift = 15,
-                       .i2s_asrc_fs_mask = 0x1f,
-
-               },
+               { ASYS_I2SO4_CON, 15, 0x1f },
+               { ASYS_I2SIN4_CON, 15, 0x1f },
        },
+       /* TODO - extend control registers supported by newer SoCs */
 };
 
 static irqreturn_t mt2701_asys_isr(int irq_id, void *dev)
@@ -1434,10 +1395,10 @@ static int mt2701_afe_pcm_dev_probe(struct platform_device *pdev)
 
        /* I2S initialize */
        for (i = 0; i < MT2701_I2S_NUM; i++) {
-               afe_priv->i2s_path[i].i2s_data[I2S_OUT]
-                       = &mt2701_i2s_data[i][I2S_OUT];
-               afe_priv->i2s_path[i].i2s_data[I2S_IN]
-                       = &mt2701_i2s_data[i][I2S_IN];
+               afe_priv->i2s_path[i].i2s_data[SNDRV_PCM_STREAM_PLAYBACK] =
+                       &mt2701_i2s_data[i][SNDRV_PCM_STREAM_PLAYBACK];
+               afe_priv->i2s_path[i].i2s_data[SNDRV_PCM_STREAM_CAPTURE] =
+                       &mt2701_i2s_data[i][SNDRV_PCM_STREAM_CAPTURE];
        }
 
        afe->mtk_afe_hardware = &mt2701_afe_hardware;