mbox series

[RFC,0/3] ASoC: Add Multi CPU DAI support

Message ID 20191225191501.23848-1-yung-chuan.liao@linux.intel.com (mailing list archive)
Headers show
Series ASoC: Add Multi CPU DAI support | expand

Message

Bard Liao Dec. 25, 2019, 7:14 p.m. UTC
From: Bard Liao <yung-chuan.liao@linux.intel.com>

As discussed in [1], ASoC core supports multi codec DAIs
on a DAI link. However it does not do so for CPU DAIs.

So, add support for multi CPU DAIs on a DAI Link by adding
multi CPU DAI in Card instantiation, suspend and resume
functions, PCM ops, stream handling functions and DAPM.

[1]: https://www.spinics.net/lists/alsa-devel/msg71369.html


Shreyas NC (3):
  ASoC: Add initial support for multiple CPU DAIs
  ASoC: Add multiple CPU DAI support for PCM ops
  ASoC: Add multiple CPU DAI support in DAPM

 include/sound/soc.h  |  15 ++
 sound/soc/soc-core.c | 183 ++++++++++--------
 sound/soc/soc-dapm.c | 131 +++++++------
 sound/soc/soc-pcm.c  | 432 ++++++++++++++++++++++++++++---------------
 4 files changed, 483 insertions(+), 278 deletions(-)

Comments

Mark Brown Jan. 14, 2020, 3:47 p.m. UTC | #1
On Thu, Dec 26, 2019 at 03:14:58AM +0800, Bard liao wrote:
> From: Bard Liao <yung-chuan.liao@linux.intel.com>
> 
> As discussed in [1], ASoC core supports multi codec DAIs
> on a DAI link. However it does not do so for CPU DAIs.

This approach seems broadly fine - I see you'd already agreed
some updates with Morimoto-san which look sensible.
Liao, Bard Jan. 15, 2020, 12:20 a.m. UTC | #2
> -----Original Message-----
> From: Mark Brown [mailto:broonie@kernel.org]
> Sent: Tuesday, January 14, 2020 11:48 PM
> To: Bard liao <yung-chuan.liao@linux.intel.com>
> Cc: alsa-devel@alsa-project.org; tiwai@suse.de;
> liam.r.girdwood@linux.intel.com; pierre-louis.bossart@linux.intel.com; Liao,
> Bard <bard.liao@intel.com>; kuninori.morimoto.gx@renesas.com
> Subject: Re: [PATCH RFC 0/3] ASoC: Add Multi CPU DAI support
> 
> On Thu, Dec 26, 2019 at 03:14:58AM +0800, Bard liao wrote:
> > From: Bard Liao <yung-chuan.liao@linux.intel.com>
> >
> > As discussed in [1], ASoC core supports multi codec DAIs on a DAI
> > link. However it does not do so for CPU DAIs.
> 
> This approach seems broadly fine - I see you'd already agreed some updates
> with Morimoto-san which look sensible.

Thanks. I will send 2nd version soon.