All of 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 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.