ASoC: Fix memory allocation for snd_soc_dapm_switch names
authorMark Brown <broonie@opensource.wolfsonmicro.com>
Thu, 5 Mar 2009 17:06:23 +0000 (17:06 +0000)
committerMark Brown <broonie@opensource.wolfsonmicro.com>
Fri, 6 Mar 2009 13:37:14 +0000 (13:37 +0000)
snd_soc_dapm_switch ends up ends up in dapm_new_mixer() (since a switch
is a special case of a mixer with only one input) but this wasn't
correctly handled in the code.

Also fix the coding style for the switch below while we're here.

Reported-by: Joonyoung Shim <dofmind@gmail.com>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
sound/soc/soc-dapm.c

index 7da6d0db40f24d2e3cd58c83494d1d1342ce62df..735903a7467500f4d93d750e527a693d67eabaa2 100644 (file)
@@ -332,7 +332,7 @@ static int dapm_new_mixer(struct snd_soc_codec *codec,
                         * kcontrol name.
                         */
                        name_len = strlen(w->kcontrols[i].name) + 1;
-                       if (w->id == snd_soc_dapm_mixer)
+                       if (w->id != snd_soc_dapm_mixer_named_ctl)
                                name_len += 1 + strlen(w->name);
 
                        path->long_name = kmalloc(name_len, GFP_KERNEL);
@@ -341,15 +341,14 @@ static int dapm_new_mixer(struct snd_soc_codec *codec,
                                return -ENOMEM;
 
                        switch (w->id) {
-                       case snd_soc_dapm_mixer:
                        default:
                                snprintf(path->long_name, name_len, "%s %s",
                                         w->name, w->kcontrols[i].name);
-                       break;
+                               break;
                        case snd_soc_dapm_mixer_named_ctl:
                                snprintf(path->long_name, name_len, "%s",
                                         w->kcontrols[i].name);
-                       break;
+                               break;
                        }
 
                        path->long_name[name_len - 1] = '\0';