[Sound-open-firmware] [PATCH 5/8] platform: add divider m/n setting for 50 bclk_fs

[Jie, Yang]

> -----Original Message-----
> From: Pierre-Louis Bossart [mailto:pierre-louis.bossart at linux.intel.com]
> Sent: Tuesday, December 20, 2016 10:32 PM
> To: Keyon Jie <yang.jie at linux.intel.com>; sound-open-firmware at alsa-
> project.org; liam.r.girdwood at linux.intel.com
> Cc: Zhang, Keqiao <keqiao.zhang at intel.com>; Jie, Yang <yang.jie at intel.com>;
> Ingalsuo, Seppo <seppo.ingalsuo at intel.com>; Lin, Mengdong
> <mengdong.lin at 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 at 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 */
> >  };
> >