]> git.karo-electronics.de Git - karo-tx-linux.git/commitdiff
ASoC: Coalesce power updates for PGAs
authorMark Brown <broonie@opensource.wolfsonmicro.com>
Sun, 7 Jun 2009 12:37:44 +0000 (13:37 +0100)
committerMark Brown <broonie@opensource.wolfsonmicro.com>
Mon, 8 Jun 2009 12:52:05 +0000 (13:52 +0100)
Handle gain ramping for PGAs so we can coalesce their power updates too.
This is not ideal since we can't cope properly with gain ramping for
stereo paths but that was the case without coalescing and gain ramping
is relatively infrequently used so the effects are limited.

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

index 3fc791c28aa83ac9d4d01fb959a64fd840a32df4..7299ce405b2d0da47dd1c7d813752f10a8904088 100644 (file)
@@ -786,6 +786,10 @@ static void dapm_seq_run_coalesced(struct snd_soc_codec *codec,
                                pr_err("%s: pre event failed: %d\n",
                                       w->name, ret);
                }
+
+               /* Lower PGA volume to reduce pops */
+               if (w->id == snd_soc_dapm_pga && !w->power)
+                       dapm_set_pga(w, w->power);
        }
 
        if (reg >= 0) {
@@ -797,6 +801,10 @@ static void dapm_seq_run_coalesced(struct snd_soc_codec *codec,
        }
 
        list_for_each_entry(w, pending, power_list) {
+               /* Raise PGA volume to reduce pops */
+               if (w->id == snd_soc_dapm_pga && w->power)
+                       dapm_set_pga(w, w->power);
+
                /* power up post event */
                if (w->power && w->event &&
                    (w->event_flags & SND_SOC_DAPM_POST_PMU)) {
@@ -886,8 +894,6 @@ static void dapm_seq_run(struct snd_soc_codec *codec, struct list_head *list,
                case snd_soc_dapm_line:
                case snd_soc_dapm_spk:
                        /* No register support currently */
-               case snd_soc_dapm_pga:
-                       /* Don't coalsece these yet due to gain ramping */
                        ret = dapm_generic_apply_power(w);
                        break;