ASoC: When disabling WM8994 FLL force a source selection
authorMark Brown <broonie@opensource.wolfsonmicro.com>
Fri, 3 Dec 2010 16:02:10 +0000 (16:02 +0000)
committerMark Brown <broonie@opensource.wolfsonmicro.com>
Fri, 3 Dec 2010 16:45:12 +0000 (16:45 +0000)
When we disable the WM8994 FLL code path sharing means that we end up
writing out a configuration. Currently this is the currently active
input and output frequency (which causes snd_soc_update_bits() to
suppress actual writes both immediately and in the common case where
we reenable the same configuration later) but we allow machine drivers
to pass through a source of zero. Since the register values written
are one less than the source constants this causes corruption of other
bitfields in the register.

Fix this by using the most recently configured FLL source when none is
provided.

Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Acked-by: Liam Girdwood <lrg@slimlogic.co.uk>
Cc: stable@kernel.org
sound/soc/codecs/wm8994.c

index af3a98ae05796885f3a38add1c5957c35ab377c0..b37e95c4e7c7491cede2c2e296aa87f88adc31c7 100644 (file)
@@ -1677,6 +1677,7 @@ static int _wm8994_set_fll(struct snd_soc_codec *codec, int id, int src,
                /* Allow no source specification when stopping */
                if (freq_out)
                        return -EINVAL;
+               src = wm8994->fll[id].src;
                break;
        case WM8994_FLL_SRC_MCLK1:
        case WM8994_FLL_SRC_MCLK2: