On Mon, Dec 8, 2008 at 1:11 PM, Mark Brown broonie@sirena.org.uk wrote:
[You might want to check your MUA - most of your recent mails have done the above and marked the first line of your reply as part of the quote.]
Grrr. It's gmail & iceweacel.
Would it work if there is some set_mclk(int enable) callback in struct aic3x_setup_data, then codec->write in aic3x_init points to higher level aic3x_write_xxx which takes care of calling set_mclk(1) when needed and
call
aic3x_write? Also delayed set_mclk(0) can be put into aic3x driver.
TBH I don't really see any benefit from doing this in the codec driver to doing it in the core.
Agree. Also other codecs might use mclk for their internal state machine. IRCC, TSC2301 had some weird internal states and timeouts when mclk was required to be on and when it can be shutdown.
Jarkko