Commit b22ead2a authored by Mark Brown's avatar Mark Brown

ASoC: Coalesce register writes for DAPM sequences

Reduce the number of register writes we need to set the power state for
a CODEC by coalescing updates to widgets with the same sequence order and
same register into a single write.

This can be a noticable performance improvement with slow or heavily
contended control buses, such as I2C controllers with a low clock
frequency, and is particularly noticable when resuming. It can also
reduce the noticability of and pops and clicks by ensuring that left
and right channels are powered simultaneously if they are in the same
register.

Currently widgets that have events are not coalesced, including PGAs
which may use the volume ramping control.
Signed-off-by: default avatarMark Brown <broonie@opensource.wolfsonmicro.com>
parent 163cac06
...@@ -713,6 +713,8 @@ static int dapm_seq_compare(struct snd_soc_dapm_widget *a, ...@@ -713,6 +713,8 @@ static int dapm_seq_compare(struct snd_soc_dapm_widget *a,
{ {
if (sort[a->id] != sort[b->id]) if (sort[a->id] != sort[b->id])
return sort[a->id] - sort[b->id]; return sort[a->id] - sort[b->id];
if (a->reg != b->reg)
return a->reg - b->reg;
return 0; return 0;
} }
...@@ -733,63 +735,133 @@ static void dapm_seq_insert(struct snd_soc_dapm_widget *new_widget, ...@@ -733,63 +735,133 @@ static void dapm_seq_insert(struct snd_soc_dapm_widget *new_widget,
list_add_tail(&new_widget->power_list, list); list_add_tail(&new_widget->power_list, list);
} }
/* Apply a DAPM power sequence */ /* Apply the coalesced changes from a DAPM sequence */
static void dapm_seq_run(struct snd_soc_codec *codec, struct list_head *list, static void dapm_seq_run_coalesced(struct snd_soc_codec *codec,
int event) struct list_head *pending)
{ {
struct snd_soc_dapm_widget *w; struct snd_soc_dapm_widget *w;
int reg, power;
unsigned int value = 0;
unsigned int mask = 0;
unsigned int cur_mask;
reg = list_first_entry(pending, struct snd_soc_dapm_widget,
power_list)->reg;
list_for_each_entry(w, pending, power_list) {
cur_mask = 1 << w->shift;
BUG_ON(reg != w->reg);
if (w->invert)
power = !w->power;
else
power = w->power;
mask |= cur_mask;
if (power)
value |= cur_mask;
pop_dbg(codec->pop_time,
"pop test : Queue %s: reg=0x%x, 0x%x/0x%x\n",
w->name, reg, value, mask);
}
pop_dbg(codec->pop_time,
"pop test : Applying 0x%x/0x%x to %x in %dms\n",
value, mask, reg, codec->pop_time);
pop_wait(codec->pop_time);
snd_soc_update_bits(codec, reg, mask, value);
}
/* Apply a DAPM power sequence.
*
* We walk over a pre-sorted list of widgets to apply power to. In
* order to minimise the number of writes to the device required
* multiple widgets will be updated in a single write where possible.
* Currently anything that requires more than a single write is not
* handled.
*/
static void dapm_seq_run(struct snd_soc_codec *codec, struct list_head *list,
int event, int sort[])
{
struct snd_soc_dapm_widget *w, *n;
LIST_HEAD(pending);
int cur_sort = -1;
int cur_reg = SND_SOC_NOPM;
int ret; int ret;
list_for_each_entry(w, list, power_list) { list_for_each_entry_safe(w, n, list, power_list) {
ret = 0;
/* Do we need to apply any queued changes? */
if (sort[w->id] != cur_sort || w->reg != cur_reg) {
if (!list_empty(&pending))
dapm_seq_run_coalesced(codec, &pending);
INIT_LIST_HEAD(&pending);
cur_sort = -1;
cur_reg = SND_SOC_NOPM;
}
switch (w->id) { switch (w->id) {
case snd_soc_dapm_pre: case snd_soc_dapm_pre:
if (!w->event) if (!w->event)
list_for_each_entry_continue(w, list, list_for_each_entry_safe_continue(w, n, list,
power_list); power_list);
if (event == SND_SOC_DAPM_STREAM_START) { if (event == SND_SOC_DAPM_STREAM_START)
ret = w->event(w, ret = w->event(w,
NULL, SND_SOC_DAPM_PRE_PMU); NULL, SND_SOC_DAPM_PRE_PMU);
if (ret < 0) else if (event == SND_SOC_DAPM_STREAM_STOP)
pr_err("PRE widget failed: %d\n",
ret);
} else if (event == SND_SOC_DAPM_STREAM_STOP) {
ret = w->event(w, ret = w->event(w,
NULL, SND_SOC_DAPM_PRE_PMD); NULL, SND_SOC_DAPM_PRE_PMD);
if (ret < 0)
pr_err("PRE widget failed: %d\n",
ret);
}
break; break;
case snd_soc_dapm_post: case snd_soc_dapm_post:
if (!w->event) if (!w->event)
list_for_each_entry_continue(w, list, list_for_each_entry_safe_continue(w, n, list,
power_list); power_list);
if (event == SND_SOC_DAPM_STREAM_START) { if (event == SND_SOC_DAPM_STREAM_START)
ret = w->event(w, ret = w->event(w,
NULL, SND_SOC_DAPM_POST_PMU); NULL, SND_SOC_DAPM_POST_PMU);
if (ret < 0) else if (event == SND_SOC_DAPM_STREAM_STOP)
pr_err("POST widget failed: %d\n",
ret);
} else if (event == SND_SOC_DAPM_STREAM_STOP) {
ret = w->event(w, ret = w->event(w,
NULL, SND_SOC_DAPM_POST_PMD); NULL, SND_SOC_DAPM_POST_PMD);
if (ret < 0)
pr_err("POST widget failed: %d\n",
ret);
}
break; break;
default: case snd_soc_dapm_input:
case snd_soc_dapm_output:
case snd_soc_dapm_hp:
case snd_soc_dapm_mic:
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); ret = dapm_generic_apply_power(w);
if (ret < 0)
pr_err("Failed to apply widget power: %d\n",
ret);
break; break;
default:
/* If there's an event or an invalid register
* then run immediately, otherwise store the
* updates so that we can coalesce. */
if (w->reg >= 0 && !w->event) {
cur_sort = sort[w->id];
cur_reg = w->reg;
list_move(&w->power_list, &pending);
} else {
ret = dapm_generic_apply_power(w);
}
} }
if (ret < 0)
pr_err("Failed to apply widget power: %d\n",
ret);
} }
if (!list_empty(&pending))
dapm_seq_run_coalesced(codec, &pending);
} }
/* /*
...@@ -857,10 +929,10 @@ static int dapm_power_widgets(struct snd_soc_codec *codec, int event) ...@@ -857,10 +929,10 @@ static int dapm_power_widgets(struct snd_soc_codec *codec, int event)
} }
/* Power down widgets first; try to avoid amplifying pops. */ /* Power down widgets first; try to avoid amplifying pops. */
dapm_seq_run(codec, &codec->down_list, event); dapm_seq_run(codec, &codec->down_list, event, dapm_down_seq);
/* Now power up. */ /* Now power up. */
dapm_seq_run(codec, &codec->up_list, event); dapm_seq_run(codec, &codec->up_list, event, dapm_up_seq);
/* If we just powered the last thing off drop to standby bias */ /* If we just powered the last thing off drop to standby bias */
if (codec->bias_level == SND_SOC_BIAS_PREPARE && !sys_power) { if (codec->bias_level == SND_SOC_BIAS_PREPARE && !sys_power) {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment