[alsa-devel] [PATCH v4-resend 1/7] ASoC: sirf: Add SiRF internal audio codec driver

[Barry Song]

2014-03-03 13:07 GMT+08:00 Mark Brown <broonie at kernel.org>:
> On Mon, Mar 03, 2014 at 02:15:17AM +0000, Rongjun Ying wrote:
>
>> The codec clock is only used by this driver. And this clock is not managed by
>> clock controller. The Audio controller codec is always enable. So I think
>> it is not need abstract to general clock API.
>
> No, you're still missing the point.  You are requesting external clocks,
> why are you not managing those here?
Hi Mark,
here the clock is an internal clock in the internal codec. the
external clock of audio controller is managed in system suspend/resume
as you see. but they are not managed in runtime suspend/resume here
because the external clock is always enabled to make sure the charge
pump to be working to make the board have stable current and voltage.
otherwise, touchscreen ADC will not be accurate.

>
> _______________________________________________
> Alsa-devel mailing list
> Alsa-devel at alsa-project.org
> http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
>

-barry