From: Vinod Koul <vinod.koul@intel.com>
To: Mark Brown <broonie@kernel.org>
Cc: liam.r.girdwood@linux.intel.com, patches.audio@intel.com,
alsa-devel@alsa-project.org
Subject: Re: [PATCH v3 0/3] ASoC: Add Multi CPU DAI support
Date: Thu, 19 Apr 2018 20:24:59 +0530 [thread overview]
Message-ID: <20180419145459.GK6014@localhost> (raw)
In-Reply-To: <20180419111239.GA27188@sirena.org.uk>
[-- Attachment #1.1: Type: text/plain, Size: 1093 bytes --]
On Thu, Apr 19, 2018 at 12:12:39PM +0100, Mark Brown wrote:
> On Thu, Apr 19, 2018 at 03:19:16PM +0530, Vinod Koul wrote:
> > 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.
>
> The main reason it's taking me a long time to get round to reviewing
> these is that they're big, invasive core changes with tiny commit logs
> that I'm going to need to go over in great detail.
Okay we can try adding more details but in essence it can be summarized to one
liner of patch series title. I.E make cpu_dai in rtd same as support for
codecs. We are adding capability to support multiple cpu_dais in a dai link.
Yes the changes are intrusive as we have to replace every instance of use of
cpu_dai with multiple cpu_dais in the core.
Do let us know if you are looking for some specific to add here..
Thanks
--
~Vinod
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
[-- Attachment #2: Type: text/plain, Size: 0 bytes --]
prev parent reply other threads:[~2018-04-19 14:50 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-04-19 9:49 [PATCH v3 0/3] ASoC: Add Multi CPU DAI support Vinod Koul
2018-04-19 9:49 ` [PATCH v3 1/3] ASoC: Add initial support for multiple CPU DAIs Vinod Koul
2018-04-30 16:22 ` Charles Keepax
2018-04-19 9:49 ` [PATCH v3 2/3] ASoC: Add Multi CPU DAI support for PCM ops Vinod Koul
2018-04-30 16:22 ` Charles Keepax
2018-05-02 9:15 ` Shreyas NC
2018-04-19 9:49 ` [PATCH v3 3/3] ASoC: Add Multi CPU DAI support in DAPM Vinod Koul
2018-04-30 16:24 ` Charles Keepax
2018-04-19 11:12 ` [PATCH v3 0/3] ASoC: Add Multi CPU DAI support Mark Brown
2018-04-19 14:54 ` Vinod Koul [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180419145459.GK6014@localhost \
--to=vinod.koul@intel.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=liam.r.girdwood@linux.intel.com \
--cc=patches.audio@intel.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).