On 1/16/19 2:26 AM, Kuninori Morimoto wrote:
Hi Mark
These add snd_soc_dai_link_component support for CPU. I know multi-CPU support patch was posted to ML, but I think it is not yet accepted (or rejected ?).
If you are referring to the Intel patches, they were really too intrusive and complicated. Quite frankly I don't think anyone at Intel (me included) understands what is "legacy" and what is "modern", so adding multiple cpu support in there was really guesswork and copy/replace of the multi-codec parts, likely to break and impossible to test.
If these patches were accepted, we can finally switch to modern style dai_link, and remove legacy style dai_link.
it might be a better idea to remove the legacy first so that we have a clean slate to add multiple CPUs?
If you have a blurb or description of the end-goal and what evolutions are required for dai links, it'd be really nice. Following incremental patches over multiple kernel versions isn't straightforward and the impacts of your changes isn't clear to me, e.g. I'd really like to know if the current way Intel uses dailinks needs to be changed or if we are good.
Thanks
-Pierre
Kuninori Morimoto (2): ASoC: soc-core: add .num_platform for dai_link ASoC: soc-core: use snd_soc_dai_link_component for cpu
For switching to modern, I added "use modern style dai_link" and "remove legacy style dai_link" patches as sample. If these have no objection, I will post full patch to ML.
Kuninori Morimoto (3): ASoC: audio-graph/simple-card: use modern style dai_link for CPU ASoC: atmel: atmel-classd: use modern style dai_link ASoC: soc-core: remove regacy dai_link binding method
Kuninori Morimoto _______________________________________________ Alsa-devel mailing list Alsa-devel@alsa-project.org http://mailman.alsa-project.org/mailman/listinfo/alsa-devel