Mark Brown wrote:
On Fri, Jan 08, 2010 at 11:36:59AM -0600, Miguel Aguilar wrote:
Mark Brown wrote:
On Thu, Jan 07, 2010 at 04:17:21PM -0600, [email protected] 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.
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.
I see what your code is doing but at the minute it's making this
decision based on the device number that's being passed in by comparing
it as a pure number. This seems fragile - something symbolic that
joined things up a bit more wouldn't raise eyebrows in the same way.
Can you check the part of this patch related to registering both codecs AIC3x
and the voice codec?
Like I say that all looks fine to me but I can't really check if the
DaVinci code is idiomatic.
Is there any way to tell alsamixer to use the controls of one subdevice or the
other?
_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source
