On 01/11/2013 02:46 PM, Mark Brown wrote:
On Fri, Jan 11, 2013 at 02:40:20PM +0100, Peter Ujfalusi wrote:
Since the core saves the previous bias_level in before suspend I can just do this here:
twl6040_set_bias_level(codec, codec->dapm.suspend_bias_level);
If we were in _OFF before it is going to do nothing, but if we were in _STANDBY it brings the codec back.
That'll work, yes (though removing the function when idle_bias_off is implemented would obviously be better).
True. Along with the suspend function since I'm not doing anything fancy there and if idle_bias_off is set the core will not call suspend of the driver if it is not already in _OFF. I'll keep this in mind for the future and resend with only with
twl6040_set_bias_level(codec, SND_SOC_BIAS_STANDBY);
in the resume. But I wait if you have other comments for the rest for the series.