Re: [PATCH 1/2] ASoC: ak4613: add use CTRL2::CKS = 00

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Mark

> > ak4613 CTRL2::CKS controls Speed Mode (Normal/Double/Quad) for Master
> > Mode. In default, Normal Mode uses 512fs, but in such case, requested
> > input clock for all mode will be same. Then, it might not be eble to
> > support correct output if input clock was not enough.
> > This patch uses 256fs, and reduse requested imput clock for Noral Mode.
> 
> >  	snd_soc_update_bits(codec, CTRL1, FMT_MASK, fmt_ctrl);
> > -	snd_soc_update_bits(codec, CTRL2, DFS_MASK, ctrl2);
> > +
> > +	/* CKS = 00 */
> > +	snd_soc_update_bits(codec, CTRL2, (CKS_MASK | DFS_MASK), ctrl2);
> 
> This looks board specific - shouldn't we be setting the speed mode based
> on a combination of the sampling rate and the master clock for the
> device?  

I removed "FIXME" comments, but OK.
I will fixup this patch-set

Best regards
---
Kuninori Morimoto



[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux