ASoC: Move call to snd_soc_dapm_new_widgets() after snd_soc_dapm_auto_nc_codec_pins()
authorLars-Peter Clausen <lars@metafoo.de>
Tue, 27 Aug 2013 13:51:00 +0000 (15:51 +0200)
committerMark Brown <broonie@linaro.org>
Tue, 27 Aug 2013 14:39:43 +0000 (15:39 +0100)
Call snd_soc_dapm_new_widgets() before the auto non-connected pins have been
marked as not connected will power the system under the assumption that those
pins are connected. Once the pins have been marked as disconnected the system
there will be an additional power run. This can cause unnecessary power
transitions. Calling snd_soc_dapm_new_widgets() only after the pins have been
marked as non-connected avoids this.

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Mark Brown <broonie@linaro.org>
sound/soc/soc-core.c

index 85e2a8b8f2888882344d0130996d00a7a18cab85..9375012ccb215fb25fc7f5f2ea86d27090cd8928 100644 (file)
@@ -1803,12 +1803,12 @@ static int snd_soc_instantiate_card(struct snd_soc_card *card)
                }
        }
 
-       snd_soc_dapm_new_widgets(&card->dapm);
-
        if (card->fully_routed)
                list_for_each_entry(codec, &card->codec_dev_list, card_list)
                        snd_soc_dapm_auto_nc_codec_pins(codec);
 
+       snd_soc_dapm_new_widgets(&card->dapm);
+
        ret = snd_card_register(card->snd_card);
        if (ret < 0) {
                dev_err(card->dev, "ASoC: failed to register soundcard %d\n",