[alsa-devel] [PATCH 2/2] Davinci: DM365: Enable DaVinci Voice Codec support for DM365 EVM

Miguel Aguilar miguel.aguilar at ridgerun.com
Fri Jan 8 18:36:59 CET 2010


Mark Brown wrote:
> On Thu, Jan 07, 2010 at 04:17:21PM -0600, miguel.aguilar at ridgerun.com wrote:
> 
>> +		if (device == 0)
>> +			davinci_cfg_reg(DM365_EVT2_ASP_TX);
>> +		else
>> +			davinci_cfg_reg(DM365_EVT2_VC_TX);
> 
> I'd be a bit more comfortable with this if it were using something more
> symbolic like a #define or enum rather than checking a bare number to
> work out which device it's talking to.
> 
> Otherwise this looks good, but I've no familiarity with DaVinci
> specifics so...


The idea of these function is check at runtime if the user space application is 
requesting the AIC3x or the voice codec, then it will set the proper source for 
the dma channels, since the ASP and the Voice Codec share the same dma channels, 
so that's why use a #define doesn't make sense.

Can you check the part of this patch related to registering both codecs AIC3x 
and the voice codec?

Thank you,

Miguel Aguilar


More information about the Alsa-devel mailing list