Re: [PATCH] ASoC: simple-card: cleanup asoc_simple_card_parse_of()code

From: Mark Brown
Date: Tue Dec 31 2013 - 07:21:40 EST


On Tue, Dec 31, 2013 at 03:27:53AM +0000, Li.Xiubo@xxxxxxxxxxxxx wrote:

> Also, for most of the devices of the same DAI link, they use the same DAI
> formats, but I have found some drivers like : "sound/soc/pxa/magician.c"
> and "sound/soc/s6000/s6105-ipcam.c", etc, may have different DAI format
> settings for some reasons(maybe some formats are the CPU/Codec devices
> default setting that it needn't set it here or others).

The ipcam still has that? It was supposed to have been fixed before it
was merged since it was identified as a bug in the CPU DAI during review.
I don't know about magician since it was before my time but I suspect it
suffers from the same issue or that there's something else in there that
needs some other hardware more explicitly representing.

> I think maybe this is also the reason why the simple-card has common DAI
> format and CPU/CODEC private DAI formats at the same time.

I'd really want to see a real use case that actually was a simple card.

Attachment: signature.asc
Description: Digital signature