1 files changed, 26 insertions, 90 deletions

diff --git a/sound/soc/codecs/jz4740.c b/sound/soc/codecs/jz4740.c
index 66557de1e4fe..16253ec9b022 100644
--- a/sound/soc/codecs/jz4740.c
+++ b/sound/soc/codecs/jz4740.c
@@ -74,29 +74,22 @@ static const uint32_t jz4740_codec_regs[] = {
 struct jz4740_codec {
         void __iomem *base;
         struct resource *mem;
-
-        uint32_t reg_cache[2];
-        struct snd_soc_codec codec;
 };
 
-static inline struct jz4740_codec *codec_to_jz4740(struct snd_soc_codec *codec)
-{
-        return container_of(codec, struct jz4740_codec, codec);
-}
-
 static unsigned int jz4740_codec_read(struct snd_soc_codec *codec,
         unsigned int reg)
 {
-        struct jz4740_codec *jz4740_codec = codec_to_jz4740(codec);
+        struct jz4740_codec *jz4740_codec = snd_soc_codec_get_drvdata(codec);
         return readl(jz4740_codec->base + (reg << 2));
 }
 
 static int jz4740_codec_write(struct snd_soc_codec *codec, unsigned int reg,
         unsigned int val)
 {
-        struct jz4740_codec *jz4740_codec = codec_to_jz4740(codec);
+        struct jz4740_codec *jz4740_codec = snd_soc_codec_get_drvdata(codec);
+        u32 *cache = codec->reg_cache;
 
-        jz4740_codec->reg_cache[reg] = val;
+        cache[reg] = val;
         writel(val, jz4740_codec->base + (reg << 2));
 
         return 0;
@@ -172,8 +165,7 @@ static int jz4740_codec_hw_params(struct snd_pcm_substream *substream,
 {
         uint32_t val;
         struct snd_soc_pcm_runtime *rtd = substream->private_data;
-        struct snd_soc_device *socdev = rtd->socdev;
-        struct snd_soc_codec *codec = socdev->card->codec;
+        struct snd_soc_codec *codec =rtd->codec;
 
         switch (params_rate(params)) {
         case 8000:
@@ -219,8 +211,8 @@ static struct snd_soc_dai_ops jz4740_codec_dai_ops = {
         .hw_params = jz4740_codec_hw_params,
 };
 
-struct snd_soc_dai jz4740_codec_dai = {
-        .name = "jz4740",
+static struct snd_soc_dai_driver jz4740_codec_dai = {
+        .name = "jz4740-hifi",
         .playback = {
                 .stream_name = "Playback",
                 .channels_min = 2,
@@ -238,7 +230,6 @@ struct snd_soc_dai jz4740_codec_dai = {
         .ops = &jz4740_codec_dai_ops,
         .symmetric_rates = 1,
 };
-EXPORT_SYMBOL_GPL(jz4740_codec_dai);
 
 static void jz4740_codec_wakeup(struct snd_soc_codec *codec)
 {
@@ -302,23 +293,10 @@ static int jz4740_codec_set_bias_level(struct snd_soc_codec *codec,
         return 0;
 }
 
-static struct snd_soc_codec *jz4740_codec_codec;
-
-static int jz4740_codec_dev_probe(struct platform_device *pdev)
+static int jz4740_codec_dev_probe(struct snd_soc_codec *codec)
 {
-        int ret;
-        struct snd_soc_device *socdev = platform_get_drvdata(pdev);
-        struct snd_soc_codec *codec = jz4740_codec_codec;
-
-        BUG_ON(!codec);
-
-        socdev->card->codec = codec;
-
-        ret = snd_soc_new_pcms(socdev, SNDRV_DEFAULT_IDX1, SNDRV_DEFAULT_STR1);
-        if (ret) {
-                dev_err(&pdev->dev, "Failed to create pcms: %d\n", ret);
-                return ret;
-        }
+        snd_soc_update_bits(codec, JZ4740_REG_CODEC_1,
+                        JZ4740_CODEC_1_SW2_ENABLE, JZ4740_CODEC_1_SW2_ENABLE);
 
         snd_soc_add_controls(codec, jz4740_codec_controls,
                 ARRAY_SIZE(jz4740_codec_controls));
@@ -331,34 +309,27 @@ static int jz4740_codec_dev_probe(struct platform_device *pdev)
 
         snd_soc_dapm_new_widgets(codec);
 
+        jz4740_codec_set_bias_level(codec, SND_SOC_BIAS_STANDBY);
+
         return 0;
 }
 
-static int jz4740_codec_dev_remove(struct platform_device *pdev)
+static int jz4740_codec_dev_remove(struct snd_soc_codec *codec)
 {
-        struct snd_soc_device *socdev = platform_get_drvdata(pdev);
-
-        snd_soc_free_pcms(socdev);
-        snd_soc_dapm_free(socdev);
+        jz4740_codec_set_bias_level(codec, SND_SOC_BIAS_OFF);
 
         return 0;
 }
 
 #ifdef CONFIG_PM_SLEEP
 
-static int jz4740_codec_suspend(struct platform_device *pdev, pm_message_t state)
+static int jz4740_codec_suspend(struct snd_soc_codec *codec, pm_message_t state)
 {
-        struct snd_soc_device *socdev = platform_get_drvdata(pdev);
-        struct snd_soc_codec *codec = socdev->card->codec;
-
         return jz4740_codec_set_bias_level(codec, SND_SOC_BIAS_OFF);
 }
 
-static int jz4740_codec_resume(struct platform_device *pdev)
+static int jz4740_codec_resume(struct snd_soc_codec *codec)
 {
-        struct snd_soc_device *socdev = platform_get_drvdata(pdev);
-        struct snd_soc_codec *codec = socdev->card->codec;
-
         return jz4740_codec_set_bias_level(codec, SND_SOC_BIAS_STANDBY);
 }
 
@@ -367,19 +338,23 @@ static int jz4740_codec_resume(struct platform_device *pdev)
 #define jz4740_codec_resume NULL
 #endif
 
-struct snd_soc_codec_device soc_codec_dev_jz4740_codec = {
+static struct snd_soc_codec_driver soc_codec_dev_jz4740_codec = {
         .probe = jz4740_codec_dev_probe,
         .remove = jz4740_codec_dev_remove,
         .suspend = jz4740_codec_suspend,
         .resume = jz4740_codec_resume,
+        .read = jz4740_codec_read,
+        .write = jz4740_codec_write,
+        .set_bias_level = jz4740_codec_set_bias_level,
+        .reg_cache_default      = jz4740_codec_regs,
+        .reg_word_size = sizeof(u32),
+        .reg_cache_size = 2,
 };
-EXPORT_SYMBOL_GPL(soc_codec_dev_jz4740_codec);
 
 static int __devinit jz4740_codec_probe(struct platform_device *pdev)
 {
         int ret;
         struct jz4740_codec *jz4740_codec;
-        struct snd_soc_codec *codec;
         struct resource *mem;
 
         jz4740_codec = kzalloc(sizeof(*jz4740_codec), GFP_KERNEL);
@@ -408,55 +383,17 @@ static int __devinit jz4740_codec_probe(struct platform_device *pdev)
         }
         jz4740_codec->mem = mem;
 
-        jz4740_codec_dai.dev = &pdev->dev;
-
-        codec = &jz4740_codec->codec;
-
-        codec->dev              = &pdev->dev;
-        codec->name             = "jz4740";
-        codec->owner            = THIS_MODULE;
-
-        codec->read             = jz4740_codec_read;
-        codec->write            = jz4740_codec_write;
-        codec->set_bias_level   = jz4740_codec_set_bias_level;
-        codec->bias_level       = SND_SOC_BIAS_OFF;
-
-        codec->dai              = &jz4740_codec_dai;
-        codec->num_dai          = 1;
-
-        codec->reg_cache        = jz4740_codec->reg_cache;
-        codec->reg_cache_size   = 2;
-        memcpy(codec->reg_cache, jz4740_codec_regs, sizeof(jz4740_codec_regs));
-
-        mutex_init(&codec->mutex);
-        INIT_LIST_HEAD(&codec->dapm_widgets);
-        INIT_LIST_HEAD(&codec->dapm_paths);
-
-        jz4740_codec_codec = codec;
-
-        snd_soc_update_bits(codec, JZ4740_REG_CODEC_1,
-                        JZ4740_CODEC_1_SW2_ENABLE, JZ4740_CODEC_1_SW2_ENABLE);
-
         platform_set_drvdata(pdev, jz4740_codec);
 
-        ret = snd_soc_register_codec(codec);
+        ret = snd_soc_register_codec(&pdev->dev,
+                        &soc_codec_dev_jz4740_codec, &jz4740_codec_dai, 1);
         if (ret) {
                 dev_err(&pdev->dev, "Failed to register codec\n");
                 goto err_iounmap;
         }
 
-        ret = snd_soc_register_dai(&jz4740_codec_dai);
-        if (ret) {
-                dev_err(&pdev->dev, "Failed to register codec dai\n");
-                goto err_unregister_codec;
-        }
-
-        jz4740_codec_set_bias_level(codec, SND_SOC_BIAS_STANDBY);
-
         return 0;
 
-err_unregister_codec:
-        snd_soc_unregister_codec(codec);
 err_iounmap:
         iounmap(jz4740_codec->base);
 err_release_mem_region:
@@ -472,8 +409,7 @@ static int __devexit jz4740_codec_remove(struct platform_device *pdev)
         struct jz4740_codec *jz4740_codec = platform_get_drvdata(pdev);
         struct resource *mem = jz4740_codec->mem;
 
-        snd_soc_unregister_dai(&jz4740_codec_dai);
-        snd_soc_unregister_codec(&jz4740_codec->codec);
+        snd_soc_unregister_codec(&pdev->dev);
 
         iounmap(jz4740_codec->base);
         release_mem_region(mem->start, resource_size(mem));