[alsa-devel] [PATCH v4] ASoC: kirkwood: add S/PDIF support

Mark Brown broonie at kernel.org
Thu Oct 24 13:08:33 CEST 2013

On Mon, Oct 21, 2013 at 12:24:22PM +0200, Jean-Francois Moine wrote:

> +static struct snd_soc_dai_driver kirkwood_i2s_dai[3] = {
> +	DAI_DEF("i2s", 0,
> +		SNDRV_PCM_RATE_44100 | SNDRV_PCM_RATE_48000 |
> +					SNDRV_PCM_RATE_96000),
> +	DAI_DEF("spdif", 1,
> +		SNDRV_PCM_RATE_44100 | SNDRV_PCM_RATE_48000 |
> +					SNDRV_PCM_RATE_96000),
> +	DAI_DEF("i2s/spdif", 2,
> +		SNDRV_PCM_RATE_44100 | SNDRV_PCM_RATE_48000 |
> +					SNDRV_PCM_RATE_96000),

I didn't take this version because this third DAI reintroduces the
problem with having both S/PDIF and I2S in the single DAI.  The problem
there is that you end up with one traditional DAI linked to two
different CODECs which the framework doesn't understand - it's
introducing a DAI which does what v1 does.  

This is what DPCM addresses for this platform, it allows the single DMA
controller to be used in conjunction with the two external DAIs by
splitting the DMA from the DAI in the model the subsystem has.
Hopefully someone will have the time and enthusiasm to update to use
this and support systems that want to use I2S and S/PDIF simultaneously.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20131024/8253f89d/attachment.sig>

More information about the Alsa-devel mailing list