-----Original Message----- From: Pierre-Louis Bossart [mailto:pierre-louis.bossart@linux.intel.com] Sent: Tuesday, December 20, 2016 10:32 PM To: Keyon Jie yang.jie@linux.intel.com; sound-open-firmware@alsa- project.org; liam.r.girdwood@linux.intel.com Cc: Zhang, Keqiao keqiao.zhang@intel.com; Jie, Yang yang.jie@intel.com; Ingalsuo, Seppo seppo.ingalsuo@intel.com; Lin, Mengdong mengdong.lin@intel.com Subject: Re: [Sound-open-firmware] [PATCH 5/8] platform: add divider m/n setting for 50 bclk_fs
On 12/20/16 1:46 AM, Keyon Jie wrote:
for 24 bits ssp setting, we need set bclk_fs to 50, here add the divider m/n setting for this case.
but we should not be using those dividers?
Yes, actually we won't use this(as patch 8/8), Liam also mentioned this, my intention here was complete these dividers for 50 bclk_fs. We can remove it(and all those dividers related code in this platform.c) if we will never use them anymore.
Thanks, ~Keyon
Signed-off-by: Keyon Jie yang.jie@linux.intel.com
src/platform/baytrail/platform.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/src/platform/baytrail/platform.c b/src/platform/baytrail/platform.c index d21845f..ee6cdd9 100644 --- a/src/platform/baytrail/platform.c +++ b/src/platform/baytrail/platform.c @@ -106,6 +106,7 @@ static const struct ssp_mn ssp_mn_conf[] = { {25000000, 400, 44100, 441, 625}, /* 17.64MHz */ {19200000, 24, 48000, 3, 50}, /* 1.152MHz */ {19200000, 32, 48000, 2, 25}, /* 1.536MHz */
- {19200000, 50, 48000, 1, 8}, /* 2.4MHz */ {19200000, 64, 48000, 4, 25}, /* 3.072MHz */ {19200000, 400, 44100, 441, 480}, /* 17.64MHz */
};