All of lore.kernel.org
 help / color / mirror / Atom feed
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

             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.