Re: [pulseaudio-discuss] [PATCH 0/3] Multichannel fixes

2015-04-02 Thread David Henningsson
On 2015-04-02 00:24, Takashi Sakamoto wrote: Hi David, On May 31 2015 21:38, David Henningsson wrote: About the first patch: I tested and refined a patch previously submitted by Takashi Sakamoto. It seems like nothing has happened on the udev side so I suggest we take this one for now. In ca

Re: [pulseaudio-discuss] [PATCH 0/3] Multichannel fixes

2015-04-02 Thread Raymond Yau
> > About the first patch: > > > > I tested and refined a patch previously submitted by Takashi > > Sakamoto. It seems like nothing has happened on the udev side so I > > suggest we take this one for now. In case udev should improve and get a > > model database for the Firewire bus, then we could c

Re: [pulseaudio-discuss] [PATCH] pactl: Fix relative volume parsing

2015-04-02 Thread Tanu Kaskinen
On Thu, 2015-04-02 at 11:37 +0530, Arun Raghavan wrote: > On 1 April 2015 at 22:10, Tanu Kaskinen wrote: > > The recent change to reject leading '+' in pa_atod() broke the > > relative volume parsing in pactl. This fixes that. > > --- > > I'm wondering if we should just have pa_atod() deal with l

Re: [pulseaudio-discuss] Patch review status wiki page updated

2015-04-02 Thread Tanu Kaskinen
On Thu, 2015-04-02 at 07:09 +0900, Takashi Sakamoto wrote: > Hi Tanu, > > On May 24 2015 17:01, Tanu Kaskinen wrote: > > Patch review status updated: > > http://www.freedesktop.org/wiki/Software/PulseAudio/PatchStatus/ > > > pulseaudio: udev: use ID_MODEL/ID_VENDOR to give friendly name for > > F

Re: [pulseaudio-discuss] [PATCH 07/19] core: Add hooks for default sink and source changes

2015-04-02 Thread David Henningsson
7/19 and 8/19 are both acked, and in fact identical to my patch http://lists.freedesktop.org/archives/pulseaudio-discuss/2015-March/023334.html On 2015-03-19 12:50, Juho Hämäläinen wrote: --- src/pulsecore/core.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/pulsecore/core.h b/sr

Re: [pulseaudio-discuss] [PATCH 09/19] dbus: Use hooks for default sink and source changes

2015-04-02 Thread David Henningsson
On 2015-03-19 12:50, Juho Hämäläinen wrote: --- src/modules/dbus/iface-core.c | 150 +- 1 file changed, 88 insertions(+), 62 deletions(-) diff --git a/src/modules/dbus/iface-core.c b/src/modules/dbus/iface-core.c index 3d58754..9555703 100644 --- a/sr

Re: [pulseaudio-discuss] [PATCH 03/19] dbus: Use hooks for sink and source events

2015-04-02 Thread David Henningsson
Acked. On 2015-03-19 12:50, Juho Hämäläinen wrote: --- src/modules/dbus/iface-device.c | 121 ++-- 1 file changed, 91 insertions(+), 30 deletions(-) diff --git a/src/modules/dbus/iface-device.c b/src/modules/dbus/iface-device.c index efa4141..2c370a8 10064

Re: [pulseaudio-discuss] [PATCH 02/19] dbus: Use hooks for client proplist changes

2015-04-02 Thread David Henningsson
On 2015-03-19 12:50, Juho Hämäläinen wrote: --- src/modules/dbus/iface-client.c | 34 +- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/src/modules/dbus/iface-client.c b/src/modules/dbus/iface-client.c index 76ad427..455ea45 100644 --- a/src/mo

Re: [pulseaudio-discuss] [PATCH 00/19] Use hooks instead of subscription for module-dbus-protocol

2015-04-02 Thread David Henningsson
Hi! I've reviewed and pushed patch 1 - 9 now. Everything looked good so far except a small fix I did in patch 9 before pushing. On 2015-03-19 12:50, Juho Hämäläinen wrote: Due to problems with asynchronous subscription system used in the module-dbus-protocol, replace all subscription system u

[pulseaudio-discuss] Sink's Default Sample Rate is Changing?

2015-04-02 Thread Andrew Kelley
I discovered a curiosity: $ play ~/tmp/44100hz.wav $ ./list_devices playback device: 44100 Hz Built-in Audio Analog Stereo (default) $ play ~/tmp/48000hz.wav $ ./list_devices playback device: 48000 Hz Built-in Audio Analog Stereo (default) $ play ~/tmp/9hz.wav playback device: 44100 Hz Built-i

Re: [pulseaudio-discuss] Sink's Default Sample Rate is Changing?

2015-04-02 Thread Alexander E. Patrakov
03.04.2015 00:36, Andrew Kelley wrote: I discovered a curiosity: $ play ~/tmp/44100hz.wav $ ./list_devices playback device: 44100 Hz Built-in Audio Analog Stereo (default) $ play ~/tmp/48000hz.wav $ ./list_devices playback device: 48000 Hz Built-in Audio Analog Stereo (default) $ play ~/tmp/

Re: [pulseaudio-discuss] ThinkPad T-510 audio output mute LED non-workingness

2015-04-02 Thread Glenn Golden
David Henningsson [2015-03-31 08:43:04 +0200]: > > On 2015-03-31 05:51, Hui Wang wrote: > > According to my understanding, it should be the former one, PA tells > > the driver to "mute the audio", and the driver sets the LED accordingly. > > Yes, this is correct. More specific, the mic mute LED