ASoC: fsl-ssi: make fsl,mode property optional

The simple soundcard binding has its own way for specifying the dai
format. To be able to use this binding we have to make the fsl,mode
property optional. As the property is used in existing devicetrees
keep the option around for compatibility reasons.

Signed-off-by: Markus Pargmann <mpa@pengutronix.de>
Tested-By: Michael Grzeschik <mgr@pengutronix.de>
Signed-off-by: Mark Brown <broonie@linaro.org>

diff --git a/sound/soc/fsl/fsl_ssi.c b/sound/soc/fsl/fsl_ssi.c
index fa650112925a07e1e39af1a919a4f6f528fb1360..18eacf5e399d7e6c22f40374f88ea0623b6c38b2 100644 (file)
--- a/sound/soc/fsl/fsl_ssi.c
+++ b/sound/soc/fsl/fsl_ssi.c
@@ -660,12 +660,9 @@ static int fsl_ssi_hw_params(struct snd_pcm_substream *substream,
        return 0;
 }
 
-/**
- * fsl_ssi_set_dai_fmt - configure Digital Audio Interface Format.
- */
-static int fsl_ssi_set_dai_fmt(struct snd_soc_dai *cpu_dai, unsigned int fmt)
+static int _fsl_ssi_set_dai_fmt(struct fsl_ssi_private *ssi_private,
+               unsigned int fmt)
 {
-       struct fsl_ssi_private *ssi_private = snd_soc_dai_get_drvdata(cpu_dai);
        struct ccsr_ssi __iomem *ssi = ssi_private->ssi;
        u32 strcr = 0, stcr, srcr, scr, mask;
        u8 wm;
@@ -804,6 +801,17 @@ static int fsl_ssi_set_dai_fmt(struct snd_soc_dai *cpu_dai, unsigned int fmt)
                fsl_ssi_setup_ac97(ssi_private);
 
        return 0;
+
+}
+
+/**
+ * fsl_ssi_set_dai_fmt - configure Digital Audio Interface Format.
+ */
+static int fsl_ssi_set_dai_fmt(struct snd_soc_dai *cpu_dai, unsigned int fmt)
+{
+       struct fsl_ssi_private *ssi_private = snd_soc_dai_get_drvdata(cpu_dai);
+
+       return _fsl_ssi_set_dai_fmt(ssi_private, fmt);
 }
 
 /**
@@ -1135,7 +1143,6 @@ static int fsl_ssi_probe(struct platform_device *pdev)
        const uint32_t *iprop;
        struct resource res;
        char name[64];
-       bool ac97 = false;
 
        /* SSIs that are not connected on the board should have a
         *      status = "disabled"
@@ -1148,14 +1155,6 @@ static int fsl_ssi_probe(struct platform_device *pdev)
        if (!of_id || !of_id->data)
                return -EINVAL;
 
-       sprop = of_get_property(np, "fsl,mode", NULL);
-       if (!sprop) {
-               dev_err(&pdev->dev, "fsl,mode property is necessary\n");
-               return -EINVAL;
-       }
-       if (!strcmp(sprop, "ac97-slave"))
-               ac97 = true;
-
        ssi_private = devm_kzalloc(&pdev->dev, sizeof(*ssi_private),
                        GFP_KERNEL);
        if (!ssi_private) {
@@ -1165,10 +1164,19 @@ static int fsl_ssi_probe(struct platform_device *pdev)
 
        ssi_private->soc = of_id->data;
 
+       sprop = of_get_property(np, "fsl,mode", NULL);
+       if (sprop) {
+               if (!strcmp(sprop, "ac97-slave"))
+                       ssi_private->dai_fmt = SND_SOC_DAIFMT_AC97;
+               else if (!strcmp(sprop, "i2s-slave"))
+                       ssi_private->dai_fmt = SND_SOC_DAIFMT_I2S |
+                               SND_SOC_DAIFMT_CBM_CFM;
+       }
+
        ssi_private->use_dma = !of_property_read_bool(np,
                        "fsl,fiq-stream-filter");
 
-       if (ac97) {
+       if (fsl_ssi_is_ac97(ssi_private)) {
                memcpy(&ssi_private->cpu_dai_drv, &fsl_ssi_ac97_dai,
                                sizeof(fsl_ssi_ac97_dai));
 
@@ -1279,6 +1287,9 @@ static int fsl_ssi_probe(struct platform_device *pdev)
        }
 
 done:
+       if (ssi_private->dai_fmt)
+               _fsl_ssi_set_dai_fmt(ssi_private, ssi_private->dai_fmt);
+
        return 0;
 
 error_sound_card: