From: Vinod Koul <vinod.koul@intel.com>
To: Lars-Peter Clausen <lars@metafoo.de>
Cc: alsa-devel@alsa-project.org, Mark Brown <broonie@kernel.org>,
lgirdwood@gmail.com
Subject: Re: [PATCH 00/19]: add mrfld DSP support
Date: Thu, 26 Jun 2014 16:01:47 +0530 [thread overview]
Message-ID: <20140626103147.GW22053@intel.com> (raw)
In-Reply-To: <53AA4FC0.5010202@metafoo.de>
On Wed, Jun 25, 2014 at 06:27:44AM +0200, Lars-Peter Clausen wrote:
> On 06/23/2014 12:09 PM, Mark Brown wrote:
> >On Fri, Jun 20, 2014 at 10:57:25AM +0200, Lars-Peter Clausen wrote:
> >
> >>The idea is to get rid of the IO abstraction layer inside ASoC and only rely
> >>on regmap for this. This is a process this has been going on ever since the
> >>regmap framework was added to the kernel and we are almost there now. The
> >>best is to implement a regmap bus or a bus-less regmap instance that
> >>implements the IPC for the DSP. If the IPC does not map nicely onto a
> >>register map, we need to come up with something better. It's basically the
> >>same problem that that HDA CODEC by RT has.
No the register file _does_ not map to IPCs, as I have explained previosuly. The
file is just to store the informationa nd dpam widget handling triggers IPCs.
Since we are NOT writing to device, I still dont think regmap will help
> >Well, it depends. If the device looks like it has a register map that's
> >sensible but if it looks nothing like that then we should be
> >implementing the widgets differently. It may be that we need to be
> >constructing and destructing objects on the DSP for example.
>
> What do you mean by 'constructing/destructing on the DSP'?
I think Mark refers to modules and processing elements on DSP. Since this is DSP
(firmware) we can tell it to create a link b/w FE and BE, throw in another link
etc. Unlike codec, everything is driver controlled and instances created and
destroyed dynamically.
yes upating this to latest dapm (thanks to Lars for that), will help in moving
code away from driver
--
~Vinod
prev parent reply other threads:[~2014-06-26 10:32 UTC|newest]
Thread overview: 56+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-06-13 12:33 [PATCH 00/19]: add mrfld DSP support Vinod Koul
2014-06-13 12:33 ` [PATCH 01/19] ASoC: Intel: add COMPILE_TEST to mfld machine Vinod Koul
2014-06-21 10:42 ` Mark Brown
2014-06-21 15:34 ` Mark Brown
2014-06-21 16:08 ` Vinod Koul
2014-06-13 12:33 ` [PATCH 02/19] ASoC: Intel: mfld_pcm: move stream handling to dai_ops Vinod Koul
2014-06-21 10:43 ` Mark Brown
2014-06-13 12:33 ` [PATCH 03/19] ASoC: Intel: mfld-pcm rename period callback arg Vinod Koul
2014-06-21 10:43 ` Mark Brown
2014-06-13 12:33 ` [PATCH 04/19] ASoc: Intel: mfld-pcm: report pcm delay Vinod Koul
2014-06-21 10:43 ` Mark Brown
2014-06-13 12:33 ` [PATCH 05/19] ASoC: Intel: add the mrfld fw IPC definations Vinod Koul
2014-06-22 11:02 ` Mark Brown
2014-06-13 12:33 ` [PATCH 06/19] ASoC: Intel: mfld-pcm: modularize stream allocation code Vinod Koul
2014-06-22 11:02 ` Mark Brown
2014-06-13 12:33 ` [PATCH 07/19] ASoC: Intel: add mrfld pipelines Vinod Koul
2014-06-23 11:24 ` Mark Brown
2014-06-13 12:33 ` [PATCH 08/19] ASoC: Intel: use common stream allocation method for compressed stream Vinod Koul
2014-06-23 11:24 ` Mark Brown
2014-06-13 12:33 ` [PATCH 09/19] ASoC: Intel: mfld-pcm: add FE and BE ops Vinod Koul
2014-06-23 11:26 ` Mark Brown
2014-06-13 12:33 ` [PATCH 10/19] ASoC: Intel: add mrfld DSP registers Vinod Koul
2014-06-20 8:22 ` Lars-Peter Clausen
2014-06-20 11:32 ` Vinod Koul
2014-06-20 12:33 ` Lars-Peter Clausen
2014-06-21 6:22 ` Vinod Koul
2014-06-21 6:56 ` Lars-Peter Clausen
2014-06-23 4:27 ` Vinod Koul
2014-06-25 4:31 ` Lars-Peter Clausen
2014-06-26 10:22 ` Vinod Koul
2014-06-13 12:34 ` [PATCH 11/19] ASoC: intel: mfld-pcm: don't call trigger ops to DSP for internal streams Vinod Koul
2014-06-13 12:34 ` [PATCH 12/19] ASoC: Intel: add generic parameter set interface Vinod Koul
2014-06-13 12:34 ` [PATCH 13/19] ASoC: Intel: mrfld: add bytes control for modules Vinod Koul
2014-06-20 8:11 ` Lars-Peter Clausen
2014-06-20 11:30 ` Vinod Koul
2014-06-20 12:27 ` Lars-Peter Clausen
2014-06-21 6:16 ` Vinod Koul
2014-06-21 6:19 ` Lars-Peter Clausen
2014-06-23 4:15 ` Vinod Koul
2014-06-25 4:23 ` Lars-Peter Clausen
2014-06-13 12:34 ` [PATCH 14/19] ASoC: Intel: mrfld: add the gain controls Vinod Koul
2014-06-13 12:34 ` [PATCH 15/19] ASoC: Intel: mfld-pcm: add control for powering up/down dsp Vinod Koul
2014-06-13 12:34 ` [PATCH 16/19] ASoC: Intel: mrfld: add DSP core controls Vinod Koul
2014-06-13 12:34 ` [PATCH 17/19] ASoC: InteL; mrfld: add the DSP DAPM widgets Vinod Koul
2014-06-13 12:34 ` [PATCH 18/19] ASoC: Intel: mfld-pcm: add the fe & be dai ops Vinod Koul
2014-06-13 12:34 ` [RFC 19/19] ASoC: Intel: mrfld: add the DSP mixers Vinod Koul
2014-06-14 15:39 ` Lars-Peter Clausen
2014-07-04 4:46 ` Vinod Koul
2014-07-04 11:21 ` Lars-Peter Clausen
2014-07-07 8:45 ` Vinod Koul
2014-06-20 8:04 ` [PATCH 00/19]: add mrfld DSP support Lars-Peter Clausen
2014-06-20 8:20 ` Vinod Koul
2014-06-20 8:57 ` Lars-Peter Clausen
2014-06-23 10:09 ` Mark Brown
2014-06-25 4:27 ` Lars-Peter Clausen
2014-06-26 10:31 ` 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=20140626103147.GW22053@intel.com \
--to=vinod.koul@intel.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=lars@metafoo.de \
--cc=lgirdwood@gmail.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).