From: Andre Guedes <andre.guedes@intel.com>
To: alsa-devel@alsa-project.org
Cc: tiwai@suse.de, pierre-louis.bossart@linux.intel.com,
liam.r.girdwood@intel.com
Subject: [PATCH v2 - AAF PCM plugin 0/7] Introduce AVTP Audio Format (AAF) plugin
Date: Wed, 24 Oct 2018 18:11:09 -0700 [thread overview]
Message-ID: <20181025011116.12360-1-andre.guedes@intel.com> (raw)
Hi all,
This new version addresses the feedback from v1 and fixes a bug on capture
mode. Code-wise, the changes are very trivial, no major diffs. Below follows
the highlights:
* Standard headers like linux/if_ether.h are now checked by configure script.
* The 'static' modifier was added to array variables in aaf_hw_constraint().
* aaf_close() callback was simplified.
* aaf_rx_pdu() was modified so it doesn't return an error in case the number
of bytes read by recv() is different from aaf->pdu_size. This is an
expected situation and it means the received AVTPDU doesn't belong to the
AAF stream the plugin is interested in so it should simply drop that
AVTPDU. This fixes a bug when running multiple streams (with different
AVPTU sizes) concurrently.
Additionally, this new version improves the instructions in doc/aaf.txt in
order to make it easier to test the plugin. Anyone should be able to test it
just by copying and pasting the instructions.
To recap what has been discussed about the plugin so far, check the comments in
RFC and PATCH series archive [1-3]. All versions of this series can be also
found in my alsa-plugins tree in github [4].
Regards,
Andre
[1] http://mailman.alsa-project.org/pipermail/alsa-devel/2018-August/139494.html
[2] http://mailman.alsa-project.org/pipermail/alsa-devel/2018-September/140290.html
[3] http://mailman.alsa-project.org/pipermail/alsa-devel/2018-October/140667.html
[4] https://github.com/aguedes/alsa-plugins
Andre Guedes (7):
aaf: Introduce plugin skeleton
aaf: Load configuration parameters
aaf: Implement Playback mode support
aaf: Prepare for Capture mode support
aaf: Implement Capture mode support
aaf: Implement dump() ioplug callback
aaf: Add support for direct read/write transfers
Makefile.am | 3 +
aaf/Makefile.am | 9 +
aaf/pcm_aaf.c | 1200 +++++++++++++++++++++++++++++++++++++++++++++++++++++++
configure.ac | 11 +
doc/aaf.txt | 157 ++++++++
5 files changed, 1380 insertions(+)
create mode 100644 aaf/Makefile.am
create mode 100644 aaf/pcm_aaf.c
create mode 100644 doc/aaf.txt
--
2.14.4
next reply other threads:[~2018-10-25 1:11 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-10-25 1:11 Andre Guedes [this message]
2018-10-25 1:11 ` [PATCH v2 - AAF PCM plugin 1/7] aaf: Introduce plugin skeleton Andre Guedes
2018-10-25 1:11 ` [PATCH v2 - AAF PCM plugin 2/7] aaf: Load configuration parameters Andre Guedes
2018-10-25 1:11 ` [PATCH v2 - AAF PCM plugin 3/7] aaf: Implement Playback mode support Andre Guedes
2018-10-25 1:11 ` [PATCH v2 - AAF PCM plugin 4/7] aaf: Prepare for Capture " Andre Guedes
2018-10-25 1:11 ` [PATCH v2 - AAF PCM plugin 5/7] aaf: Implement " Andre Guedes
2018-10-25 1:11 ` [PATCH v2 - AAF PCM plugin 6/7] aaf: Implement dump() ioplug callback Andre Guedes
2018-10-25 1:11 ` [PATCH v2 - AAF PCM plugin 7/7] aaf: Add support for direct read/write transfers Andre Guedes
2018-10-25 6:41 ` [PATCH v2 - AAF PCM plugin 0/7] Introduce AVTP Audio Format (AAF) plugin Jaroslav Kysela
2018-10-25 6:51 ` github repo (Re: [PATCH v2 - AAF PCM plugin 0/7] Introduce AVTP Audio Format (AAF) plugin) Takashi Iwai
2018-10-25 7:36 ` Jaroslav Kysela
2018-10-25 7:42 ` Takashi Iwai
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=20181025011116.12360-1-andre.guedes@intel.com \
--to=andre.guedes@intel.com \
--cc=alsa-devel@alsa-project.org \
--cc=liam.r.girdwood@intel.com \
--cc=pierre-louis.bossart@linux.intel.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.