ASoC: simple-card: Fix breakage on kontron-sl28-var3-ads2
[linux-2.6-microblaze.git] / sound / soc / generic / simple-card.c
index 6a04632..a1373be 100644 (file)
@@ -93,11 +93,12 @@ static void simple_parse_convert(struct device *dev,
 }
 
 static void simple_parse_mclk_fs(struct device_node *top,
-                                struct device_node *np,
+                                struct device_node *cpu,
+                                struct device_node *codec,
                                 struct simple_dai_props *props,
                                 char *prefix)
 {
-       struct device_node *node = of_get_parent(np);
+       struct device_node *node = of_get_parent(cpu);
        char prop[128];
 
        snprintf(prop, sizeof(prop), "%smclk-fs", PREFIX);
@@ -105,79 +106,12 @@ static void simple_parse_mclk_fs(struct device_node *top,
 
        snprintf(prop, sizeof(prop), "%smclk-fs", prefix);
        of_property_read_u32(node,      prop, &props->mclk_fs);
-       of_property_read_u32(np,        prop, &props->mclk_fs);
+       of_property_read_u32(cpu,       prop, &props->mclk_fs);
+       of_property_read_u32(codec,     prop, &props->mclk_fs);
 
        of_node_put(node);
 }
 
-static int simple_parse_node(struct asoc_simple_priv *priv,
-                            struct device_node *np,
-                            struct link_info *li,
-                            char *prefix,
-                            int is_cpu)
-{
-       struct device *dev = simple_priv_to_dev(priv);
-       struct device_node *top = dev->of_node;
-       struct snd_soc_dai_link *dai_link = simple_priv_to_link(priv, li->link);
-       struct simple_dai_props *dai_props = simple_priv_to_props(priv, li->link);
-       struct snd_soc_dai_link_component *dlc;
-       struct asoc_simple_dai *dai;
-       int ret, single = 0;
-
-       if (is_cpu) {
-               dlc = asoc_link_to_cpu(dai_link, 0);
-               dai = simple_props_to_dai_cpu(dai_props, 0);
-       } else {
-               dlc = asoc_link_to_codec(dai_link, 0);
-               dai = simple_props_to_dai_codec(dai_props, 0);
-       }
-
-       simple_parse_mclk_fs(top, np, dai_props, prefix);
-
-       ret = asoc_simple_parse_dai(np, dlc, &single);
-       if (ret)
-               return ret;
-
-       ret = asoc_simple_parse_clk(dev, np, dai, dlc);
-       if (ret)
-               return ret;
-
-       ret = asoc_simple_parse_tdm(np, dai);
-       if (ret)
-               return ret;
-
-       if (is_cpu) {
-               struct snd_soc_dai_link_component *platforms = asoc_link_to_platform(dai_link, 0);
-
-               asoc_simple_canonicalize_cpu(dlc, single);
-               asoc_simple_canonicalize_platform(platforms, dlc);
-       }
-
-       return 0;
-}
-
-static int simple_link_init(struct asoc_simple_priv *priv,
-                           struct device_node *node,
-                           struct device_node *codec,
-                           struct link_info *li,
-                           char *prefix,
-                           char *name)
-{
-       struct device *dev = simple_priv_to_dev(priv);
-       struct snd_soc_dai_link *dai_link = simple_priv_to_link(priv, li->link);
-       int ret;
-
-       ret = asoc_simple_parse_daifmt(dev, node, codec,
-                                      prefix, &dai_link->dai_fmt);
-       if (ret < 0)
-               return 0;
-
-       dai_link->init                  = asoc_simple_dai_init;
-       dai_link->ops                   = &simple_ops;
-
-       return asoc_simple_set_dailink_name(dev, dai_link, name);
-}
-
 static int simple_dai_link_of_dpcm(struct asoc_simple_priv *priv,
                                   struct device_node *np,
                                   struct device_node *codec,
@@ -187,32 +121,50 @@ static int simple_dai_link_of_dpcm(struct asoc_simple_priv *priv,
        struct device *dev = simple_priv_to_dev(priv);
        struct snd_soc_dai_link *dai_link = simple_priv_to_link(priv, li->link);
        struct simple_dai_props *dai_props = simple_priv_to_props(priv, li->link);
+       struct asoc_simple_dai *dai;
        struct snd_soc_dai_link_component *cpus = asoc_link_to_cpu(dai_link, 0);
        struct snd_soc_dai_link_component *codecs = asoc_link_to_codec(dai_link, 0);
+       struct snd_soc_dai_link_component *platforms = asoc_link_to_platform(dai_link, 0);
        struct device_node *top = dev->of_node;
        struct device_node *node = of_get_parent(np);
        char *prefix = "";
-       char dai_name[64];
        int ret;
 
        dev_dbg(dev, "link_of DPCM (%pOF)\n", np);
 
+       li->link++;
+
        /* For single DAI link & old style of DT node */
        if (is_top)
                prefix = PREFIX;
 
        if (li->cpu) {
+               int is_single_links = 0;
+
                /* Codec is dummy */
 
                /* FE settings */
                dai_link->dynamic               = 1;
                dai_link->dpcm_merged_format    = 1;
 
-               ret = simple_parse_node(priv, np, li, prefix, 1);
+               dai = simple_props_to_dai_cpu(dai_props, 0);
+
+               ret = asoc_simple_parse_dai(np, cpus, &is_single_links);
+               if (ret)
+                       goto out_put_node;
+
+               ret = asoc_simple_parse_clk(dev, np, dai, cpus);
                if (ret < 0)
                        goto out_put_node;
 
-               snprintf(dai_name, sizeof(dai_name), "fe.%s", cpus->dai_name);
+               ret = asoc_simple_set_dailink_name(dev, dai_link,
+                                                  "fe.%s",
+                                                  cpus->dai_name);
+               if (ret < 0)
+                       goto out_put_node;
+
+               asoc_simple_canonicalize_cpu(cpus, is_single_links);
+               asoc_simple_canonicalize_platform(platforms, cpus);
        } else {
                struct snd_soc_codec_conf *cconf;
 
@@ -222,13 +174,22 @@ static int simple_dai_link_of_dpcm(struct asoc_simple_priv *priv,
                dai_link->no_pcm                = 1;
                dai_link->be_hw_params_fixup    = asoc_simple_be_hw_params_fixup;
 
+               dai     = simple_props_to_dai_codec(dai_props, 0);
                cconf   = simple_props_to_codec_conf(dai_props, 0);
 
-               ret = simple_parse_node(priv, np, li, prefix, 0);
+               ret = asoc_simple_parse_dai(np, codecs, NULL);
+               if (ret < 0)
+                       goto out_put_node;
+
+               ret = asoc_simple_parse_clk(dev, np, dai, codecs);
                if (ret < 0)
                        goto out_put_node;
 
-               snprintf(dai_name, sizeof(dai_name), "be.%s", codecs->dai_name);
+               ret = asoc_simple_set_dailink_name(dev, dai_link,
+                                                  "be.%s",
+                                                  codecs->dai_name);
+               if (ret < 0)
+                       goto out_put_node;
 
                /* check "prefix" from top node */
                snd_soc_of_parse_node_prefix(top, cconf, codecs->of_node,
@@ -240,14 +201,23 @@ static int simple_dai_link_of_dpcm(struct asoc_simple_priv *priv,
        }
 
        simple_parse_convert(dev, np, &dai_props->adata);
+       simple_parse_mclk_fs(top, np, codec, dai_props, prefix);
+
+       ret = asoc_simple_parse_tdm(np, dai);
+       if (ret)
+               goto out_put_node;
+
+       ret = asoc_simple_parse_daifmt(dev, node, codec,
+                                      prefix, &dai_link->dai_fmt);
+       if (ret < 0)
+               goto out_put_node;
 
        snd_soc_dai_link_set_capabilities(dai_link);
 
-       ret = simple_link_init(priv, node, codec, li, prefix, dai_name);
+       dai_link->ops                   = &simple_ops;
+       dai_link->init                  = asoc_simple_dai_init;
 
 out_put_node:
-       li->link++;
-
        of_node_put(node);
        return ret;
 }
@@ -260,19 +230,23 @@ static int simple_dai_link_of(struct asoc_simple_priv *priv,
 {
        struct device *dev = simple_priv_to_dev(priv);
        struct snd_soc_dai_link *dai_link = simple_priv_to_link(priv, li->link);
+       struct simple_dai_props *dai_props = simple_priv_to_props(priv, li->link);
+       struct asoc_simple_dai *cpu_dai = simple_props_to_dai_cpu(dai_props, 0);
+       struct asoc_simple_dai *codec_dai = simple_props_to_dai_codec(dai_props, 0);
        struct snd_soc_dai_link_component *cpus = asoc_link_to_cpu(dai_link, 0);
        struct snd_soc_dai_link_component *codecs = asoc_link_to_codec(dai_link, 0);
        struct snd_soc_dai_link_component *platforms = asoc_link_to_platform(dai_link, 0);
+       struct device_node *top = dev->of_node;
        struct device_node *cpu = NULL;
        struct device_node *node = NULL;
        struct device_node *plat = NULL;
-       char dai_name[64];
        char prop[128];
        char *prefix = "";
-       int ret;
+       int ret, single_cpu = 0;
 
        cpu  = np;
        node = of_get_parent(np);
+       li->link++;
 
        dev_dbg(dev, "link_of (%pOF)\n", node);
 
@@ -283,11 +257,18 @@ static int simple_dai_link_of(struct asoc_simple_priv *priv,
        snprintf(prop, sizeof(prop), "%splat", prefix);
        plat = of_get_child_by_name(node, prop);
 
-       ret = simple_parse_node(priv, cpu, li, prefix, 1);
+       ret = asoc_simple_parse_daifmt(dev, node, codec,
+                                      prefix, &dai_link->dai_fmt);
        if (ret < 0)
                goto dai_link_of_err;
 
-       ret = simple_parse_node(priv, codec, li, prefix, 0);
+       simple_parse_mclk_fs(top, cpu, codec, dai_props, prefix);
+
+       ret = asoc_simple_parse_dai(cpu, cpus, &single_cpu);
+       if (ret < 0)
+               goto dai_link_of_err;
+
+       ret = asoc_simple_parse_dai(codec, codecs, NULL);
        if (ret < 0)
                goto dai_link_of_err;
 
@@ -295,18 +276,39 @@ static int simple_dai_link_of(struct asoc_simple_priv *priv,
        if (ret < 0)
                goto dai_link_of_err;
 
-       snprintf(dai_name, sizeof(dai_name),
-                "%s-%s", cpus->dai_name, codecs->dai_name);
-       ret = simple_link_init(priv, node, codec, li, prefix, dai_name);
+       ret = asoc_simple_parse_tdm(cpu, cpu_dai);
+       if (ret < 0)
+               goto dai_link_of_err;
+
+       ret = asoc_simple_parse_tdm(codec, codec_dai);
        if (ret < 0)
                goto dai_link_of_err;
 
+       ret = asoc_simple_parse_clk(dev, cpu, cpu_dai, cpus);
+       if (ret < 0)
+               goto dai_link_of_err;
+
+       ret = asoc_simple_parse_clk(dev, codec, codec_dai, codecs);
+       if (ret < 0)
+               goto dai_link_of_err;
+
+       ret = asoc_simple_set_dailink_name(dev, dai_link,
+                                          "%s-%s",
+                                          cpus->dai_name,
+                                          codecs->dai_name);
+       if (ret < 0)
+               goto dai_link_of_err;
+
+       dai_link->ops = &simple_ops;
+       dai_link->init = asoc_simple_dai_init;
+
+       asoc_simple_canonicalize_cpu(cpus, single_cpu);
+       asoc_simple_canonicalize_platform(platforms, cpus);
+
 dai_link_of_err:
        of_node_put(plat);
        of_node_put(node);
 
-       li->link++;
-
        return ret;
 }