From: Colin Guthrie <gmane@colin.guthr.ie>
To: alsa-devel@alsa-project.org
Subject: Re: [PATCH] Per Application Volume Control plugin for ALSA
Date: Sun, 17 Oct 2010 12:21:21 +0100 [thread overview]
Message-ID: <i9em7h$kdf$2@dough.gmane.org> (raw)
In-Reply-To: <s5hsk05urj6.wl%tiwai@suse.de>
'Twas brillig, and Takashi Iwai at 17/10/10 09:55 did gyre and gimble:
> At Wed, 29 Sep 2010 18:56:59 +0200,
> James Huk wrote:
>>
>> Hello everybody.
>>
>> This is my first "post" here, so please be "gentle".
>>
>> I have created PAVC plugin for ALSA, it is based on COPY plugin, and
>> use SOFTVOL plugin for sound output. As far as I can tell from current
>> tests - It works perfectly, and it doesn't break compatibility with
>> any apps I use (and PulseAudio does). So I was wondering - is there
>> any chance this plugin would be added to the next official ALSA
>> release?
>>
>> Here is the patch that enables it on current alsa-lib-1.0.23
>> (hopefully this mailing list is a good place to "post" this):
>
> Thanks for the patch. I looked over this post until now.
>
> The idea is interesting, and I thought of a similar thing before PA
> gets popularized, too.
>
> Though, this implementation is a bit too hackish, especially the part
> using pipe & co. If we have a more simpler and cleaner way for this,
> it's worth including to the standard plugin, I think.
I was thinking much the same thing when I read over this patch a few
weeks ago.
While it could be interesting to include this in alsa, I struggled to
think of a use case where PA would not be more appropriate.
Is there a specific use case targeted with this patch or is it really
just a "proof of concept" demonstration that alsa can do this sans PA?
Col
--
Colin Guthrie
gmane(at)colin.guthr.ie
http://colin.guthr.ie/
Day Job:
Tribalogic Limited [http://www.tribalogic.net/]
Open Source:
Mageia Contributor [http://www.mageia.org/]
PulseAudio Hacker [http://www.pulseaudio.org/]
Trac Hacker [http://trac.edgewall.org/]
next prev parent reply other threads:[~2010-10-17 11:25 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-09-29 16:56 [PATCH] Per Application Volume Control plugin for ALSA James Huk
2010-10-17 8:55 ` Takashi Iwai
2010-10-17 11:20 ` Raymond Yau
2010-10-17 11:21 ` Colin Guthrie [this message]
2010-10-17 21:54 ` James Huk
2010-10-18 1:48 ` Raymond Yau
2010-10-18 12:14 ` Colin Guthrie
2010-10-18 21:21 ` James Huk
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='i9em7h$kdf$2@dough.gmane.org' \
--to=gmane@colin.guthr.ie \
--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.