ASoC: omap-mcbsp: Single function CLKR/FSR source mux configuration

Use single function for the CLKR/FSR mux configuration.
OMAP2/3 has 6 pin configuration on McBSP1 instance, while on OMAP4
McBSP4 instance have the 6 pin configuration so the omap2_mcbsp1_mux_* is
not correct name for all support OMAP versions

Signed-off-by: Peter Ujfalusi <peter.ujfalusi@ti.com>
Acked-by: Jarkko Nikula <jarkko.nikula@bitmer.com>
Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Signed-off-by: Liam Girdwood <lrg@ti.com>

3 files changed, 32 insertions, 39 deletions

diff --git a/sound/soc/omap/mcbsp.c b/sound/soc/omap/mcbsp.c
index c3e31deafa07..95413a168087 100644
--- a/sound/soc/omap/mcbsp.c
+++ b/sound/soc/omap/mcbsp.c
@@ -687,40 +687,36 @@ int omap2_mcbsp_set_clks_src(struct omap_mcbsp *mcbsp, u8 fck_src_id)
                 return -EINVAL;
 }
 
-void omap2_mcbsp1_mux_clkr_src(struct omap_mcbsp *mcbsp, u8 mux)
+int omap_mcbsp_6pin_src_mux(struct omap_mcbsp *mcbsp, u8 mux)
 {
-        const char *src;
-
-        if (mcbsp->id != 1)
-                return;
+        const char *signal, *src;
+        int ret = 0;
 
-        if (mux == CLKR_SRC_CLKR)
+        switch (mux) {
+        case CLKR_SRC_CLKR:
+                signal = "clkr";
                 src = "clkr";
-        else if (mux == CLKR_SRC_CLKX)
+                break;
+        case CLKR_SRC_CLKX:
+                signal = "clkr";
                 src = "clkx";
-        else
-                return;
-
-        if (mcbsp->pdata->mux_signal)
-                mcbsp->pdata->mux_signal(mcbsp->dev, "clkr", src);
-}
-
-void omap2_mcbsp1_mux_fsr_src(struct omap_mcbsp *mcbsp, u8 mux)
-{
-        const char *src;
-
-        if (mcbsp->id != 1)
-                return;
-
-        if (mux == FSR_SRC_FSR)
+                break;
+        case FSR_SRC_FSR:
+                signal = "fsr";
                 src = "fsr";
-        else if (mux == FSR_SRC_FSX)
+                break;
+        case FSR_SRC_FSX:
+                signal = "fsr";
                 src = "fsx";
-        else
-                return;
+                break;
+        default:
+                return -EINVAL;
+        }
 
         if (mcbsp->pdata->mux_signal)
-                mcbsp->pdata->mux_signal(mcbsp->dev, "fsr", src);
+                ret = mcbsp->pdata->mux_signal(mcbsp->dev, signal, src);
+
+        return ret;
 }
 
 #define max_thres(m)                    (mcbsp->pdata->buffer_size)
diff --git a/sound/soc/omap/mcbsp.h b/sound/soc/omap/mcbsp.h
index acc94700f5b7..a944fcc9073c 100644
--- a/sound/soc/omap/mcbsp.h
+++ b/sound/soc/omap/mcbsp.h
@@ -230,13 +230,11 @@ enum {
 #define XRDYEN                  BIT(10)
 #define XEMPTYEOFEN             BIT(14)
 
-/* CLKR signal muxing options */
-#define CLKR_SRC_CLKR           0
-#define CLKR_SRC_CLKX           1
-
-/* FSR signal muxing options */
-#define FSR_SRC_FSR             0
-#define FSR_SRC_FSX             1
+/* Clock signal muxing options */
+#define CLKR_SRC_CLKR           0 /* CLKR signal is from the CLKR pin */
+#define CLKR_SRC_CLKX           1 /* CLKR signal is from the CLKX pin */
+#define FSR_SRC_FSR             2 /* FSR signal is from the FSR pin */
+#define FSR_SRC_FSX             3 /* FSR signal is from the FSX pin */
 
 /* McBSP functional clock sources */
 #define MCBSP_CLKS_PRCM_SRC     0
@@ -333,8 +331,7 @@ void omap_mcbsp_stop(struct omap_mcbsp *mcbsp, int tx, int rx);
 int omap2_mcbsp_set_clks_src(struct omap_mcbsp *mcbsp, u8 fck_src_id);
 
 /* McBSP signal muxing API */
-void omap2_mcbsp1_mux_clkr_src(struct omap_mcbsp *mcbsp, u8 mux);
-void omap2_mcbsp1_mux_fsr_src(struct omap_mcbsp *mcbsp, u8 mux);
+int omap_mcbsp_6pin_src_mux(struct omap_mcbsp *mcbsp, u8 mux);
 
 /* Sidetone specific API */
 int omap_st_set_chgain(struct omap_mcbsp *mcbsp, int channel, s16 chgain);
diff --git a/sound/soc/omap/omap-mcbsp.c b/sound/soc/omap/omap-mcbsp.c
index 10eb645ceeef..d8409b008431 100644
--- a/sound/soc/omap/omap-mcbsp.c
+++ b/sound/soc/omap/omap-mcbsp.c
@@ -554,22 +554,22 @@ static int omap_mcbsp_dai_set_dai_sysclk(struct snd_soc_dai *cpu_dai,
         case OMAP_MCBSP_CLKR_SRC_CLKR:
                 if (cpu_class_is_omap1())
                         break;
-                omap2_mcbsp1_mux_clkr_src(mcbsp, CLKR_SRC_CLKR);
+                err = omap_mcbsp_6pin_src_mux(mcbsp, CLKR_SRC_CLKR);
                 break;
         case OMAP_MCBSP_CLKR_SRC_CLKX:
                 if (cpu_class_is_omap1())
                         break;
-                omap2_mcbsp1_mux_clkr_src(mcbsp, CLKR_SRC_CLKX);
+                err = omap_mcbsp_6pin_src_mux(mcbsp, CLKR_SRC_CLKX);
                 break;
         case OMAP_MCBSP_FSR_SRC_FSR:
                 if (cpu_class_is_omap1())
                         break;
-                omap2_mcbsp1_mux_fsr_src(mcbsp, FSR_SRC_FSR);
+                err = omap_mcbsp_6pin_src_mux(mcbsp, FSR_SRC_FSR);
                 break;
         case OMAP_MCBSP_FSR_SRC_FSX:
                 if (cpu_class_is_omap1())
                         break;
-                omap2_mcbsp1_mux_fsr_src(mcbsp, FSR_SRC_FSX);
+                err = omap_mcbsp_6pin_src_mux(mcbsp, FSR_SRC_FSX);
                 break;
         default:
                 err = -ENODEV;