ASoC: Convert new i.MX SSI driver to use static DAI array

While dynamically allocated DAIs are the way forward the core doesn't
yet support anything except matching with a pointer to the actual DAI
so convert to doing that so that machine drivers don't have to jump
through hoops to register themselves.

Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Acked-by: Javier Martin <javier.martin@vista-silicon.com>

diff --git a/sound/soc/imx/imx-ssi.c b/sound/soc/imx/imx-ssi.c
index c57a11f..ccb7ec9 100644 (file)
--- a/sound/soc/imx/imx-ssi.c
+++ b/sound/soc/imx/imx-ssi.c
@@ -60,7 +60,7 @@
 static int imx_ssi_set_dai_tdm_slot(struct snd_soc_dai *cpu_dai,
        unsigned int tx_mask, unsigned int rx_mask, int slots, int slot_width)
 {
-       struct imx_ssi *ssi = container_of(cpu_dai, struct imx_ssi, dai);
+       struct imx_ssi *ssi = cpu_dai->private_data;
        u32 sccr;
 
        sccr = readl(ssi->base + SSI_STCCR);
@@ -87,7 +87,7 @@ static int imx_ssi_set_dai_tdm_slot(struct snd_soc_dai *cpu_dai,
  */
 static int imx_ssi_set_dai_fmt(struct snd_soc_dai *cpu_dai, unsigned int fmt)
 {
-       struct imx_ssi *ssi = container_of(cpu_dai, struct imx_ssi, dai);
+       struct imx_ssi *ssi = cpu_dai->private_data;
        u32 strcr = 0, scr;
 
        scr = readl(ssi->base + SSI_SCR) & ~(SSI_SCR_SYN | SSI_SCR_NET);
@@ -160,7 +160,7 @@ static int imx_ssi_set_dai_fmt(struct snd_soc_dai *cpu_dai, unsigned int fmt)
 static int imx_ssi_set_dai_sysclk(struct snd_soc_dai *cpu_dai,
                                  int clk_id, unsigned int freq, int dir)
 {
-       struct imx_ssi *ssi = container_of(cpu_dai, struct imx_ssi, dai);
+       struct imx_ssi *ssi = cpu_dai->private_data;
        u32 scr;
 
        scr = readl(ssi->base + SSI_SCR);
@@ -188,7 +188,7 @@ static int imx_ssi_set_dai_sysclk(struct snd_soc_dai *cpu_dai,
 static int imx_ssi_set_dai_clkdiv(struct snd_soc_dai *cpu_dai,
                                  int div_id, int div)
 {
-       struct imx_ssi *ssi = container_of(cpu_dai, struct imx_ssi, dai);
+       struct imx_ssi *ssi = cpu_dai->private_data;
        u32 stccr, srccr;
 
        stccr = readl(ssi->base + SSI_STCCR);
@@ -237,7 +237,7 @@ static int imx_ssi_hw_params(struct snd_pcm_substream *substream,
                             struct snd_pcm_hw_params *params,
                             struct snd_soc_dai *cpu_dai)
 {
-       struct imx_ssi *ssi = container_of(cpu_dai, struct imx_ssi, dai);
+       struct imx_ssi *ssi = cpu_dai->private_data;
        u32 reg, sccr;
 
        /* Tx/Rx config */
@@ -274,7 +274,7 @@ static int imx_ssi_trigger(struct snd_pcm_substream *substream, int cmd,
 {
        struct snd_soc_pcm_runtime *rtd = substream->private_data;
        struct snd_soc_dai *cpu_dai = rtd->dai->cpu_dai;
-       struct imx_ssi *ssi = container_of(cpu_dai, struct imx_ssi, dai);
+       struct imx_ssi *ssi = cpu_dai->private_data;
        unsigned int sier_bits, sier;
        unsigned int scr;
 
@@ -570,7 +570,7 @@ struct snd_ac97_bus_ops soc_ac97_ops = {
 };
 EXPORT_SYMBOL_GPL(soc_ac97_ops);
 
-struct snd_soc_dai *imx_ssi_pcm_dai[2];
+struct snd_soc_dai imx_ssi_pcm_dai[2];
 EXPORT_SYMBOL_GPL(imx_ssi_pcm_dai);
 
 static int imx_ssi_probe(struct platform_device *pdev)
@@ -581,6 +581,10 @@ static int imx_ssi_probe(struct platform_device *pdev)
        struct snd_soc_platform *platform;
        int ret = 0;
        unsigned int val;
+       struct snd_soc_dai *dai = &imx_ssi_pcm_dai[pdev->id];
+
+       if (dai->id >= ARRAY_SIZE(imx_ssi_pcm_dai))
+               return -EINVAL;
 
        ssi = kzalloc(sizeof(*ssi), GFP_KERNEL);
        if (!ssi)
@@ -592,8 +596,6 @@ static int imx_ssi_probe(struct platform_device *pdev)
                ssi->flags = pdata->flags;
        }
 
-       imx_ssi_pcm_dai[pdev->id] = &ssi->dai;
-
        ssi->irq = platform_get_irq(pdev, 0);
 
        ssi->clk = clk_get(&pdev->dev, NULL);
@@ -631,13 +633,9 @@ static int imx_ssi_probe(struct platform_device *pdev)
                }
                ac97_ssi = ssi;
                setup_channel_to_ac97(ssi);
-               memcpy(&ssi->dai, &imx_ac97_dai, sizeof(imx_ac97_dai));
+               memcpy(dai, &imx_ac97_dai, sizeof(imx_ac97_dai));
        } else
-               memcpy(&ssi->dai, &imx_ssi_dai, sizeof(imx_ssi_dai));
-
-       ssi->dai.id = pdev->id;
-       ssi->dai.dev = &pdev->dev;
-       ssi->dai.name = kasprintf(GFP_KERNEL, "imx-ssi.%d", pdev->id);
+               memcpy(dai, &imx_ssi_dai, sizeof(imx_ssi_dai));
 
        writel(0x0, ssi->base + SSI_SIER);
 
@@ -652,9 +650,10 @@ static int imx_ssi_probe(struct platform_device *pdev)
        if (res)
                ssi->dma_params_rx.dma = res->start;
 
-       ssi->dai.id = pdev->id;
-       ssi->dai.dev = &pdev->dev;
-       ssi->dai.name = kasprintf(GFP_KERNEL, "imx-ssi.%d", pdev->id);
+       dai->id = pdev->id;
+       dai->dev = &pdev->dev;
+       dai->name = kasprintf(GFP_KERNEL, "imx-ssi.%d", pdev->id);
+       dai->private_data = ssi;
 
        if ((cpu_is_mx27() || cpu_is_mx21()) &&
                        !(ssi->flags & IMX_SSI_USE_AC97)) {
@@ -671,7 +670,7 @@ static int imx_ssi_probe(struct platform_device *pdev)
                SSI_SFCSR_RFWM0(ssi->dma_params_rx.burstsize);
        writel(val, ssi->base + SSI_SFCSR);
 
-       ret = snd_soc_register_dai(&ssi->dai);
+       ret = snd_soc_register_dai(dai);
        if (ret) {
                dev_err(&pdev->dev, "register DAI failed\n");
                goto failed_register;
@@ -699,8 +698,9 @@ static int __devexit imx_ssi_remove(struct platform_device *pdev)
 {
        struct resource *res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
        struct imx_ssi *ssi = platform_get_drvdata(pdev);
+       struct snd_soc_dai *dai = &imx_ssi_pcm_dai[pdev->id];
 
-       snd_soc_unregister_dai(&ssi->dai);
+       snd_soc_unregister_dai(dai);
 
        if (ssi->flags & IMX_SSI_USE_AC97)
                ac97_ssi = NULL;

diff --git a/sound/soc/imx/imx-ssi.h b/sound/soc/imx/imx-ssi.h
index cb2c81f..55f26eb 100644 (file)
--- a/sound/soc/imx/imx-ssi.h
+++ b/sound/soc/imx/imx-ssi.h
@@ -183,7 +183,7 @@
 #define IMX_SSI_RX_DIV_PSR     4
 #define IMX_SSI_RX_DIV_PM      5
 
-extern struct snd_soc_dai *imx_ssi_pcm_dai[2];
+extern struct snd_soc_dai imx_ssi_pcm_dai[2];
 extern struct snd_soc_platform imx_soc_platform;
 
 #define DRV_NAME "imx-ssi"
@@ -195,7 +195,6 @@ struct imx_pcm_dma_params {
 };
 
 struct imx_ssi {
-       struct snd_soc_dai dai;
        struct platform_device *ac97_dev;
 
        struct snd_soc_device imx_ac97;