On Thu, 2009-03-05 at 14:00 -0500, Hugo Villeneuve wrote:
> On Thu, 05 Mar 2009 09:30:05 -0800
> Kevin Hilman <[email protected]> wrote:
> 
> > Hugo Villeneuve <[email protected]> writes:
> > 
> > > Hi Kevin,
> > > can you apply those two patches from ALSA to the
> > > linux-davinci-2.6.git tree to fix some running and compilation bugs?
> > >
> > > Here are the relevant patches infos:
> > >
> > > http://git.alsa-project.org/?p=alsa-kernel.git;a=commitdiff;h=7f7ebad4cb7dba50c22e290363e113f055400d6a
> > 
> > This one is already in DaVinci git as commit
> > 3a4250d4d77b5afc4195214e72439602c592f5e6
> > 
> > > http://git.alsa-project.org/?p=alsa-kernel.git;a=commitdiff;h=04c5ca6a180c478beca953b3268cc3b806b6015d

In the patch

+       /* Fsref can be 32000, 44100 or 48000. */
+       fs = params_rate(params);
+
+#ifndef CONFIG_SFFSDR_FPGA
+       /* Without the FPGA module, the Fs is fixed at 44100 Hz */
+       if (fs != 44100) {
+               pr_debug("warning: only 44.1 kHz is supported without SFFSDR 
FPGA module\n");
+               return -EINVAL;
+       }
+#endif

You may want to consider modifying capture/playback rates (defined by
PCM3008_RATES).  I believe ALSA check and printout a warning message
if the rate is not supported.  Also, you won't be able to take advantage
of
ALSA's automatic rate conversion with this implementation.


Regards,

Steve


_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to