3 Jul
2014
3 Jul
'14
5:12 p.m.
On Thu, Jul 03, 2014 at 04:56:43PM +0200, Daniel Mack wrote:
The TAS5086 has two power domains, DVDD and AVDD. Enable them both as long as the codec is in use.
Also, switch on the power to identify the chip at device probe level, and switch it off again afterwards. The codec level will take care for power handling later.
Applied, thanks. One small thing:
- /*
* The chip has been identified, so we can turn off the power* again until the dai link is set up.*/- regulator_bulk_disable(ARRAY_SIZE(priv->supplies), priv->supplies);
- if (ret == 0)
ret = snd_soc_register_codec(&i2c->dev, &soc_codec_dev_tas5086,&tas5086_dai, 1);
It can be slightly smoother to do the disable after registering so that if we immediately cause a card to register and that causes the device to become active again we don't bounce the power. Nobody is ever likely to notice the difference though.