Re: [PATCH 0/2] ASoC: soc-core: use snd_soc_dai_link_component for cpu

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Pierre

Thank you for your feedback

> > 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.

Sorry, what does your "Intel patches" means ?
Do you mean like this ?

	Subject:  [PATCH v6 0/3] ASoC: Add Multi CPU DAI support
	Date: Wed, 20 Jun 2018 16:24:14 +0530

If so, actually I'm very waiting this patch series.
But, it seems nothing happen on ALSA ML after that.
So, I thought it was rejected or something happen.
I think my patch and above are basically using same idea for dai_link,
I think there is no conflict around dai_link.
If above patch series was delayed or temporary aborted,
and will be restart/retry multi CPU support someday,
then, it can based on it, I thought.
If Intel still have plan about it, I'm very happy to wait.

> 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.


Best regards
---
Kuninori Morimoto
_______________________________________________
Alsa-devel mailing list
Alsa-devel@xxxxxxxxxxxxxxxx
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel



[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux