Steve,

I'm not sure that this is the same case, please take a look into this thread between Mark and me, where I pointed the problem in the machine driver for handling the two codecs:

Hi Mark,

I have a specific doubt about the codec_dev element snd_soc_device struct of the DM365 EVM.

Since I have two codecs on the EVM, I have two snd_soc_codec_device structures:

• AIC3x

struct snd_soc_codec_device soc_codec_dev_aic3x = { .probe = aic3x_probe, .remove = aic3x_remove, .suspend = aic3x_suspend, .resume = aic3x_resume, };

• Voice Codec

struct snd_soc_codec_device soc_codec_dev_cq93vc = { .probe = cq93vc_probe, .remove = cq93vc_remove, .suspend = cq93vc_suspend, .resume = cq93vc_resume, };

But I can set only one at a time in the dm365_evm_snd_devdata:

/* evm audio subsystem */ static struct snd_soc_device dm365_evm_snd_devdata = { .card = &dm365_snd_soc_card_evm,

---

.codec_dev = &soc_codec_dev_aic3x, or .codec_dev = &soc_codec_dev_cq93vc,

---

.codec_data = &aic3x_setup, };

So in case what is the proper way to do this?

Thanks,

Miguel Aguilar

Mark's answer:

On Tue, Jan 19, 2010 at 02:47:16PM -0600, Miguel Aguilar wrote:

...

But I can set only one at a time in the dm365_evm_snd_devdata:

...

...

...

So in case what is the proper way to do this?

For now use an ifdef in the machine driver. There's some work due to begin fairly shortly (I've CCed in my co-maintainer Liam who's actually going to be doing the work) to allow multiple CODECs to work simultaneously, if you put the ifdef in the machine driver now then it should be straightforward to update the machine driver to make use of this new feature when it's ready.

Regards, Miguel Aguilar

Steve Chen wrote:

On Thu, 2010-01-21 at 11:35 -0600, Miguel Aguilar wrote:

...

Hi,

The following patch series adds support for the Voice Codec in the Davinci platform and in this particular case for the DM365 EVM.

The DM365 EVM has two codecs the AIC3x and the Voice Codec, however at this point is not possible to have both enabled simultaneously since the soc-core doesn't support that, so a patch for enable both simultaneously will be sent when this support is available in the soc-core.

I thought, perhaps incorrectly, that soc-core already has the ability to support multiple codecs. When I worked on ALSA soc support for DM6467, one of the McASP was connected to AIC3x and the other is SPDIF output. Both interface can work at the same time. I just assumed that if we have two McASP (or other audio controllers) connected to two codecs. They would both just work under ALSA soc. Please let me know if I'm mistaken.

Thanks,

Steve

On Thu, 2010-01-21 at 19:25 +0000, Mark Brown wrote:

On 21 Jan 2010, at 18:52, Steve Chen schen@mvista.com wrote:

...

On Thu, 2010-01-21 at 11:35 -0600, Miguel Aguilar wrote:

...

Hi,

The following patch series adds support for the Voice Codec in the Davinci platform and in this particular case for the DM365 EVM.

The DM365 EVM has two codecs the AIC3x and the Voice Codec, however at this point is not possible to have both enabled simultaneously since the soc-core doesn't support that, so a patch for enable both simultaneously will be sent when this support is available in the soc-core.

I thought, perhaps incorrectly, that soc-core already has the ability to support multiple codecs. When I worked on ALSA soc support for DM6467, one of the McASP was connected to AIC3x and the other is SPDIF output. Both interface can work at the same time. I just assumed that if we have two McASP (or other audio controllers) connected to two codecs. They would both just work under ALSA soc. Please let me know if I'm mistaken.

It can only cope with one CODEC with registers. That S/PDIF interface has no registers so could be made to work.

I just pull up the code and looked at it again. You are right. Sorry about the noise.

Steve