From: Vinod Koul <vinod.koul@intel.com>
To: Qais Yousef <qais.yousef@imgtec.com>
Cc: alsa-devel@alsa-project.org, Lars-Peter Clausen <lars@metafoo.de>,
Takashi Iwai <tiwai@suse.de>,
Clemens Ladisch <clemens@ladisch.de>,
Neil Jones <neil.jones@imgtec.com>,
Liam Girdwood <lgirdwood@gmail.com>,
Mark Brown <broonie@kernel.org>
Subject: Re: [RFC] AXD Audio Processing IP ALSA support - Questions
Date: Tue, 4 Nov 2014 16:10:02 +0530 [thread overview]
Message-ID: <20141104104002.GH1870@intel.com> (raw)
In-Reply-To: <5458A0E2.4010602@imgtec.com>
On Tue, Nov 04, 2014 at 09:48:18AM +0000, Qais Yousef wrote:
> Hi,
>
> I have several questions on the best way to add AXD support in ALSA.
1st rule pls CC maintainers, so that it gets rights attention.
> The discussion of the previous patch can be found here:
>
> https://lkml.org/lkml/2014/10/28/465
>
> Questions:
>
> 1- What is the best example to follow to add a simple mp3 support
> for AXD? The only one I find is in sst-mfld-platform-compress.c in
> sound/soc/intel directory but it's a bit confusing. I think because
> it's sharing code with several other sst drivers/platforms.
There are two ways
1. If you are a ASoC driver which is most likely the case, then add a
compress dai and then a compress dai-link. The device with compress device
will be created.
2. Directly call compress_register the way asoc does
For both you need to implement the compressed ops
> I find the documentation for compress_offload generally lacking. Is
> there a plan to improve on that? Being a new comer to ALSA framework
> api, I'm confused what is the correct way to do things :-/
Are you talking about kernel API or driver API? Can you please elaborate
>
> So far I know I need to call snd_soc_register_card(). I thought
> snd_compress_register() (from compress_driver.h) is how you add
> compressed nodes to the card but apparently not. It looks like I
> need to define a compress_dai? Hmmm.
You need to define a compress_dai if you are a asoc device just like the pcm
dai's, it is similar to what you would need to do for PCM
> 2- Is tinycompress the only userland support for compress_offload?
> Is there anyone working on gstreamer and omx plugins to support
> that?
Yes, I don't know of anyone working on omx support.
>
> Would tinycompress be part of alsa-utils and alsa-libs in the
> future? I know it needs more work at the moment but it'd be nice if
> compress_offload support is part of the standard alsa-utils and
> alsa-libs.
It is alsa-lib, for packaging we can make it part of alsa packages. Most
users are right now in Android so no one asked yet
> 3- Can we get an example of how transcoding (back to disk) is
> supposed to be working?
As I have replied to you last week, it would be done using two FEs and these
FEs should be "routed"
>
> 4- How can we reconfigure complex audio effect components (like
> shelving filters) which need filter co-effecient changes to be
> applied all at once atomically to avoid instability?
Add an ALSA control which models sync, then in driver apply once you
get sync control
--
~Vinod
> My plan is to add mp3 driver first then build on that to add the
> rest of the features. Hopefully I can minimise the amount of work
> that needs doing (due to me doing it wrongly) and get features
> upstream sooner.
>
> Thanks,
> Qais
--
next prev parent reply other threads:[~2014-11-04 10:39 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-11-04 9:48 [RFC] AXD Audio Processing IP ALSA support - Questions Qais Yousef
2014-11-04 9:58 ` Clemens Ladisch
2014-11-04 10:40 ` Vinod Koul
2014-11-04 10:40 ` Vinod Koul [this message]
2014-11-04 12:04 ` Qais Yousef
2014-11-04 16:14 ` Pierre-Louis Bossart
2014-11-04 17:41 ` Mark Brown
2014-11-05 8:24 ` Qais Yousef
2014-11-05 8:38 ` Vinod Koul
2014-11-05 8:56 ` Qais Yousef
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=20141104104002.GH1870@intel.com \
--to=vinod.koul@intel.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=clemens@ladisch.de \
--cc=lars@metafoo.de \
--cc=lgirdwood@gmail.com \
--cc=neil.jones@imgtec.com \
--cc=qais.yousef@imgtec.com \
--cc=tiwai@suse.de \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.