[PATCH 08/10] S3C64XX I2S: Return correct codec clock
jassi brar
jassisinghbrar at gmail.com
Tue Sep 15 07:21:05 EDT 2009
On Tue, Sep 15, 2009 at 8:06 PM, Mark Brown
<broonie at opensource.wolfsonmicro.com> wrote:
> On Tue, Sep 15, 2009 at 07:02:40PM +0900, Jassi wrote:
>
>> - return i2s->iis_cclk;
>> + if(iismod & S3C64XX_IISMOD_IMS_SYSMUX)
>> + return i2s->iis_cclk;
>> + else
>> + return i2s->iis_pclk;
>
> There's three possible clocks here according to the datasheet - PCLK,
> CLKAUDIO and I2SCLK - so this will still give the wrong clock some of
> the time. Also, please use checkpatch
Yes, but the existing code doesn't consider I2SCLK at all.
The infra isn't there to provide the clock pointer anyhow.
Will use checkpatch, thanks.
More information about the linux-arm-kernel
mailing list