ASoC: atmel: tse850: fix off-by-one in the "ANA" enumeration count
authorPeter Rosin <peda@axentia.se>
Wed, 31 May 2017 12:32:33 +0000 (14:32 +0200)
committerMark Brown <broonie@kernel.org>
Tue, 6 Jun 2017 18:53:36 +0000 (19:53 +0100)
At some point I added the "Low" entry at the beginning of the array
without bumping the enumeration count from 9 to 10. Fix this. While at
it, fix the anti-pattern for the other enumeration (used by MUX{1,2}).

Fixes: aa43112445f0 ("ASoC: atmel: tse850: add ASoC driver for the Axentia TSE-850")
Signed-off-by: Peter Rosin <peda@axentia.se>
Signed-off-by: Mark Brown <broonie@kernel.org>
Cc: stable@vger.kernel.org
sound/soc/atmel/tse850-pcm5142.c

index a72c7d642026e131a27b8082f5a4b1d27d61e72f..3a13932831564662b70dc7646ed9ddaaf58a21fa 100644 (file)
@@ -227,7 +227,7 @@ int tse850_put_ana(struct snd_kcontrol *kctrl,
 static const char * const mux_text[] = { "Mixer", "Loop" };
 
 static const struct soc_enum mux_enum =
-       SOC_ENUM_SINGLE(SND_SOC_NOPM, 0, 2, mux_text);
+       SOC_ENUM_SINGLE(SND_SOC_NOPM, 0, ARRAY_SIZE(mux_text), mux_text);
 
 static const struct snd_kcontrol_new mux1 =
        SOC_DAPM_ENUM_EXT("MUX1", mux_enum, tse850_get_mux1, tse850_put_mux1);
@@ -252,7 +252,7 @@ static const char * const ana_text[] = {
 };
 
 static const struct soc_enum ana_enum =
-       SOC_ENUM_SINGLE(SND_SOC_NOPM, 0, 9, ana_text);
+       SOC_ENUM_SINGLE(SND_SOC_NOPM, 0, ARRAY_SIZE(ana_text), ana_text);
 
 static const struct snd_kcontrol_new out =
        SOC_DAPM_ENUM_EXT("ANA", ana_enum, tse850_get_ana, tse850_put_ana);