Re: [pulseaudio-discuss] [PATCH] module: new null-source module

2011-04-28 Thread Colin Guthrie
'Twas brillig, and Tanu Kaskinen at 27/04/11 12:59 did gyre and gimble: On Wed, 2011-04-27 at 14:50 +0300, Tanu Kaskinen wrote: From: Marc-André Lureau marcandre.lur...@gmail.com --- src/Makefile.am |6 + src/modules/module-null-source.c | 294

[pulseaudio-discuss] [PATCH] module: new null-source module

2011-04-27 Thread Tanu Kaskinen
From: Marc-André Lureau marcandre.lur...@gmail.com --- src/Makefile.am |6 + src/modules/module-null-source.c | 294 ++ 2 files changed, 300 insertions(+), 0 deletions(-) create mode 100644 src/modules/module-null-source.c diff --git

Re: [pulseaudio-discuss] [PATCH] module: new null-source module

2011-04-27 Thread Tanu Kaskinen
On Wed, 2011-04-27 at 14:50 +0300, Tanu Kaskinen wrote: From: Marc-André Lureau marcandre.lur...@gmail.com --- src/Makefile.am |6 + src/modules/module-null-source.c | 294 ++ 2 files changed, 300 insertions(+), 0 deletions(-)

Re: [pulseaudio-discuss] [PATCH] module: new null-source module

2011-04-27 Thread Colin Guthrie
'Twas brillig, and Tanu Kaskinen at 27/04/11 12:59 did gyre and gimble: On Wed, 2011-04-27 at 14:50 +0300, Tanu Kaskinen wrote: From: Marc-André Lureau marcandre.lur...@gmail.com --- src/Makefile.am |6 + src/modules/module-null-source.c | 294

Re: [pulseaudio-discuss] [PATCH] module: new null-source module

2011-04-27 Thread David Henningsson
On 2011-04-27 15:18, Colin Guthrie wrote: 'Twas brillig, and Tanu Kaskinen at 27/04/11 12:59 did gyre and gimble: On Wed, 2011-04-27 at 14:50 +0300, Tanu Kaskinen wrote: From: Marc-André Lureaumarcandre.lur...@gmail.com --- src/Makefile.am |6 +

Re: [pulseaudio-discuss] [PATCH] module: new null-source module

2011-04-27 Thread David Henningsson
On 2011-04-27 16:55, David Henningsson wrote: So I'll do some testing then grab it at some point (tho' need to spend some time reviewing Margarita's work before next week otherwise Liam and Mark will find some way to punish me!) FYI, I've done some review of that code already, and I agree with

Re: [pulseaudio-discuss] [PATCH] module: new null-source module

2011-04-27 Thread Mark Brown
On Wed, Apr 27, 2011 at 05:07:44PM +0200, David Henningsson wrote: Adding Marga and Liam, not cutting text as a result. On 2011-04-27 16:55, David Henningsson wrote: So I'll do some testing then grab it at some point (tho' need to spend some time reviewing Margarita's work before next week

Re: [pulseaudio-discuss] [PATCH] module: new null-source module

2011-04-27 Thread David Henningsson
On 2011-04-27 20:06, Mark Brown wrote: On Wed, Apr 27, 2011 at 05:07:44PM +0200, David Henningsson wrote: Adding Marga and Liam, not cutting text as a result. On 2011-04-27 16:55, David Henningsson wrote: So I'll do some testing then grab it at some point (tho' need to spend some time

Re: [pulseaudio-discuss] [PATCH] module: new null-source module

2011-04-27 Thread Mark Brown
On Wed, Apr 27, 2011 at 08:29:34PM +0200, David Henningsson wrote: On 2011-04-27 20:06, Mark Brown wrote: You're not really supposed to use alsaucm directly except for testing and developing new configurations, That's reason enough to have a man page and some documentation, if you ask me.

Re: [pulseaudio-discuss] [PATCH] module: new null-source module

2011-04-27 Thread Liam Girdwood
On Wed, 2011-04-27 at 20:29 +0200, David Henningsson wrote: The overall problem IMHO is that it is overly complex: 1) The kernel might set some basic volumes 2) Alsactl can store/restore volumes 3) Alsaucm can set volumes 4) PulseAudio (for the gdm user) can store/restore volumes 5)

Re: [pulseaudio-discuss] [PATCH] module: new null-source module

2011-04-27 Thread Mark Brown
On Wed, Apr 27, 2011 at 08:04:06PM +0100, Liam Girdwood wrote: On Wed, 2011-04-27 at 20:29 +0200, David Henningsson wrote: The patches basically provide the ability to :- 1) Store UCM verb and devices configuration in proplists. 2) Allow pulseaudio to change the UCM usecase verb and device.