Alsa-Devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Lennart Poettering <mznyfn@0pointer.de>
To: ALSA Development Mailing List <alsa-devel@alsa-project.org>
Subject: Pull request for alsa-plugins
Date: Wed, 3 Sep 2008 21:17:41 +0200	[thread overview]
Message-ID: <20080903191741.GA13536@tango.0pointer.de> (raw)

Takashi, Jaroslav,

please pull and merge a series of 26 patches for alsa-plugins'
PulseAudio driver which I prepared in this repository of mine:

    git://git.0pointer.de/alsa-plugins.git

gitweb is here:

    http://git.0pointer.de/?p=alsa-plugins.git

There are quite a few bug fixes and feature enhancements among those
patches. It also includes some clean-ups in the coding
style. i.e. previously the indenting of the PA driver was pure chaos,
every single patch seemed to introduce a new indentation width. The
first patch in this series simply reindents the driver to Linux
style. All other patches are on top of that one.

Here's the shortlog:

Lennart Poettering (26):
      Reindent to Linux kernel style
      Add Emacs-style /*-*- linux-c -*-*/ header comment
      Make pulse_new() a proper C function
      Don't modify the SIGPIPE handler
      Call pa_context_disconnect() explicitly
      use SNDERR instead of fprintf to print error messages
      Support S32 sample types
      Add trailing NUL character to snprintf output
      Get rid of pulse_poll_revents()
      Add more error checking
      Remove fix for bug 0003470
      Rework hardware parameter selection
      A bag of minor clean ups for ctl_pulse.c
      Make pulse_ext_callback const
      Drop our own implementation of the poll() callbacks
      A bag of minor clean-ups for pulse.c
      Split out O_NONBLOCK setting into seperate function
      Save a byte of memory
      Adjust buffering metrics to match what PA internally uses
      Make sure we always have a sensible channel mapping
      Use PA_STREAM_EARLY_REQUESTS if available
      Use S32/FLOAT32 only where available in the PA libs
      Add const to our snd_pcm_ioplug_callback_t instances
      Don't implement our own poll handlers, we can use the default
      ones
      Remove our own poll handler implementation entirely
      A bag of clean-ups for pcm_ctl.c

Those patches only touch the pulse/ subdir. 

The tree is freshly rebased against current alsa-plugins master.

May I ask you to make me the "semi-official" maintainer of this
driver? I.e. I'd like to be consulted (as in 'Signed-Off-by') before
any patches for it are merged?

If requested I can also mail this as series of 26 seperate patches.

Thank you,

Lennart

-- 
Lennart Poettering                        Red Hat, Inc.
lennart [at] poettering [dot] net         ICQ# 11060553
http://0pointer.net/lennart/           GnuPG 0x1A015CC4

             reply	other threads:[~2008-09-03 19:17 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-09-03 19:17 Lennart Poettering [this message]
2008-09-04  7:48 ` Pull request for alsa-plugins Takashi Iwai
2008-09-04 10:23   ` Colin Guthrie
2008-09-04 14:03   ` Lennart Poettering

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=20080903191741.GA13536@tango.0pointer.de \
    --to=mznyfn@0pointer.de \
    --cc=alsa-devel@alsa-project.org \
    /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