> -----Original Message----- > From: [email protected] > [mailto:[email protected]] On Behalf > Of Troy Kisky > Sent: Friday, December 19, 2008 11:51 PM > To: Mark Brown > Cc: [email protected]; davinci-linux-open- > [email protected]; Medisetty, Naresh > Subject: Re: [alsa-devel] [PATCH 0/8]ALSA: ASoc: DaVinci: cleanup > > Mark Brown wrote: > > On Thu, Dec 18, 2008 at 12:36:38PM -0700, Troy Kisky wrote: > > > >> Sorry, for the long series to do something so simple, but there were > >> unresolved objections when I first posted this to the davinci list. > > > > What were the issues people had? > > > >> Hopefully, having this split into tiny pieces will make resolving > >> those differences easier. > > > > It certainly makes review easier. > > _______________________________________________ > > Alsa-devel mailing list > > [email protected] > > http://mailman.alsa-project.org/mailman/listinfo/alsa-devel > > > > The meaning change of SND_SOC_DAIFMT_NB_NF was not liked. > And my comment that I2S mode is not supported on davinci > was not liked. I'll let Naresh explain. >
The comment regarding I2S mode is absolutely correct, since the davinci cannot support I2S where the codecs (like TLV320AIC23) has fixed bit clock frequency. Documenting this limitation is a very good idea. Still I find difficulty to agree with Troy regarding the meaning change of SND_SOC_DAIFMT_NB_NF, since the existing meaning is correct w.r.t davinci McBSP. > _______________________________________________ > Davinci-linux-open-source mailing list > [email protected] > http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source _______________________________________________ Davinci-linux-open-source mailing list [email protected] http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
