Re: [pulseaudio-discuss] [PACKAGERS] New dep on json-c

2011-05-18 Thread Paul Menzel
Dear Colin,


Am Montag, den 16.05.2011, 11:56 +0100 schrieb Colin Guthrie:

 I've just pushed Arun's (mostly, Pierre-Louis also had a hand!)
 passthrough work to master.
 
 This carries with it a new dependency: json-c
 
 We may yet remove this and write our own parser for the simple subset of
 JSON formatting we use but there may ultimately be other reasons to keep
 this longer term.
 
 But in the interests of simplicity, I'd certainly not be against any
 native implementation patches that came along provided they were simple
 and clean.
 
 Please test these changes. I'm sure Arun will post more details and test
 requests in due course.

looking at the XBMC commit log [1] they seem to move from JsonCpp [2] to
YAJL [3].


Thanks,

Paul


[1] 
https://github.com/xbmc/xbmc/commit/56ac2a40cc3d721d428f026bfe11173b27596fe3/
[2] http://lloyd.github.com/yajl/
[3] http://jsoncpp.sourceforge.net/


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


[pulseaudio-discuss] unambiguous tag names in the repository

2011-05-07 Thread Paul Menzel
Dear PulseAudio folks,


I just read the blog post

Please use unambiguous tag names in your DVCS [1]

which for security reasons suggests to use unambiguous tag names.

Therefore I suggest to change our tag name scheme to the following.

pa_0.9.22 or pa_1.0


Thanks,

Paul


[1] http://www.debian-administration.org/users/dkg/weblog/81


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] ORC buildsystem problems

2011-05-03 Thread Paul Menzel
Am Dienstag, den 03.05.2011, 08:39 +0530 schrieb Arun Raghavan:
 On Mon, 2011-05-02 at 11:52 +0200, Maarten Bosmans wrote:
  Recently, I encountered some problems when enabling orc in some less
  usual situations.
  
  When compiling with --enable-orc from a tarball generated from a
  --disable-orc configured tree, the following error occurs.
  make[2]: *** No rule to make target `pulsecore/svolume-orc-gen.c',
  needed by `all'.  Stop
  I haven't really looked at a solution. Perhaps the nodist_ prefix for
  some files inside if HAVE_ORC in src/Makefile.am should be dropped?
  May be Colin needs to do his make distchecks with this situation, in
  order to catch it earlier.
 
 There should not be a dist'ed tarball without those generated files.
 Whether they are used or not is a configure-time option, then.
 
  Secondly, there is a problem when cross-compiling. The pkg-config
  check for ORC is used to find the usual include files and linking
  flags, but also to find the location of orcc. This is a problem,
  because when configure is run with the correct configuration, such
  that pkg-config finds the host package, it also finds the host orcc
  (in the case of my mingw32 test, it finds orcc.exe), which is of
  course useless in the build environment.
 
 Why is it useless in your environment? The files generated by orcc are
 architecture-neutral.

For your second issue please take a look at the thread on the
gstreamer-devel list [1].


Thanks,

Paul


[1] http://lists.freedesktop.org/archives/gstreamer-devel/2011-March/030983.html


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] ORC buildsystem problems

2011-05-03 Thread Paul Menzel
Am Dienstag, den 03.05.2011, 10:40 +0200 schrieb Maarten Bosmans:
 2011/5/3 Paul Menzel paulepan...@users.sourceforge.net:
  Am Dienstag, den 03.05.2011, 08:39 +0530 schrieb Arun Raghavan:
  On Mon, 2011-05-02 at 11:52 +0200, Maarten Bosmans wrote:

[…]

   Secondly, there is a problem when cross-compiling. The pkg-config
   check for ORC is used to find the usual include files and linking
   flags, but also to find the location of orcc. This is a problem,
   because when configure is run with the correct configuration, such
   that pkg-config finds the host package, it also finds the host orcc
   (in the case of my mingw32 test, it finds orcc.exe), which is of
   course useless in the build environment.
 
  Why is it useless in your environment? The files generated by orcc are
  architecture-neutral.
 
 The binary that make ends up trying to use (orcc.exe) is useless,
 because it's a win32 binary on a Linux platform. So the
 architecture-neutral files can't even be generated.
 
  For your second issue please take a look at the thread on the
  gstreamer-devel list [1].
 
 Ah, yes. It seems that my workaround of setting ORCC= is the advised
 solution. Still it would be better to autodetect a broken return value
 of pkg-config. I could make a patch to orc.m4 that does just that, but
 I can't seem to find the file in the orc git repo. Is orc.m4 managed
 locally in the pulse tree, or is there some upstream that improvements
 should also be submitted to?

As far as I know `orc.m4` is managed locally by all projects depending
on Orc like PulseAudio, Gstreamer or Schroedinger. I guess you or I (if
your are not subscribed) could forward your patch to the gstreamer-devel
list.


Thanks,

Paul


  [1] 
  http://lists.freedesktop.org/archives/gstreamer-devel/2011-March/030983.html


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


[pulseaudio-discuss] [PATCH] .gitignore: add `.tarball-version`

2011-03-29 Thread Paul Menzel
Date: Tue, 29 Mar 2011 11:49:52 +0200

`.tarball-version` is created by `.git-version-gen`.

Signed-off-by: Paul Menzel paulepan...@users.sourceforge.net
---
 .gitignore |1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/.gitignore b/.gitignore
index 3a840d9..eb33e91 100644
--- a/.gitignore
+++ b/.gitignore
@@ -1,3 +1,4 @@
+.tarball-version
 .version
 .*.swp
 ABOUT-NLS
-- 
1.7.4.1



signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


[pulseaudio-discuss] [PATCH] bluetooth: run `make update-sbc` to pull in build fix for thumb mode

2011-03-29 Thread Paul Menzel
Date: Tue, 29 Mar 2011 12:14:27 +0200

This update pulls in commit c495077c [1] to fix a build error.

commit c495077cf8a8c37afd90875ec5a5b16b294be15e
Author: Siarhei Siamashka siarhei.siamas...@nokia.com
Date:   Tue Mar 29 01:57:39 2011 +0300

sbc: better compatibility with ARM thumb/thumb2

ARM assembly optimizations fail to compile in thumb mode, but are 
fine
for thumb2. Update ifdefs in the code to make use of ARM assembly 
only
when it is safe and also make sure that no optimizations are missed
when compiling for thumb2.

The problem was reported by Paul Menzel:

https://tango.0pointer.de/pipermail/pulseaudio-discuss/2011-February/009022.html

This patch is tested with OpenEmbedded using `minimal-uclibc` for `MACHINE = 
at91sam9260ek`.

Note that changes to ipc.h from 8f3ef04b had to be manually reapplied.

[1] 
http://git.kernel.org/?p=bluetooth/bluez.git;a=commit;h=c495077cf8a8c37afd90875ec5a5b16b294be15e

Signed-off-by: Paul Menzel paulepan...@users.sourceforge.net
CC: Luiz Augusto von Dentz luiz.de...@gmail.com
CC: Arun Raghavan arun.ragha...@collabora.co.uk
---
Please update the commit summary to something that is technically correct.
---
 src/modules/bluetooth/sbc/sbc_math.h |2 +-
 src/modules/bluetooth/sbc/sbc_primitives_armv6.h |4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/modules/bluetooth/sbc/sbc_math.h 
b/src/modules/bluetooth/sbc/sbc_math.h
index 9f126c6..5476860 100644
--- a/src/modules/bluetooth/sbc/sbc_math.h
+++ b/src/modules/bluetooth/sbc/sbc_math.h
@@ -48,7 +48,7 @@ typedef int32_t sbc_fixed_t;
 
 #define SBC_FIXED_0(val) { val = 0; }
 #define MUL(a, b)((a) * (b))
-#ifdef __arm__
+#if defined(__arm__)  (!defined(__thumb__) || defined(__thumb2__))
 #define MULA(a, b, res) ({ \
int tmp = res;  \
__asm__(\
diff --git a/src/modules/bluetooth/sbc/sbc_primitives_armv6.h 
b/src/modules/bluetooth/sbc/sbc_primitives_armv6.h
index 1862aed..6a9efe5 100644
--- a/src/modules/bluetooth/sbc/sbc_primitives_armv6.h
+++ b/src/modules/bluetooth/sbc/sbc_primitives_armv6.h
@@ -40,8 +40,8 @@
 
 #if !defined(SBC_HIGH_PRECISION)  (SCALE_OUT_BITS == 15)  \
defined(__GNUC__)  defined(SBC_HAVE_ARMV6)  \
-   defined(__ARM_EABI__)  !defined(__thumb__)  \
-   !defined(__ARM_NEON__)
+   defined(__ARM_EABI__)  !defined(__ARM_NEON__)  \
+   (!defined(__thumb__) || defined(__thumb2__))
 
 #define SBC_BUILD_WITH_ARMV6_SUPPORT
 
-- 
1.7.4.1


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] [PATCH] sbc_math.h: add explicit check for ARMv6 instructions

2011-03-28 Thread Paul Menzel
Dear BlueZ folks,


Am Montag, den 28.03.2011, 18:41 +0300 schrieb Luiz Augusto von Dentz:

 On Fri, Mar 25, 2011 at 1:34 PM, Paul Menzel wrote:

  Am Samstag, den 19.03.2011, 16:14 +0530 schrieb Arun Raghavan:
 
  On Wed, 2011-02-23 at 01:07 +0530, Arun Raghavan wrote:
  [...]
   The correct fix for this, imo, is in bluez (there is a new
   sbc_primitives_armv6.h that can probably be used at least as a
   template). We need to do an sbc-udpate on the PA side anyway, and can
   pull this when we do.
 
  Could you see if the attached patch works for you? If it does, I can
  push this to the bluez folks.
 
  My tag on the other thread [1] is ambiguous. The error is indeed caused
  by your patch. I will try to implement your recommended changes but will
  probably not get to it before Sunday.
 
 Im including Siarhei Siamashka since he did most, if not all, of this
 code and linux-bluetooth to cc.

to fix [1] I tried to add `#include sbc_tables.h` to
`src/modules/bluetooth/sbc/sbc_primitives.h`

diff --git a/src/modules/bluetooth/sbc/sbc_primitives.h 
b/src/modules/bluetooth/
index 3fec8d5..9544826 100644
--- a/src/modules/bluetooth/sbc/sbc_primitives.h
+++ b/src/modules/bluetooth/sbc/sbc_primitives.h
@@ -24,6 +24,8 @@
  *
  */
 
+#include sbc_tables.h
+
 #ifndef __SBC_PRIMITIVES_H
 #define __SBC_PRIMITIVES_H

But this seems to result in a need for `sbc_math.h` which leads to a
circular inclusion.

[…]
In file included from modules/bluetooth/sbc/sbc_primitives.h:27:0,
 from modules/bluetooth/sbc/sbc_primitives_armv6.h:30,
 from modules/bluetooth/sbc/sbc_math.h:27,
 from modules/bluetooth/sbc/sbc.c:46:
modules/bluetooth/sbc/sbc_tables.h:50:2: warning: implicit declaration 
of function 'ASR' [-Wimplicit-function-declaration]
modules/bluetooth/sbc/sbc_tables.h:50:2: error: 'SCALE_SPROTO4_TBL' 
undeclared here (not in a function)
modules/bluetooth/sbc/sbc_tables.h:58:2: error: initializer element is 
not constant
modules/bluetooth/sbc/sbc_tables.h:58:2: error: (near initialization 
for 'sbc_proto_4_40m1[0]')
modules/bluetooth/sbc/sbc_tables.h:58:2: error: initializer element is 
not constant
modules/bluetooth/sbc/sbc_tables.h:58:2: error: (near initialization 
for 'sbc_proto_4_40m1[1]')
modules/bluetooth/sbc/sbc_tables.h:58:2: error: initializer element is 
not constant
modules/bluetooth/sbc/sbc_tables.h:58:2: error: (near initialization 
for 'sbc_proto_4_40m1[2]')
modules/bluetooth/sbc/sbc_tables.h:58:2: error: initializer element is 
not constant
modules/bluetooth/sbc/sbc_tables.h:58:2: error: (near initialization 
for 'sbc_proto_4_40m1[3]')
modules/bluetooth/sbc/sbc_tables.h:59:2: error: initializer element is 
not constant
modules/bluetooth/sbc/sbc_tables.h:59:2: error: (near initialization 
for 'sbc_proto_4_40m1[4]')
modules/bluetooth/sbc/sbc_tables.h:59:2: error: initializer element is 
not constant
modules/bluetooth/sbc/sbc_tables.h:59:2: error: (near initialization 
for 'sbc_proto_4_40m1[5]')
modules/bluetooth/sbc/sbc_tables.h:59:2: error: initializer element is 
not constant
modules/bluetooth/sbc/sbc_tables.h:59:2: error: (near initialization 
for 'sbc_proto_4_40m1[6]')
modules/bluetooth/sbc/sbc_tables.h:59:2: error: initializer element is 
not constant
modules/bluetooth/sbc/sbc_tables.h:59:2: error: (near initialization 
for 'sbc_proto_4_40m1[7]')
modules/bluetooth/sbc/sbc_tables.h:60:2: error: initializer element is 
not constant
modules/bluetooth/sbc/sbc_tables.h:60:2: error: (near initialization 
for 'sbc_proto_4_40m1[8]')
modules/bluetooth/sbc/sbc_tables.h:60:2: error: initializer element is 
not constant
modules/bluetooth/sbc/sbc_tables.h:60:2: error: (near initialization 
for 'sbc_proto_4_40m1[9]')
modules/bluetooth/sbc/sbc_tables.h:60:2: error: initializer element is 
not constant
modules/bluetooth/sbc/sbc_tables.h:60:2: error: (near initialization 
for 'sbc_proto_4_40m1[10]')
modules/bluetooth/sbc/sbc_tables.h:60:2: error: initializer element is 
not constant
modules/bluetooth/sbc/sbc_tables.h:60:2: error: (near initialization 
for 'sbc_proto_4_40m1[11]')
modules/bluetooth/sbc/sbc_tables.h:61:2: error: initializer element is 
not constant
modules/bluetooth/sbc/sbc_tables.h:61:2: error: (near initialization 
for 'sbc_proto_4_40m1[12]')
modules/bluetooth/sbc/sbc_tables.h:61:2: error: initializer element is 
not constant
modules/bluetooth/sbc/sbc_tables.h:61:2: error: (near initialization 
for 'sbc_proto_4_40m1[13]')
modules/bluetooth/sbc/sbc_tables.h:61:2: error: initializer element is 
not constant
modules/bluetooth/sbc/sbc_tables.h:61:2: error: (near

Re: [pulseaudio-discuss] svolume_orc.c: error: line 67: unknown directive: .longparam

2011-03-25 Thread Paul Menzel
Am Donnerstag, den 24.03.2011, 11:56 +0530 schrieb Arun Raghavan:
 On Mon, 2011-03-21 at 00:06 +0100, Paul Menzel wrote:

 [...]
  I guess it has something to do with
  
  commit 4cd90d9e32ca9a23e3c0f7615974ea0c55ff3e49
  Author: Arun Raghavan arun.ragha...@collabora.co.uk
  Date:   Mon Oct 25 17:59:08 2010 +0100
  
  volume: Add Orc-based optimised volume scaling
  
  This adds volume scaling for 1- and 2-channel software volume 
  scaling
  using Orc. While testing the MMX and SSE backends on a Core2, I 
  see an
  ~2x performance benefit over the hand-rolled MMX and SSE code. 
  Since I
  haven't been able to test on other architectures, the Orc code 
  is only
  used when MMX/SSE* is present. This can be changed in the 
  future after
  testing on AMD and ARM machines.
  
  but I do not know anything about this.
  
  I am using OpenEmbedded with `minimal` or `minimal-uclibc` for `MACHINE
  = at91sam9260ek`. ORCC 0.4.9 is used on this system.
 
 Could you try with Orc 0.4.10? Unfortunately, I don't have a quick way
 to downgrade my local Orc version to verify?

I verified that it works with Orc 0.4.10.

 If this is not possible, we
 can just bump the patch to 0.4.11, which was released a while ago (and
 is what I'm using).

You send the patch today and it was already pushed in f49711c9 [1].

 BTW, for 0.9.22 and current stable-queue, the Orc stuff will not get
 used for anything on ARM.

Understood.


Thanks,

Paul


[1] 
http://git.0pointer.de/?p=pulseaudio.git;a=commit;h=f49711c99a86075de0a2337c4ea71049a9aa146a


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] [PATCH] build: Bump Orc dependency to 0.4.11

2011-03-25 Thread Paul Menzel
Am Freitag, den 25.03.2011, 10:59 + schrieb Colin Guthrie:
 'Twas brillig, and Arun Raghavan at 25/03/11 10:46 did gyre and gimble:
  0.4.9 errors out at compile time,

The error is the following [1].

svolume_orc.c: error: line 67: unknown directive: .longparam

  and might as well bump to 0.4.11 since
  that's the version being tested with and has been around for a while
  now. Thanks to Paul Menzel paulepan...@users.sourceforge.net for
  pointing this out.

Well I am undecided if the depends should reflect the necessary version
or the recommended one. Orc 0.4.11 seems to fix some bugs [2], but it
should be the job of the distribution/person to decide what version to
use? For example OpenEmbedded still carries a patch to decrease the
version of Autoconf [3] and it seems to work alright.

 Thanks. Pushed.

This is f49711c9 [4].


Thanks,

Paul


[1] 
https://tango.0pointer.de/pipermail/pulseaudio-discuss/2011-March/009519.html
[2] 
http://code.entropywave.com/git?p=orc.git;a=blob;f=RELEASE;h=a2629c6e339398b3dc5da700dee325c25feef26a;hb=HEAD
[3] 
http://cgit.openembedded.org/cgit.cgi/openembedded/commit/recipes/pulseaudio/pulseaudio-0.9.21/autoconf_version.patch?id=0e0de60f4443c143fa92068932722f5d75b0999b
[4] 
http://git.0pointer.de/?p=pulseaudio.git;a=commit;h=f49711c99a86075de0a2337c4ea71049a9aa146a


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] [PATCH] sbc_math.h: add explicit check for ARMv6 instructions

2011-03-25 Thread Paul Menzel
Dear Arun,


Am Samstag, den 19.03.2011, 16:14 +0530 schrieb Arun Raghavan:

 On Wed, 2011-02-23 at 01:07 +0530, Arun Raghavan wrote:
 [...]
  The correct fix for this, imo, is in bluez (there is a new
  sbc_primitives_armv6.h that can probably be used at least as a
  template). We need to do an sbc-udpate on the PA side anyway, and can
  pull this when we do.
 
 Could you see if the attached patch works for you? If it does, I can
 push this to the bluez folks.

My tag on the other thread [1] is ambiguous. The error is indeed caused
by your patch. I will try to implement your recommended changes but will
probably not get to it before Sunday.


Thanks,

Paul
 

[1] 
https://tango.0pointer.de/pipermail/pulseaudio-discuss/2011-March/009594.html


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


[pulseaudio-discuss] modules/bluetooth/sbc/sbc_primitives.h:41:22: error: expected ':', ',', '; ', '}' or '__attribute__' before 'X'

2011-03-24 Thread Paul Menzel
Dear PulseAudio folks,


I get the following error with latest master (a9c8f904).

  CC libbluetooth_sbc_la-sbc.lo
In file included from modules/bluetooth/sbc/sbc_primitives_armv6.h:30:0,
 from modules/bluetooth/sbc/sbc_math.h:27,
 from modules/bluetooth/sbc/sbc.c:46:
modules/bluetooth/sbc/sbc_primitives.h:41:22: error: expected ':', ',', 
';', '}' or '__attribute__' before 'X'
modules/bluetooth/sbc/sbc.c: In function 'sbc_synthesize_four':
modules/bluetooth/sbc/sbc.c:577:18: warning: declaration of 'tmp' 
shadows a previous local [-Wshadow]
modules/bluetooth/sbc/sbc.c:577:18: warning: shadowed declaration is 
here [-Wshadow]
modules/bluetooth/sbc/sbc.c:577:18: warning: declaration of 'tmp' 
shadows a previous local [-Wshadow]
modules/bluetooth/sbc/sbc.c:577:18: warning: shadowed declaration is 
here [-Wshadow]
modules/bluetooth/sbc/sbc.c:577:18: warning: declaration of 'tmp' 
shadows a previous local [-Wshadow]
modules/bluetooth/sbc/sbc.c:577:18: warning: shadowed declaration is 
here [-Wshadow]
modules/bluetooth/sbc/sbc.c:577:18: warning: declaration of 'tmp' 
shadows a previous local [-Wshadow]
modules/bluetooth/sbc/sbc.c:577:18: warning: shadowed declaration is 
here [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' 
shadows a previous local [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is 
here [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' 
shadows a previous local [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is 
here [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' 
shadows a previous local [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is 
here [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' 
shadows a previous local [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is 
here [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' 
shadows a previous local [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is 
here [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' 
shadows a previous local [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is 
here [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' 
shadows a previous local [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is 
here [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' 
shadows a previous local [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is 
here [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' 
shadows a previous local [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is 
here [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' 
shadows a previous local [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is 
here [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' 
shadows a previous local [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is 
here [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' 
shadows a previous local [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is 
here [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' 
shadows a previous local [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is 
here [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' 
shadows a previous local [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is 
here [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' 
shadows a previous local [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is 
here [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: declaration of 'tmp' 
shadows a previous local [-Wshadow]
modules/bluetooth/sbc/sbc.c:589:51: warning: shadowed declaration is 
here [-Wshadow]
modules/bluetooth/sbc/sbc.c: In function 'sbc_synthesize_eight':
modules/bluetooth/sbc/sbc.c:619:29: warning: declaration of 'tmp' 
shadows a previous local [-Wshadow]
modules/bluetooth/sbc/sbc.c:619:29: warning: shadowed declaration is 
here [-Wshadow]
modules/bluetooth/sbc/sbc.c:619:29: warning: declaration of 'tmp' 
shadows a 

[pulseaudio-discuss] svolume_orc.c: error: line 67: unknown directive: .longparam

2011-03-23 Thread Paul Menzel
Dear PulseAudio folks,


I get the following error with latest master (c0392af2).

| + do_compile
| + base_do_compile
| + '[' -e Makefile -o -e makefile -o -e GNUmakefile ']'
| + oe_runmake
| + oenote make
| + echo NOTE: make
| NOTE: make
| + make
| make  all-recursive
| make[1]: Entering directory 
`/oe/build-minimal-uclibc/minimal-uclibc-dev/work/armv5te-oe-linux-uclibceabi/pulseaudio-0.9.22+git-r11.1-r0+c0392af20a504d7ced715cbd7152fd6d032125aa/git'
| Making all in src
| make[2]: Entering directory 
`/oe/build-minimal-uclibc/minimal-uclibc-dev/work/armv5te-oe-linux-uclibceabi/pulseaudio-0.9.22+git-r11.1-r0+c0392af20a504d7ced715cbd7152fd6d032125aa/git/src'
|   ORCC   pulsecore/svolume-orc-gen.c
| In function pa_volume_s16ne_orc_2ch:
| error: line 67: unknown directive: .longparam
| error: variable (null) used before being written
| Failed to compile pa_volume_s16ne_orc_2ch
| Failed to compile pa_volume_s16ne_orc_2ch
| make[2]: *** [pulsecore/svolume-orc-gen.c] Error 1
| make[2]: Leaving directory 
`/oe/build-minimal-uclibc/minimal-uclibc-dev/work/armv5te-oe-linux-uclibceabi/pulseaudio-0.9.22+git-r11.1-r0+c0392af20a504d7ced715cbd7152fd6d032125aa/git/src'
| make[1]: *** [all-recursive] Error 1
| make[1]: Leaving directory 
`/oe/build-minimal-uclibc/minimal-uclibc-dev/work/armv5te-oe-linux-uclibceabi/pulseaudio-0.9.22+git-r11.1-r0+c0392af20a504d7ced715cbd7152fd6d032125aa/git'
| make: *** [all] Error 2
| + die 'oe_runmake failed'
| + oefatal 'oe_runmake failed'
| + echo FATAL: 'oe_runmake failed'
| FATAL: oe_runmake failed
| + exit 1

I guess it has something to do with

commit 4cd90d9e32ca9a23e3c0f7615974ea0c55ff3e49
Author: Arun Raghavan arun.ragha...@collabora.co.uk
Date:   Mon Oct 25 17:59:08 2010 +0100

volume: Add Orc-based optimised volume scaling

This adds volume scaling for 1- and 2-channel software volume 
scaling
using Orc. While testing the MMX and SSE backends on a Core2, I see 
an
~2x performance benefit over the hand-rolled MMX and SSE code. 
Since I
haven't been able to test on other architectures, the Orc code is 
only
used when MMX/SSE* is present. This can be changed in the future 
after
testing on AMD and ARM machines.

but I do not know anything about this.

I am using OpenEmbedded with `minimal` or `minimal-uclibc` for `MACHINE
= at91sam9260ek`. ORCC 0.4.9 is used on this system.


Thanks,

Paul


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


[pulseaudio-discuss] [PATCH] src/modules/module-tunnel.c: fix typo (s/passthough/passthrough/)

2011-03-20 Thread Paul Menzel
Date: Sun, 20 Mar 2011 23:00:19 +0100

---
 src/modules/module-tunnel.c |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/modules/module-tunnel.c b/src/modules/module-tunnel.c
index 65fb67d..a8af6c1 100644
--- a/src/modules/module-tunnel.c
+++ b/src/modules/module-tunnel.c
@@ -1690,7 +1690,7 @@ static void setup_complete_callback(pa_pdispatch *pd, 
uint32_t command, uint32_t
 pa_tagstruct_put_boolean(reply, FALSE); /* relative volume */
 
 if (u-version = 18) {
-pa_tagstruct_put_boolean(reply, FALSE); /* passthough stream */
+pa_tagstruct_put_boolean(reply, FALSE); /* passthrough stream */
 #endif
 
 pa_pstream_send_tagstruct(u-pstream, reply);
-- 
1.7.4.1



signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


[pulseaudio-discuss] [PATCH] src/modules/module-tunnel.c: remove unneeded `{`

2011-03-20 Thread Paul Menzel
Date: Sun, 20 Mar 2011 23:01:13 +0100

• fix up for

commit 36e7f8a92e8b340dd4034a39a655ba93bcd41886
Author: Colin Guthrie cguth...@mandriva.org
Date:   Sun Mar 20 14:09:14 2011 +

  tunnel: Fix tunnel streams with recent servers

  Prompted by bug #750. Thanks to Emil Renner Berthing for the 
original
  patch.
---
 src/modules/module-tunnel.c |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/modules/module-tunnel.c b/src/modules/module-tunnel.c
index a8af6c1..6cb22e0 100644
--- a/src/modules/module-tunnel.c
+++ b/src/modules/module-tunnel.c
@@ -1689,7 +1689,7 @@ static void setup_complete_callback(pa_pdispatch *pd, 
uint32_t command, uint32_t
 if (u-version = 17)
 pa_tagstruct_put_boolean(reply, FALSE); /* relative volume */
 
-if (u-version = 18) {
+if (u-version = 18)
 pa_tagstruct_put_boolean(reply, FALSE); /* passthrough stream */
 #endif
 
-- 
1.7.4.1



signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


[pulseaudio-discuss] What tests on AMD and ARM are needed for Orc-based optimised volume scaling?

2011-03-10 Thread Paul Menzel
Dear Arun,


your commit messages of commit 4cd90d9e [1] says the following.

… Since I haven't been able to test on other architectures, the
Orc code is only used when MMX/SSE* is present. This can be
changed in the future after testing on AMD and ARM machines.

What tests need to be performed or what tests did you run to figure out
that it works.


Thanks,

Paul


[1] 
http://git.0pointer.de/?p=pulseaudio.git;a=commit;h=4cd90d9e32ca9a23e3c0f7615974ea0c55ff3e49


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] Nvidia hdmi passthrough patch

2011-03-04 Thread Paul Menzel
Dear Kelly,


Am Freitag, den 04.03.2011, 14:50 -0700 schrieb Kelly Anderson:

 Here's a patch that I created for git that adds passthrough support for 
 the Nvidia hdmi port.  Pretty much just copy/paste from the iec958 sections.

 --- ./src/modules/alsa/alsa-sink.c.orig2011-02-16 22:58:11.0 
 -0700
 +++ ./src/modules/alsa/alsa-sink.c2011-02-17 02:53:21.452857569 -0700
 @@ -1708,7 +1708,8 @@ static int setup_mixer(struct userdata *

your MUA seems to have mangled your patch. Please turn off automatic
line breaks or use `git send-email` to send the patch.

To ease the work for developers please also add a commit summary and
commit message (maybe with a link to the list discussion).

[…]


Thanks,

Paul


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] [PATCH 1/2] Add a target to the PA log feature

2011-02-24 Thread Paul Menzel
Dear VincentX,


please note in the subject what iteration your patch is, e. g. »[PATCH
1/2 v2]«. See `--subject-prefix` in `git help format-patch`.

Am Donnerstag, den 24.02.2011, 16:30 + schrieb Becker, VincentX:
 From 55f84afd8575dadba697c746daa61ee07b333c57 Mon Sep 17 00:00:00 2001
 From: Vincent Becker vincentx.bec...@intel.com
 Date: Thu, 24 Feb 2011 16:52:05 +0100
 Subject: [PATCH] Add a new log target to a file descriptor in daemon 
 configuration
  Signed-off-by: Vincent Becker vincentx.bec...@intel.com
  This patches adds the option to log pulseaudio messages into a file 
 descriptor.
  The file descriptor can be a regular file or other type (character, block..).
  If the file given in parameter already exists, it is automatically renamed 
 with current date infos.

You are missing a blank line after the commit summary (first line).
Applying your patch, e. g. with `git am …`, needs therefore manual
editing.

It is also common to add the Signed-off-by line at the end.

Could you add an example to the commit message about how to use the new
functionality so that people can easily test you new feature?

 ---
  src/daemon/cmdline.c |   98 +-
  src/daemon/daemon-conf.c |  108 
 +-
  src/pulsecore/log.c  |6 +++
  src/pulsecore/log.h  |5 ++
  4 files changed, 166 insertions(+), 51 deletions(-)
 
 diff --git a/src/daemon/cmdline.c b/src/daemon/cmdline.c
 index f6cdcdc..7f72b8c 100644
 --- a/src/daemon/cmdline.c
 +++ b/src/daemon/cmdline.c
 @@ -114,59 +114,59 @@ void pa_cmdline_help(const char *argv0) {
  
  printf(_(%s [options]\n\n
 COMMANDS:\n
 - -h, --helpShow this help\n
 - --version Show version\n
 - --dump-conf   Dump default 
 configuration\n
 - --dump-modulesDump list of available 
 modules\n
 - --dump-resample-methods   Dump available resample 
 methods\n
 - --cleanup-shm Cleanup stale shared 
 memory segments\n
 - --start   Start the daemon if it 
 is not running\n
 - -k  --killKill a running daemon\n
 - --check   Check for a running 
 daemon (only returns exit code)\n\n
 + -h, --help   Show this 
 help\n
 + --versionShow 
 version\n
 + --dump-conf  Dump default 
 configuration\n
 + --dump-modules   Dump list of 
 available modules\n
 + --dump-resample-methods  Dump 
 available resample methods\n
 + --cleanup-shmCleanup stale 
 shared memory segments\n
 + --start  Start the 
 daemon if it is not running\n
 + -k  --kill   Kill a 
 running daemon\n
 + --check  Check for a 
 running daemon (only returns exit code)\n\n
  
 OPTIONS:\n
 - --system[=BOOL]   Run as system-wide 
 instance\n
 - -D, --daemonize[=BOOL]Daemonize after 
 startup\n
 - --fail[=BOOL] Quit when startup 
 fails\n
 - --high-priority[=BOOL]Try to set high nice 
 level\n
 -   (only available as root, 
 when SUID or\n
 -   with elevated 
 RLIMIT_NICE)\n
 - --realtime[=BOOL] Try to enable realtime 
 scheduling\n
 -   (only available as root, 
 when SUID or\n
 -   with elevated 
 RLIMIT_RTPRIO)\n
 - --disallow-module-loading[=BOOL]  Disallow module user 
 requested module\n
 -   loading/unloading after 
 startup\n
 - --disallow-exit[=BOOL]Disallow user requested 
 exit\n
 - --exit-idle-time=SECS Terminate the daemon 
 when idle and this\n
 -   time passed\n
 - --module-idle-time=SECS   Unload autoloaded 
 modules when idle and\n
 -   this time passed\n
 - --scache-idle-time=SECS   Unload autoloaded 
 samples when idle and\n
 -   this time passed\n
 - --log-level[=LEVEL]   Increase 

[pulseaudio-discuss] patches in OpenEmbedded (was: [PATCH] src/Makefile.am: add missing space to fix build using uClibc)

2011-02-22 Thread Paul Menzel
Am Dienstag, den 22.02.2011, 10:02 + schrieb Colin Guthrie:
 'Twas brillig, and Paul Menzel at 19/02/11 20:19 did gyre and gimble:
  Concerning OpenEmbedded you can see the patches in the repository [1]. I
  am trying to send them upstream during the next days.
  
  [1] 
  http://cgit.openembedded.org/cgit.cgi/openembedded/tree/recipes/pulseaudio/
 
 I think the most interesting one there is the ARM fixes:
 http://cgit.openembedded.org/cgit.cgi/openembedded/tree/recipes/pulseaudio/pulseaudio-0.9.21/armv4+v5asm.patch

That is exactly what »[PATCH] sbc_math.h: add explicit check for ARMv6
instructions« [1] is about. I even referenced that link in the commit
message.

 Arun, as my new resident go-to-guy for ARM stuff, what's your take on it?

Please see the other thread.


Thanks,

Paul


[1] 
https://tango.0pointer.de/pipermail/pulseaudio-discuss/2011-February/009022.html


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] [PATCH] sbc_math.h: add explicit check for ARMv6 instructions

2011-02-22 Thread Paul Menzel
Am Dienstag, den 22.02.2011, 09:48 + schrieb Colin Guthrie:
 'Twas brillig, and Arun Raghavan at 21/02/11 20:31 did gyre and gimble:
  On Sun, 2011-02-20 at 19:20 +0100, Paul Menzel wrote:
  Date: Sun, 20 Feb 2011 15:57:55 +0100
 
  Building PulseAudio 051d8213 [1] using OpenEmbedded with distribution 
  `minimal-uclibc` for `MACHINE=om-gta01` having an ARMv4T architecture 
  (armv4t-oe-linux-uclibceabi) compilation fails with the following error.
  [...]
 {standard input}:4099: Error: selected processor does not support Thumb 
  mode `mla r3,r2,r6,r3'
 {standard input}:4114: Error: selected processor does not support Thumb 
  mode `mla r3,r2,ip,r3'
 {standard input}:4125: Error: selected processor does not support Thumb 
  mode `mla r3,r6,r2,r3'
 make[3]: *** [libbluetooth_sbc_la-sbc.lo] Error 1
 
  Apply the same fix as in [2], which is similar to the patch applied in 
  OpenEmbedded since commit 976ab4b5 [3].
 
  [1] 
  http://git.0pointer.de/?p=pulseaudio.git;a=commit;h=051d82133f0ae6a57bf66fd200bc8e3591a7d5ca
  [2] 
  http://git.0pointer.de/?p=pulseaudio.git;a=commit;h=12b900858ae82d435c100d6eb94cb7bb22fe5e29
  [3] 
  http://cgit.openembedded.org/cgit.cgi/openembedded/commit/?id=976ab4b4587d548c0483a274058c5359cb72bf1b
 
  Signed-off-by: Paul Menzel paulepan...@users.sourceforge.net
  CC: Marcel Holtmann mar...@holtmann.org
  ---
  I do not know if these files are just copied from linux-bluetooth 
  upstream, so I am adding Marcel to CC anyway because he is mentioned in 
  the copyright section.
  
  It does appear that the files are a copy of the bluez package. If we're
  fixing this on the PulseAudio side, I think we should include something
  like diff below configure.ac, so that we can actually verify that the
  mla instruction is available for the arch being used.
  
  -- Arun
  
  
  diff --git a/configure.ac b/configure.ac
  index 08c947a..77e9823 100644
  --- a/configure.ac
  +++ b/configure.ac
  @@ -257,13 +257,14 @@ case $host in
asm volatile (ldr r0, %2 \n
  ldr r2, %3 \n
  ldr r3, %4 \n
  +   mla r4, r1, r2, r3 \n
  ssat r1, #8, r0 \n
  str r1, %0 \n
  pkhbt r1, r3, r2, LSL #8 \n
  str r1, %1 \n
  : =m (a), =m (b)
  : m (a), m (b), m (c)
  -   : r0, r1, r2, r3, cc);
  +   : r0, r1, r2, r3, r4, cc);
return (a == -128  b == 0xaabb) ? 0 : -1;
  ]]),
[pulseaudio_cv_support_armv6=yes],
 
 Paul, can you confirm if this avoids the problem for you? If so, then
 Arun, can you make a proper patch? You can? Awesome, thanks :)

Does not Arun’s patch just improve the check for `HAVE_ARMV6`? My patch
just adds the check `if defined(HAVE_ARMV6)` to `sbc_math.h`. Without
that addition all the configure checks are not evaluated at all in
`sbc_math.h`.


Thanks,

Paul


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


[pulseaudio-discuss] [PATCH] sbc_math.h: add explicit check for ARMv6 instructions

2011-02-20 Thread Paul Menzel
Date: Sun, 20 Feb 2011 15:57:55 +0100

Building PulseAudio 051d8213 [1] using OpenEmbedded with distribution 
`minimal-uclibc` for `MACHINE=om-gta01` having an ARMv4T architecture 
(armv4t-oe-linux-uclibceabi) compilation fails with the following error.

[…]
modules/bluetooth/sbc.c:485:10: warning: cannot optimize possibly 
infinite loops [-Wunsafe-loop-optimizations]
{standard input}: Assembler messages:
{standard input}:3628: Error: selected processor does not support Thumb 
mode `mla r3,r0,r1,r3'
{standard input}:3635: Error: selected processor does not support Thumb 
mode `mla r3,r2,r0,r3'
{standard input}:3643: Error: selected processor does not support Thumb 
mode `mla r3,r1,r0,r3'
{standard input}:3705: Error: selected processor does not support Thumb 
mode `mla r3,r7,ip,r3'
{standard input}:3717: Error: selected processor does not support Thumb 
mode `mla r3,r7,ip,r3'
{standard input}:3729: Error: selected processor does not support Thumb 
mode `mla r3,r7,ip,r3'
{standard input}:3741: Error: selected processor does not support Thumb 
mode `mla r3,r7,ip,r3'
{standard input}:3753: Error: selected processor does not support Thumb 
mode `mla r3,r7,ip,r3'
{standard input}:3767: Error: selected processor does not support Thumb 
mode `mla r3,r7,ip,r3'
{standard input}:3780: Error: selected processor does not support Thumb 
mode `mla r3,r7,ip,r3'
{standard input}:3786: Error: selected processor does not support Thumb 
mode `mla r3,r4,r7,r3'
{standard input}:3794: Error: selected processor does not support Thumb 
mode `mla r3,r0,r4,r3'
{standard input}:3895: Error: selected processor does not support Thumb 
mode `mla r1,r5,r6,r1'
{standard input}:3902: Error: selected processor does not support Thumb 
mode `mla r1,r5,r6,r1'
{standard input}:3909: Error: selected processor does not support Thumb 
mode `mla r1,r5,r6,r1'
{standard input}:3916: Error: selected processor does not support Thumb 
mode `mla r1,r5,r6,r1'
{standard input}:3923: Error: selected processor does not support Thumb 
mode `mla r1,r5,r6,r1'
{standard input}:3930: Error: selected processor does not support Thumb 
mode `mla r1,r5,r6,r1'
{standard input}:3937: Error: selected processor does not support Thumb 
mode `mla r1,r5,r6,r1'
{standard input}:4018: Error: selected processor does not support Thumb 
mode `mla r3,r6,r2,r3'
{standard input}:4034: Error: selected processor does not support Thumb 
mode `mla r3,r2,ip,r3'
{standard input}:4045: Error: selected processor does not support Thumb 
mode `mla r3,r2,r6,r3'
{standard input}:4061: Error: selected processor does not support Thumb 
mode `mla r3,r2,ip,r3'
{standard input}:4072: Error: selected processor does not support Thumb 
mode `mla r3,r2,r6,r3'
{standard input}:4088: Error: selected processor does not support Thumb 
mode `mla r3,r2,ip,r3'
{standard input}:4099: Error: selected processor does not support Thumb 
mode `mla r3,r2,r6,r3'
{standard input}:4114: Error: selected processor does not support Thumb 
mode `mla r3,r2,ip,r3'
{standard input}:4125: Error: selected processor does not support Thumb 
mode `mla r3,r6,r2,r3'
make[3]: *** [libbluetooth_sbc_la-sbc.lo] Error 1

Apply the same fix as in [2], which is similar to the patch applied in 
OpenEmbedded since commit 976ab4b5 [3].

[1] 
http://git.0pointer.de/?p=pulseaudio.git;a=commit;h=051d82133f0ae6a57bf66fd200bc8e3591a7d5ca
[2] 
http://git.0pointer.de/?p=pulseaudio.git;a=commit;h=12b900858ae82d435c100d6eb94cb7bb22fe5e29
[3] 
http://cgit.openembedded.org/cgit.cgi/openembedded/commit/?id=976ab4b4587d548c0483a274058c5359cb72bf1b

Signed-off-by: Paul Menzel paulepan...@users.sourceforge.net
CC: Marcel Holtmann mar...@holtmann.org
---
I do not know if these files are just copied from linux-bluetooth upstream, so 
I am adding Marcel to CC anyway because he is mentioned in the copyright 
section.
---
 src/modules/bluetooth/sbc_math.h |4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/modules/bluetooth/sbc_math.h b/src/modules/bluetooth/sbc_math.h
index b87bc81..b0f46e5 100644
--- a/src/modules/bluetooth/sbc_math.h
+++ b/src/modules/bluetooth/sbc_math.h
@@ -47,7 +47,7 @@ typedef int32_t sbc_fixed_t;
 
 #define SBC_FIXED_0(val) { val = 0; }
 #define MUL(a, b)((a) * (b))
-#ifdef __arm__
+#if defined(__arm__)  defined(HAVE_ARMV6)
 #define MULA(a, b, res) ({ \
int tmp = res;  \
__asm__(\
@@ -57,4 +57,4 @@ typedef int32_t sbc_fixed_t;
tmp; })
 #else
 #define MULA(a, b, res)  ((a) * (b) + (res))
-#endif
+#endif /* defined (__arm__)  defined (HAVE_ARMV6) */
-- 
1.7.4.1


signature.asc
Description: This is a digitally signed message part

[pulseaudio-discuss] [PATCH] src/Makefile.am: add missing space to fix build using uClibc

2011-02-19 Thread Paul Menzel
From: Henning Heinold hein...@inf.fu-berlin.de
Date: Fri Jul 10 12:33:30 2009 +0200

Using uClibc

AM_LIBADD = $(PTHREAD_LIBS) $(INTLLIBS)

is not empty because `$(INTLLIBS)` is set to `-lintl`. This uncovered a missing 
space in commit ef0cc745 which resulted in the following build error.

[…]
CC libbluetooth_ipc_la-ipc.lo
modules/bluetooth/ipc.c: In function 'bt_audio_service_open':
modules/bluetooth/ipc.c:65:26: warning: dereferencing type-punned 
pointer might break strict-aliasing rules [-Wstrict-aliasing]
modules/bluetooth/ipc.c: In function 'bt_audio_service_get_data_fd':
modules/bluetooth/ipc.c:110:13: warning: cast increases required 
alignment of target type [-Wcast-align]
modules/bluetooth/ipc.c:110:4: warning: dereferencing type-punned 
pointer will break strict-aliasing rules [-Wstrict-aliasing]
make[3]: *** No rule to make target `-lintl', needed by 
`libbluetooth-ipc.la'.  Stop.
make[3]: Leaving directory 
`/oe/build-minimal-uclibc/minimal-uclibc-dev/work/armv7a-oe-linux-uclibceabi/pulseaudio-0.9.22-r11.0+gitr0+051d82133f0ae6a57bf66fd200bc8e3591a7d5ca/git/src'
make[2]: *** [all] Error 2
make[2]: Leaving directory 
`/oe/build-minimal-uclibc/minimal-uclibc-dev/work/armv7a-oe-linux-uclibceabi/pulseaudio-0.9.22-r11.0+gitr0+051d82133f0ae6a57bf66fd200bc8e3591a7d5ca/git/src'
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory 
`/oe/build-minimal-uclibc/minimal-uclibc-dev/work/armv7a-oe-linux-uclibceabi/pulseaudio-0.9.22-r11.0+gitr0+051d82133f0ae6a57bf66fd200bc8e3591a7d5ca/git'
make: *** [all] Error 2

This patch is taken from OpenEmbedded where it has been present since 2009 for 
PulseAudio 0.9.15 and greater [1].

[1] 
http://git.0pointer.de/?p=pulseaudio.git;a=commit;h=ef0cc74567b3bb98378c17f6a523bf18ba132ed7
[2] 
http://cgit.openembedded.org/cgit.cgi/openembedded/commit/?id=7c33f2e906a20e139d53b4f2d8fbc2773a4725b3

Signed-off-by: Henning Heinold hein...@inf.fu-berlin.de
Signed-off-by: Paul Menzel paulepan...@users.sourceforge.net
CC: Henning Heinold hein...@inf.fu-berlin.de
---
 src/Makefile.am |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/Makefile.am b/src/Makefile.am
index 0687e40..0ee7abc 100644
--- a/src/Makefile.am
+++ b/src/Makefile.am
@@ -1804,7 +1804,7 @@ BLUETOOTH_SBC_FILES = $(subst 
modules/bluetooth/,,$(libbluetooth_sbc_la_SOURCES)
 
 libbluetooth_ipc_la_SOURCES = modules/bluetooth/ipc.c modules/bluetooth/ipc.h
 libbluetooth_ipc_la_LDFLAGS = -avoid-version
-libbluetooth_ipc_la_LIBADD = $(AM_LIBADD)libpulsecore-@PA_MAJORMINOR@.la 
libpulsecommon-@PA_MAJORMINOR@.la libpulse.la
+libbluetooth_ipc_la_LIBADD = $(AM_LIBADD) libpulsecore-@PA_MAJORMINOR@.la 
libpulsecommon-@PA_MAJORMINOR@.la libpulse.la
 libbluetooth_ipc_la_CFLAGS = $(AM_CFLAGS)
 BLUETOOTH_IPC_FILES = $(subst 
modules/bluetooth/,,$(libbluetooth_ipc_la_SOURCES)) rtp.h
 
-- 
1.7.4.1


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] [PATCH] src/Makefile.am: add missing space to fix build using uClibc

2011-02-19 Thread Paul Menzel
Am Samstag, den 19.02.2011, 14:19 +0100 schrieb Maarten Bosmans:
 2011/2/19 Paul Menzel paulepan...@users.sourceforge.net:
  From: Henning Heinold hein...@inf.fu-berlin.de
  Date: Fri Jul 10 12:33:30 2009 +0200
 
  Using uClibc
 
 AM_LIBADD = $(PTHREAD_LIBS) $(INTLLIBS)
 
  is not empty because `$(INTLLIBS)` is set to `-lintl`. This uncovered a 
  missing space in commit ef0cc745 which resulted in the following build 
  error.
 
 […]
 CC libbluetooth_ipc_la-ipc.lo
 modules/bluetooth/ipc.c: In function 'bt_audio_service_open':
 modules/bluetooth/ipc.c:65:26: warning: dereferencing type-punned 
  pointer might break strict-aliasing rules [-Wstrict-aliasing]
 modules/bluetooth/ipc.c: In function 'bt_audio_service_get_data_fd':
 modules/bluetooth/ipc.c:110:13: warning: cast increases required 
  alignment of target type [-Wcast-align]
 modules/bluetooth/ipc.c:110:4: warning: dereferencing type-punned 
  pointer will break strict-aliasing rules [-Wstrict-aliasing]
 make[3]: *** No rule to make target `-lintl', needed by 
  `libbluetooth-ipc.la'.  Stop.
 make[3]: Leaving directory 
  `/oe/build-minimal-uclibc/minimal-uclibc-dev/work/armv7a-oe-linux-uclibceabi/pulseaudio-0.9.22-r11.0+gitr0+051d82133f0ae6a57bf66fd200bc8e3591a7d5ca/git/src'
 make[2]: *** [all] Error 2
 make[2]: Leaving directory 
  `/oe/build-minimal-uclibc/minimal-uclibc-dev/work/armv7a-oe-linux-uclibceabi/pulseaudio-0.9.22-r11.0+gitr0+051d82133f0ae6a57bf66fd200bc8e3591a7d5ca/git/src'
 make[1]: *** [all-recursive] Error 1
 make[1]: Leaving directory 
  `/oe/build-minimal-uclibc/minimal-uclibc-dev/work/armv7a-oe-linux-uclibceabi/pulseaudio-0.9.22-r11.0+gitr0+051d82133f0ae6a57bf66fd200bc8e3591a7d5ca/git'
 make: *** [all] Error 2
 
  This patch is taken from OpenEmbedded where it has been present since 2009 
  for PulseAudio 0.9.15 and greater [1].
 
 Hmm, yes indeed. What ever happened to upstreaming patches?

Well what I saw in OpenEmbedded is that there only some dozens of people
seem to maintain a whole frame work for building distributions and so
they seem to not have much time to get patches upstream. Maybe Henning –
keep him in CC please – can comment what could be improved.

Additionally there is no maintainer per recipe so it takes even more
time to figure out how patches have to be formatted and submitted. And
at least for PulseAudio the Web site says or suggests – as far as I know
– to open a Trac ticket for which you have to register and operate
through a Web interface. And if there is a list you often have to
subscribe to send messages.

Well it is good to know that patches can be sent to the list too.

 Is there any more interesting stuff OpenEmbedded (or any other
 distro/downstream for that matter) that isn't in git master yet?

Concerning OpenEmbedded you can see the patches in the repository [1]. I
am trying to send them upstream during the next days.

[…]


Thanks,

Paul


[1] http://cgit.openembedded.org/cgit.cgi/openembedded/tree/recipes/pulseaudio/


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


[pulseaudio-discuss] [PATCH 2/3] configure.ac: Typo. s/DBUS/D-Bus/ [1]

2011-02-18 Thread Paul Menzel
Date: Fri, 18 Feb 2011 23:30:04 +0100

[1] http://www.freedesktop.org/wiki/Software/dbus

Signed-off-by: Paul Menzel paulepan...@users.sourceforge.net
---
 configure.ac |8 
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/configure.ac b/configure.ac
index b4fa78b..7db5870 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1259,7 +1259,7 @@ AC_SUBST(HAVE_DBUS)
 AM_CONDITIONAL([HAVE_DBUS], [test x$HAVE_DBUS = x1])
 
 
-# udev and HAL depend on DBUS: So double check if they were explicitly enabled.
+# udev and HAL depend on D-Bus: So double check if they were explicitly 
enabled.
 if test x$HAVE_DBUS != x1 ; then
 HAVE_HAL=0
 if test x${hal} = xyes ; then
@@ -1660,7 +1660,7 @@ echo 
 Enable Jack:   ${ENABLE_JACK}
 Enable Async DNS:  ${ENABLE_LIBASYNCNS}
 Enable LIRC:   ${ENABLE_LIRC}
-Enable DBUS:   ${ENABLE_DBUS}
+Enable D-Bus:   ${ENABLE_DBUS}
   Enable HAL:  ${ENABLE_HAL}
   Enable BlueZ:${ENABLE_BLUEZ}
 Enable udev:   ${ENABLE_UDEV}
@@ -1686,8 +1686,8 @@ echo 
 if test ${ENABLE_DBUS} = no ; then
echo 
 = WARNING WARNING WARNING WARNING WARNING WARNING WARNING =
-You do not have DBUS support enabled. It is strongly recommended
-that you enable DBUS support if you platform supports it.
+You do not have D-Bus support enabled. It is strongly recommended
+that you enable D-Bus support if you platform supports it.
 Many parts of PulseAudio use udev, from ConsoleKit interaction
 to the Device Reservation Protocol to speak to JACK, Bluetooth
 support and even a native control protocol for communicating and
-- 
1.7.4.1


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


[pulseaudio-discuss] [PATCH 3/3] configure.ac: fix spelling and context in warning

2011-02-18 Thread Paul Menzel
Date: Fri, 18 Feb 2011 23:31:47 +0100

Signed-off-by: Paul Menzel paulepan...@users.sourceforge.net
---
 configure.ac |6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/configure.ac b/configure.ac
index 7db5870..b80799f 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1687,8 +1687,8 @@ if test ${ENABLE_DBUS} = no ; then
echo 
 = WARNING WARNING WARNING WARNING WARNING WARNING WARNING =
 You do not have D-Bus support enabled. It is strongly recommended
-that you enable D-Bus support if you platform supports it.
-Many parts of PulseAudio use udev, from ConsoleKit interaction
+that you enable D-Bus support if your platform supports it.
+Many parts of PulseAudio use D-Bus, from ConsoleKit interaction
 to the Device Reservation Protocol to speak to JACK, Bluetooth
 support and even a native control protocol for communicating and
 controling the PulseAudio daemon itself.
@@ -1700,7 +1700,7 @@ if test ${ENABLE_UDEV} = no ; then
echo 
 = WARNING WARNING WARNING WARNING WARNING WARNING WARNING =
 You do not have udev support enabled. It is strongly recommended
-that you enable udev support if you platform supports it as it is
+that you enable udev support if your platform supports it as it is
 the primary method used to detect hardware audio devices (on Linux)
 and is thus a critical part of PulseAudio on that platform.
 = WARNING WARNING WARNING WARNING WARNING WARNING WARNING =
-- 
1.7.4.1



signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


[pulseaudio-discuss] [PATCH 1/3] configure.ac: Typo. s/Console Kit/ConsoleKit/ [1]

2011-02-18 Thread Paul Menzel
Date: Fri, 18 Feb 2011 23:27:20 +0100

[1] http://www.freedesktop.org/wiki/Software/ConsoleKit
---
 configure.ac |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/configure.ac b/configure.ac
index 08c947a..b4fa78b 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1688,7 +1688,7 @@ if test ${ENABLE_DBUS} = no ; then
 = WARNING WARNING WARNING WARNING WARNING WARNING WARNING =
 You do not have DBUS support enabled. It is strongly recommended
 that you enable DBUS support if you platform supports it.
-Many parts of PulseAudio use udev, from Console Kit interaction
+Many parts of PulseAudio use udev, from ConsoleKit interaction
 to the Device Reservation Protocol to speak to JACK, Bluetooth
 support and even a native control protocol for communicating and
 controling the PulseAudio daemon itself.
-- 
1.7.4.1



signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] [PATCH] Small changes to make module-coreaduio-detect more safe

2010-12-21 Thread Paul Menzel
Dear Cai,


Am Dienstag, den 21.12.2010, 14:11 +0800 schrieb caiiiyua:

Typo in commit summary.

s/coreaduio/coreaudio/

[…]

 From: Cai.Yuanqing yuanqing@tieto.com

Maybe replace ».« by space. Also in your Git configuration: 

git config user.name Cai Yuanqing

[…]


Thanks,

Paul


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] Couple of PA problems

2010-11-28 Thread Paul Menzel
Am Sonntag, den 28.11.2010, 17:12 +0800 schrieb James Harkins:
 At Sat, 27 Nov 2010 06:02:51 -0500, Sean McNamara wrote:
  Why they are not independent: Good question ;-) The answer is that
  PA's flat-volumes feature is enabled by default.
 
 It gets a little more weird. Today, I tried to reproduce the behavior after 
 opening rhythmbox. No problem -- the volume controls seemed to be independent.
 
 After a couple hours, and listening to some music, they are linked again.
 
 It seems like, if the master volume is at a high level, lowering the volume 
 in rhythm box doesn't change the master volume. But, if the master is turned 
 down, bringing up the rhythm box volume slider also increases the master.
 
 Changing the master (in either direction) also changes the rhythm box setting.
 
 I wonder... the problem started to reproduce after plugging in my iPod. I 
 can't imagine why that would make a difference, but this whole thing is 
 strange enough, anything is possible.

Could you please post more information.

1. What version of PA do you run?
2. Did you try the latest release 0.9.22?
3. Does that happen with another media player, for example Banshee, too?
4. What version of Rhythmbox do you use?


Thanks,

Paul


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


[pulseaudio-discuss] [PATCH] man pulse-daemon.conf: Reference correct `--use-pid-file` and fix typo (`s/Of/If/`).

2010-11-26 Thread Paul Menzel
Date: Sun, 31 Jan 2010 14:15:31 +0100

Signed-off-by: Paul Menzel paulepan...@users.sourceforge.net
---
 man/pulse-daemon.conf.5.xml.in |4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/man/pulse-daemon.conf.5.xml.in b/man/pulse-daemon.conf.5.xml.in
index f0e1de4..299af6a 100644
--- a/man/pulse-daemon.conf.5.xml.in
+++ b/man/pulse-daemon.conf.5.xml.in
@@ -123,12 +123,12 @@ USA.
 
 option
   poptuse-pid-file=/opt Create a PID file in
-  file/tmp/pulse-$USER/pid/file. Of this is enabled you may
+  file/tmp/pulse-$USER/pid/file. If this is enabled you may
   use commands like opt--kill/opt or opt--check/opt. If
   you are planning to start more than one PulseAudio process per
   user, you better disable this option since it effectively
   disables multiple instances. Takes a boolean argument, defaults
-  to optyes/opt. The opt--no-cpu-limit/opt command line
+  to optyes/opt. The opt--use-pid-file/opt command line
   option takes precedence./p
 /option
 
-- 
1.7.2.3


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] Choose remote default sink from commandline?

2010-11-13 Thread Paul Menzel
Am Freitag, den 12.11.2010, 21:55 -0500 schrieb Dave:

[…]

 - How do I set this tunnel as the default sink (so apps launched from 
 gnome will use it)?

As they wrote multiple times already. Just move the stream to this sink
in `pavucontrol` and PA will remember that setting.

Your subject line seems to require a solution for the command line, but
since you used the *deprecated* `padevchooser` (annotation for users
reading the archives) I guess you will be able to run `pavucontrol`.


Thanks,

Paul


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] Add a new target to Pulseaudio log feature

2010-11-05 Thread Paul Menzel
Dear Vincent,


Am Freitag, den 05.11.2010, 14:10 + schrieb Becker, VincentX:

[…]

 Would it be possible to add them in Pulseaudio and shall I create a ticket 
 for this ?
 
 Could you provide me some details in the process of upstreaming on PA.org ?
 
 I can quickly provide the code for our needs.

please send a patch to this mailing list for review.

$ git format-patch -1 -s
$ git send-email 0001… # not required, you can also attach the file


Thanks,

Paul


PS: Please send only plain text messages.


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] Compiler optimisation dependency?

2010-10-24 Thread Paul Menzel
Am Sonntag, den 24.10.2010, 19:37 +0100 schrieb tarantism:
 I'm building pulse under Scratchbox for Maemo 5 then copying the
 module-sine.so file to Nokia N900 to test.
 
 If I use compiler options -O0 or -O1, it works fine.
 If I use -O2 or -O3 I just get a sqwarking noise.
 
 Any ideas?

What versions of PulseAudio and your tool chain do you use?


Thanks,

Paul


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] Compiler optimisation dependency?

2010-10-24 Thread Paul Menzel
Am Sonntag, den 24.10.2010, 21:11 +0100 schrieb tarantism:
 On Sun, 2010-10-24 at 21:25 +0200, Paul Menzel wrote:
   Any ideas?
  
  What versions of PulseAudio and your tool chain do you use?
 PulseAudio 0.9.15
 sbox-arm-none-linux-gnueabi-gcc (GCC) 4.2.1
 ltmain.sh (GNU libtool) 2.2.6
 automake (GNU automake) 1.11.1
 
 ... is that everything?

Looks good to me, but I am not a developer.

I think everybody will be interested if the latest development version
works for you and if this problem is fixed there. Could you install from
Git and try `stable-queue` (and `master`(?)) [1]?


Thanks,

Paul


[1] http://pulseaudio.org/wiki/DownloadPulseAudio#Sourcecode
[2] http://colin.guthr.ie/2010/09/compiling-and-running-pulseaudio-from-git/


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] Man page clarifications

2010-10-21 Thread Paul Menzel
Am Donnerstag, den 21.10.2010, 08:49 +0200 schrieb David Henningsson:
 I'm relaying a bug comment to this list, the reporter has ideas about 
 the PulseAudio documentation. He originally filed a bug about that when 
 adding a pair of lines to ~/.pulse/default.pa and PulseAudio stopped 
 working, as he was unaware that it caused /etc/pulse/default.pa not 
 being used. I think it's healthy to get bug reports like these, could to 
 know what confuses users and what makes PA harder to use.
 
 Bug report link: https://bugs.launchpad.net/bugs/663019
 
 = Comment from B Bobo begins here ===

[…]

I agree that it is good to get this user feedback. Please thank B Bobo
for taking the time to write this down.

Could you ask him, if he is willing to provide a patch with the changes
he can do.

$ git config --global user.name Real name
$ git config --global user.email b.b...@example.org
$ git clone git://git.0pointer.de/pulseaudio.git
$ cd pulseaudio
$ gedit *files for manpages*
$ git commit -a
$ git format-patch -n # n = number of commits
$ # either send those files to the mailing list 
http://pulseaudio.org/wiki/Community#MailingLists
$ # or use git send-email


Thanks,

Paul


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


[pulseaudio-discuss] [PATCH 1/2] man/pulse-daemon.conf.5.xml.in: Typo and uniform spelling. s/synchornised/synchronized/

2010-10-17 Thread Paul Menzel
Date: Sun, 17 Oct 2010 19:28:27 +0200

Signed-off-by: Paul Menzel paulepan...@users.sourceforge.net
---
 man/pulse-daemon.conf.5.xml.in |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/man/pulse-daemon.conf.5.xml.in b/man/pulse-daemon.conf.5.xml.in
index b3c3e5a..e0ac038 100644
--- a/man/pulse-daemon.conf.5.xml.in
+++ b/man/pulse-daemon.conf.5.xml.in
@@ -447,7 +447,7 @@ USA.
 cause the resulting output sound to be momentarily too loud or too soft.
 So to ensure SW and HW volumes are applied concurrently without any
 glitches, their application needs to be synchronized. The sink
-implementation needs to support synchornised volumes can use the following
+implementation needs to support synchronized volumes can use the following
 parameters to refine the process./p
 
 option
-- 
1.7.1



signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


[pulseaudio-discuss] [PATCH 2/2] man/pulse-daemon.conf.5: Fix sentence.

2010-10-17 Thread Paul Menzel
Date: Sun, 17 Oct 2010 18:51:18 +0200

This section was added in commit 58bdd9 [1].

[1] 
http://git.0pointer.de/?p=pulseaudio.git;a=commit;h=58bdd97260b8acdc07bedd4cc3112786cb43ddbe

Signed-off-by: Paul Menzel paulepan...@users.sourceforge.net
---
 man/pulse-daemon.conf.5.xml.in |4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/man/pulse-daemon.conf.5.xml.in b/man/pulse-daemon.conf.5.xml.in
index e0ac038..96bec7b 100644
--- a/man/pulse-daemon.conf.5.xml.in
+++ b/man/pulse-daemon.conf.5.xml.in
@@ -447,8 +447,8 @@ USA.
 cause the resulting output sound to be momentarily too loud or too soft.
 So to ensure SW and HW volumes are applied concurrently without any
 glitches, their application needs to be synchronized. The sink
-implementation needs to support synchronized volumes can use the following
-parameters to refine the process./p
+implementation needs to support synchronized volumes. The following
+parameters can be used to refine the process./p
 
 option
   poptenable-sync-volume=/opt Enable sync volume for the sinks that
-- 
1.7.1



signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] [PATCH] alsa-mixer: add profile for Native Instruments Korecontroller

2010-10-16 Thread Paul Menzel
Dear Daniel,


Am Freitag, den 15.10.2010, 18:18 +0200 schrieb Daniel Mack:
 ---
  src/Makefile.am|3 +-
  .../alsa/mixer/profile-sets/90-pulseaudio.rules|3 +
  .../native-instruments-korecontroller.conf |   85 
 
  3 files changed, 90 insertions(+), 1 deletions(-)
  create mode 100644 
 src/modules/alsa/mixer/profile-sets/native-instruments-korecontroller.conf
 
 diff --git a/src/Makefile.am b/src/Makefile.am
 index e55c31c..87f1178 100644
 --- a/src/Makefile.am
 +++ b/src/Makefile.am
 @@ -1100,7 +1100,8 @@ dist_alsaprofilesets_DATA = \
   modules/alsa/mixer/profile-sets/maudio-fasttrack-pro.conf \
   
 modules/alsa/mixer/profile-sets/native-instruments-audio4dj.conf \
   
 modules/alsa/mixer/profile-sets/native-instruments-audio8dj.conf \
 - 
 modules/alsa/mixer/profile-sets/native-instruments-traktorkontrol-s4.conf
 + 
 modules/alsa/mixer/profile-sets/native-instruments-traktorkontrol-s4.conf \
 + 
 modules/alsa/mixer/profile-sets/native-instruments-korecontroller.conf
  
  if HAVE_UDEV
  dist_udevrules_DATA = \
 diff --git a/src/modules/alsa/mixer/profile-sets/90-pulseaudio.rules 
 b/src/modules/alsa/mixer/profile-sets/90-pulseaudio.rules
 index 7a425b5..2c3821b 100644
 --- a/src/modules/alsa/mixer/profile-sets/90-pulseaudio.rules
 +++ b/src/modules/alsa/mixer/profile-sets/90-pulseaudio.rules
 @@ -22,7 +22,10 @@ KERNEL!=card*, GOTO=pulseaudio_end
  
  SUBSYSTEMS==usb, ATTRS{idVendor}==17cc, ATTRS{idProduct}==1978, 
 ENV{PULSE_PROFILE_SET}=native-instruments-audio8dj.conf
  SUBSYSTEMS==usb, ATTRS{idVendor}==17cc, ATTRS{idProduct}==0839, 
 ENV{PULSE_PROFILE_SET}=native-instruments-audio4dj.conf
 +SUBSYSTEMS==usb, ATTRS{idVendor}==17cc, ATTRS{idProduct}==1010, 
 ENV{PULSE_PROFILE_SET}=native-instruments-audio6dj.conf
 +SUBSYSTEMS==usb, ATTRS{idVendor}==17cc, ATTRS{idProduct}==1021, 
 ENV{PULSE_PROFILE_SET}=native-instruments-audio10dj.conf

Did this slip in by accident? I do not see why this got added.

  SUBSYSTEMS==usb, ATTRS{idVendor}==17cc, ATTRS{idProduct}==baff, 
 ENV{PULSE_PROFILE_SET}=native-instruments-traktorkontrol-s4.conf
 +SUBSYSTEMS==usb, ATTRS{idVendor}==17cc, ATTRS{idProduct}==4711, 
 ENV{PULSE_PROFILE_SET}=native-instruments-korecontroller.conf
  SUBSYSTEMS==usb, ATTRS{idVendor}==0763, ATTRS{idProduct}==2012, 
 ENV{PULSE_PROFILE_SET}=maudio-fasttrack-pro.conf
  
  LABEL=pulseaudio_end
 diff --git 
 a/src/modules/alsa/mixer/profile-sets/native-instruments-korecontroller.conf 
 b/src/modules/alsa/mixer/profile-sets/native-instruments-korecontroller.conf
 new file mode 100644
 index 000..904357d
 --- /dev/null
 +++ 
 b/src/modules/alsa/mixer/profile-sets/native-instruments-korecontroller.conf

[…]


Thanks,

Paul


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] [PATCH 5/5] man: sync_volume parameters to manual page

2010-10-15 Thread Paul Menzel
Dear Colin,


Am Donnerstag, den 14.10.2010, 20:30 +0100 schrieb Colin Guthrie:
 Subject: [PATCH] man: sync_volume parameters to manual page
 
 Some wording teaks by Colin Guthrie
 
 Signed-off-by: Jyri Sarha jyri.sa...@nokia.com
 Reviewed-by: Tanu Kaskinen tanu.kaski...@digia.com
 ---
  man/pulse-daemon.conf.5.xml.in |   26 ++
  1 files changed, 26 insertions(+), 0 deletions(-)
 
 diff --git a/man/pulse-daemon.conf.5.xml.in b/man/pulse-daemon.conf.5.xml.in
 index b5c9e92..f0a642b 100644
 --- a/man/pulse-daemon.conf.5.xml.in
 +++ b/man/pulse-daemon.conf.5.xml.in
 @@ -437,6 +437,32 @@ USA.
  
/section
  
 +  section name=Default Sync Volume Settings
 +
 +pThe flat volume feature sets the sink HW volume according to the
 +highest volume input stream and reduces the volume any other streams
 +(with lower volumes) in SW. To make this concurrent and ensure SW and
 +HW volume work together without any glitches, their application needs to
 +be synchronized. The sink implementation needs to support this
 +feature in order to these parameters to take effect./p
 +
 +option
 +  poptenable-sync-volume=/opt Enable sync volume for the sinks that
 +  support it. This feature is enabled by default./p
 +/option
 +option
 +  poptsync-volume-safety-margin-usec=/opt The unit of time by which
 +  the HW volume increases are delayed and HW volume decreases are 
 advanced.
 +  Defaults to 8000 usec./p

I am not sure, but I would read it that I have to give the unit of time,
i. e., seconds, hours, milliseconds, …, as a value. So I think the
former wording is more correct. Since the unit of time is `usecs`.

 +/option
 +option
 +  poptsync-volume-extra-delay-usec=/opt The unit of time all HW
 +  volume changes are delayed. Negative values are also allowed.
 +  Defaults to 0./p
 +/option
 +
 +  /section

I am still thinking that a „normal“ users does not have a clue on how to
determine to what value these options should be set. So it should be
marked as expert value or something or a hint should be given on how one
can determine what values should be used.

 +
section name=Authors
  pThe PulseAudio Developers lt;@PACKAGE_BUGREPORT@gt;; PulseAudio is 
 available from url href=@PACKAGE_URL@//p
/section


Thanks,

Paul


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


[pulseaudio-discuss] Wanted: native English speakers for review: [PATCH 5/5] man: sync_volume parameters to manual page

2010-10-14 Thread Paul Menzel
Am Mittwoch, den 13.10.2010, 19:40 +0300 schrieb o...@iki.fi:
 From: Jyri Sarha jyri.sa...@nokia.com
 
 Signed-off-by: Jyri Sarha jyri.sa...@nokia.com
 Reviewed-by: Tanu Kaskinen tanu.kaski...@digia.com

It would be great if some native English speakers could take a look at
the following sentences. Maybe David (CCed) has some time too.

I am giving it a shot too, but I do not know if everything is correct.

 ---
  man/pulse-daemon.conf.5.xml.in |   26 ++
  1 files changed, 26 insertions(+), 0 deletions(-)
 
 diff --git a/man/pulse-daemon.conf.5.xml.in b/man/pulse-daemon.conf.5.xml.in
 index b5c9e92..ccc2db7 100644
 --- a/man/pulse-daemon.conf.5.xml.in
 +++ b/man/pulse-daemon.conf.5.xml.in
 @@ -437,6 +437,32 @@ USA.
  
/section
  
 +  section name=Default Sync Volume Settings
 +
 +pThe flat volume feature sets the sink HW volume according to

according to *the*

 +highest volume input stream and lowers the lower volume streams
 +with SW volume. To make this concurrent use of SW and HW volume to
 +work without a glitch their application needs to be
 +synchronized. The sink implementation needs to support this
 +feature in order to these parameters to take effect./p
 +
 +option
 +  poptenable-sync-volume=/opt Enable sync volume for the sinks that
 +  support it. This feature is enabled by default./p
 +/option
 +option
 +  poptsync-volume-safety-margin-usec=/opt The amount usecs the HW

amount *of* usecs

 +  volume increases are delayed and HW volume decreases are advanced. 
 Defaults

As someone who does not have a clue, could that be a little bit more
elaborate explained. Why should increases be delayed?

Could an example be give and would it be useful?

 +  to 8000 usec./p
 +/option
 +option
 +  poptsync-volume-extra-delay-usec=/opt The amount usecs the

The amount *of* usesc

 +  all HW volume changes are delayed. Negative values are also allowed.

»the all«?

 +  Defaults to 0./p
 +/option
 +
 +  /section
 +
section name=Authors
  pThe PulseAudio Developers lt;@PACKAGE_BUGREPORT@gt;; PulseAudio is 
 available from url href=@PACKAGE_URL@//p
/section


Thanks,

Paul


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] [PATCH] Two identical USB sound-cards - avahi fails to add service.

2010-09-10 Thread Paul Menzel
Dear Ivar,


Am Donnerstag, den 09.09.2010, 20:20 +0200 schrieb Ivar Mossin:

[…]

thank you for your patch. Could you please send the patch using the Git
tools (`git format-patch -s -1` (and maybe) `git send-email`). This way
developers do not need to copy paste your commit message from your
messages but just use the tools to get this done automatically.

I do not know if a Signed-off-by line is required though.


Thanks,

Paul


PS: And please also just send plain text messages.


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] How to save mixer settings?

2010-09-07 Thread Paul Menzel
Dear Jan,


Am Montag, den 06.09.2010, 09:51 +0200 schrieb Jan Kasprzak:

 On my laptop the Front mic input is routed to the speakers/headphones
 after boot, causing unnecessary noise and sometimes even a feedback loop.
 I can mute it using amixer -D hw:0 set 'Front mic' mute, but it becomes
 unmuted every time I suspend and resume the laptop, or every time I log in.
 
 Is it possible to save the mixer settings w/ pulse?

I am sorry, I cannot answer your question. But I guess PA is not
responsible for that and this should be ALSA’s responsibility.

 My laptop is ASUS F3e with intel HDA sound with Realtek ALC660-VD codec chip
 (snd-hda-intel module is loaded with model=lenovo switch because of
 an unrelated problem described here:
 http://www.fi.muni.cz/~kas/blog/index.cgi/computers/intel-hda-headphone-speakers.html
  ).

1. Did you report that issue to the alsa-devel list [1][2]?
2. Your issue is from September 2008. What ALSA version did you use back
then and what version do you use now?
3. Have you tried more recent ALSA versions? I think a lot of fixes have
been made.
4. Since you have an ASUS system there might be a better value for
`model`?
5. If nothing helps and the problem is still present, maybe HDA-Analyzer
[3] can help you to fix it correctly.


Thanks,

Paul


[1] http://mailman.alsa-project.org/mailman/listinfo/alsa-devel
[2] http://alsa-project.org/main/index.php/Help_To_Debug
[3] http://alsa-project.org/main/index.php/Help_To_Debug_Intel_HDA


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] Volume control interactions

2010-08-15 Thread Paul Menzel
Am Sonntag, den 15.08.2010, 17:03 +0100 schrieb Dylan:
 It seems to me that the behaviour of various volume controls and their 
 interactions is wrong. Let me explain ...

Have you already read [1]?

[…]


Thanks,

Paul


[1] http://pulseaudio.org/wiki/PulseAudioStoleMyVolumes


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


[pulseaudio-discuss] translation and transifex: Trying to fix a typo

2010-05-23 Thread Paul Menzel
Dear PA folks,


I noticed a typo in the German translation and read [1], so instead of
just sending a patch to this list, I registered at Transifex [2] and
tried to submit the fix. Since trying to edit the translation using the
Transifex Web interface did show to me the already translated strings, I
went ahead, checked out the branch `master-tx`, edited `de.po` and chose
at the overview »Submit a translation for po/de« and selected my edited
(s/hochauslösend/hochauflösend/) file.

But looking at the commit [3] I noticed that only meta data changed and
my actual fix (s/hochauslösend/hochauflösend/) did not apply.

I am pretty sure I saved the file. Still I am confused what happened.
Can someone please advice me how to go on from here. Is there a way to
revert that?


Thanks,

Paul


[1] http://pulseaudio.org/wiki/Community#BugsPatchesTranslations
[2] http://www.transifex.net/
[3] 
http://git.0pointer.de/?p=pulseaudio.git;a=blobdiff;f=po/de.po;h=38a50062a5249a65c1c1d5c9210b742d818fe614;hp=d08a3fa439d9d371aa6f74fb4eb050673fb9;hb=88681bd93d7bba693cbf6ff2e6380ead5aaa8901;hpb=2c888d747cdc1cec4200f27d59c4925264ee125e


signature.asc
Description: This is a digitally signed message part
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] 0.9.22 on Ubuntu Lucid gobbling up CPU

2010-05-13 Thread Paul Menzel
Am Donnerstag, den 13.05.2010, 20:57 + schrieb Brian J.Murrell:
 I was so hoping that the PA on Ubuntu Lucid was going to be much more stable 
 than previous releases but alas, I don't think it's to be.  Currently 
 installed 
 on a machine here is 0.9.22~0.9.21+stable-queue-32-g8478-0ubuntu14 which 
 presumably is some form of 0.9.22.
 
 I have it currently running with - due to the instability and desire to
 at least log what it's doing.  As such, I can provide a full log upon request.
 
 But currently, at this moment, there is not even any audio running through it 
 and it's using 90% of a pretty modern (and hence, somewhat beefy) processor.
 
 The only thing the log is showing currently, while this is happening:
 
 D: module-rtp-recv.c: Checking for dead streams ...
 W: ratelimit.c: 3267 events suppressed
 I: alsa-source.c: Overrun!
 I: alsa-source.c: Overrun!
 I: alsa-source.c: Overrun!
 I: alsa-source.c: Overrun!
 I: alsa-source.c: Overrun!
 I: alsa-source.c: Overrun!
 D: alsa-source.c: Wakeup from ALSA!
 I: alsa-source.c: Overrun!
 I: alsa-source.c: Overrun!
 I: alsa-source.c: Overrun!
 I: alsa-source.c: Overrun!
 W: ratelimit.c: 2993 events suppressed
 I: alsa-source.c: Overrun!
 I: alsa-source.c: Overrun!
 I: alsa-source.c: Overrun!
 I: alsa-source.c: Overrun!
 I: alsa-source.c: Overrun!
 D: alsa-source.c: Wakeup from ALSA!
 I: alsa-source.c: Overrun!
 I: alsa-source.c: Overrun!
 I: alsa-source.c: Overrun!
 I: alsa-source.c: Overrun!
 I: alsa-source.c: Overrun!
 W: ratelimit.c: 3302 events suppressed
 I: alsa-source.c: Overrun!
 I: alsa-source.c: Overrun!
 I: alsa-source.c: Overrun!
 
 The hardware in this machine is:
 
 $ lspci | grep -i audio
 04:01.0 Audio device: VIA Technologies, Inc. VT1708/A [Azalia HDAC] (VIA High 
 Definition Audio Controller) (rev 10)
 05:05.0 Multimedia audio controller: Ensoniq ES1371 [AudioPCI-97] (rev 07)
 
 $ cat /proc/asound/cards
  0 [VT82xx ]: HDA-Intel - HDA VIA VT82xx
   HDA VIA VT82xx at 0xdfafc000 irq 17
  1 [AudioPCI   ]: ENS1371 - Ensoniq AudioPCI
   Ensoniq AudioPCI ENS1371 at 0xaf00, irq 18
 $ uname -r
 2.6.32-22-generic
 
 Again, a full log is available if desired.
 
 Thots?

You already posted the same question to the list four weeks ago [1].
Please keep the threading next time. Also you did not answer my
questions back then.


Thanks,

Paul


[1] 
https://tango.0pointer.de/pipermail/pulseaudio-discuss/2010-April/006857.html


signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] Pulse audio does not work with my hdmi output

2010-04-18 Thread Paul Menzel
Am Samstag, den 17.04.2010, 11:00 -0600 schrieb Dark Shadow:
 I could not get a pulseaudio -vvv because every time I killed it it
 re-spawned on it's own before I could start it.

Did you search for »pulseaudio respawn« on the Web? You would have found
[1].

Anyway the term used in PA seems to be »autospawn«.

Reading `man pulse-client.conf` is also informational. (Although finding
that is a little harder, because it is not referenced by `man
pulseaudio`.)

[2] and [3] in the PA Wiki are also good to read when debugging a
problem.

[…]


Thanks,

Paul


[1] https://fedoraproject.org/wiki/How_to_debug_PulseAudio_problems
[2] http://pulseaudio.org/wiki/Community#BugsPatchesTranslations
[3] http://pulseaudio.org/wiki/Troubleshooting


signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] using near 100% CPU when doing nothing

2010-04-11 Thread Paul Menzel
Am Samstag, den 10.04.2010, 19:55 + schrieb Brian J.Murrell:
 I have a(one of a number of)Ubuntu Karmic systems here running pulseaudio
 0.9.19.  On this particular system, PA wants to eat as much CPU as is
 available.  It will use up near 100% on this system when it's otherwise idle.

Did you try to record something or use the microphone?

 This particular machine is the only one of several, as I say, with this 
 problem 
 and they are all up-to-date, so I wonder if this is particular to the sound 
 hardware in this machine.  It has in it:
 
 04:01.0 Audio device: VIA Technologies, Inc. VT1708/A [Azalia HDAC] (VIA High 
 Definition Audio Controller) (rev 10)
 05:05.0 Multimedia audio controller: Ensoniq ES1371 [AudioPCI-97] (rev 07)
 
 What other information can I gather to help diagnose this problem?

Maybe it is related to ticket #758 [1] which commit [2][3][4] in ALSA
fixed for me kind of. What CODEC chip do you have in your system?


Thanks,

Paul


[1] http://pulseaudio.org/ticket/758
[2] 
http://mailman.alsa-project.org/pipermail/alsa-devel/2010-February/thread.html#25109
[3] 
http://git.kernel.org/?p=linux/kernel/git/tiwai/sound-2.6.git;a=commit;h=0708cc582f0fe2578eaab722841caf2b4f8cfe37
[4] 
http://sourceforge.net/mailarchive/forum.php?thread_name=1265550675.4642.24.camel%40mattotaupaforum_name=alsa-user


signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] Frequent trivially-reproducible client hangs with stable-queue PA

2010-02-22 Thread Paul Menzel
Am Montag, den 22.02.2010, 18:50 + schrieb Nix:
 Ever since I started using PA 0.9.21 (and possibly before: it's been
 long enough that my memory has faded), lots of applications have started
 hanging hard when fast-forwarded/rewound. This includes MPD and every
 single video player I own capable of fullscreen playback. As you might
 imagine, having to stop PA playing back sound and forcing all my video
 and music players not to use PA renders it... less useful than it might
 be, and the constant need to pay attention to ALSA volume levels makes
 PA's otherwise admirable ALSA volume-level frotzing annoying as
 well. (I'd use the ALSA-PA plugin, but as reported earlier, this
 doesn't work either, stuttering like crazy and frequently locking up.
 It may be that the ALSA plugin lockup is a special case of this one.)
 
 It seems unlikely to me that every one of these applications is buggy:
 the problem more probably lies in PA itself.
 
 This is still reproducible with stable-queue HEAD.
 
 In all cases they seem to get stuck waiting for PA to send something
 that it never sends (or, at least, all are stuck in poll()...)
 
 Examples, with client backtraces, include:

[…]

Is there anything in the log files [1]?


Thanks,

Paul


[1] http://pulseaudio.org/wiki/Community#BugsPatchesTranslations


signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] No or only random connections to PulseAudio server after `pulseaudio -k`.

2010-02-02 Thread Paul Menzel
Am Dienstag, den 02.02.2010, 09:42 + schrieb Colin Guthrie:
 'Twas brillig, and Lennart Poettering at 02/02/10 07:58 did gyre and gimble:
  On Sat, 30.01.10 15:59, Paul Menzel (paulepan...@users.sourceforge.net) 
  wrote:

[…]

  D: module-udev-detect.c: /dev/snd/controlC0 is accessible: yes
  D: module-console-kit.c: dbus: 
  interface=org.freedesktop.ConsoleKit.Seat, 
  path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded
  D: module-console-kit.c: dbus: 
  interface=org.freedesktop.ConsoleKit.Seat, 
  path=/org/freedesktop/ConsoleKit/Seat1, member=SessionRemoved
  D: module-console-kit.c: dbus: 
  interface=org.freedesktop.ConsoleKit.Seat, 
  path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded
  
  There's something seriously wrong with your CK setup.
 
 FWIW, there is more debug regarding this issue in the ticket Paul opened
 for it here:
 http://pulseaudio.org/ticket/785
 
 I tried to deconstruct the connection failures, but I'm failing.

Please do not spend anymore time with this issue. I think it was a
stupid configuration error on my site and then making wrong assumptions
afterward. I am updating the report just now.

 I suspect that the problem is in some way related to the consolekit stuff
 as you point out.

I reported the ConsoleKit error to the Debian BTS [1].


Thanks,

Paul


[1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=568076


signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] [module-console-kit.c] SessionAdded/SessionRemoved messages every 5 seconds with `log-level = debug`

2010-02-02 Thread Paul Menzel
Am Dienstag, den 02.02.2010, 08:54 +0100 schrieb Lennart Poettering:
 On Sun, 31.01.10 12:31, Paul Menzel (paulepan...@users.sourceforge.net) wrote:
 
  If I understood the code correctly, it looks like the CK-module is
  initialized every five seconds on my system. Is that expected/normal?
 
 No. This is not what happens. The filter stays in place the entire
 runtime. What apparently happens is that a CK session is set up and
 teared down every 5s on your system. dbus-monitor might help you
 figure out what session that actually is and what is going on.

Thank you for your answer. I reported this issue to the Debian BTS [1].


Thanks,

Paul


[1] http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=568076


signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


[pulseaudio-discuss] [module-console-kit.c] SessionAdded/SessionRemoved messages every 5 seconds with `log-level = debug`

2010-01-31 Thread Paul Menzel
Dear PulseAudio users,


I know that setting `log-level = debug` should increase the messages
written out to `log-target`. I noticed that the following messages are
printed every five seconds to `log-target`.

[…]
Jan 31 11:47:21 x pulseaudio[3489]: module-console-kit.c: dbus: 
interface=org.freedesktop.ConsoleKit.Seat, 
path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded
Jan 31 11:47:21 x pulseaudio[3489]: module-console-kit.c: dbus: 
interface=org.freedesktop.ConsoleKit.Seat, 
path=/org/freedesktop/ConsoleKit/Seat1, member=SessionRemoved
Jan 31 11:47:26 x pulseaudio[3489]: module-console-kit.c: dbus: 
interface=org.freedesktop.ConsoleKit.Seat, 
path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded
Jan 31 11:47:26 x pulseaudio[3489]: module-console-kit.c: dbus: 
interface=org.freedesktop.ConsoleKit.Seat, 
path=/org/freedesktop/ConsoleKit/Seat1, member=SessionRemoved
[…]

In `module-console-kit.c` [1] the module gets initialized, and the D-Bus
connection gets a `filter_cb` added to it.

int pa__init(pa_module*m) {
[…]
if (!dbus_connection_add_filter(pa_dbus_connection_get(connection), 
filter_cb, u, NULL)) {
pa_log_error(Failed to add filter function);
goto fail;
}

if (pa_dbus_add_matches(
pa_dbus_connection_get(connection), error,

type='signal',sender='org.freedesktop.ConsoleKit',interface='org.freedesktop.ConsoleKit.Seat',member='SessionAdded',

type='signal',sender='org.freedesktop.ConsoleKit',interface='org.freedesktop.ConsoleKit.Seat',member='SessionRemoved',
 NULL)  0) {
pa_log_error(Unable to subscribe to ConsoleKit signals: %s: 
%s, error.name, error.message);
goto fail;
}
[…]

Looking at `filter_cb` there the message seems to be printed by
`pa_log_debug(…)`.

static DBusHandlerResult filter_cb(DBusConnection *bus, DBusMessage 
*message, void *userdata) {
[…]

pa_log_debug(dbus: interface=%s, path=%s, member=%s\n,
 dbus_message_get_interface(message),
 dbus_message_get_path(message),
 dbus_message_get_member(message));

if (dbus_message_is_signal(message, 
org.freedesktop.ConsoleKit.Seat, SessionAdded)) {

[…]
}

add_session(u, path);

} else if (dbus_message_is_signal(message, 
org.freedesktop.ConsoleKit.Seat, SessionRemoved)) {

[…]

remove_session(u, path);
}

[…]
}

If I understood the code correctly, it looks like the CK-module is
initialized every five seconds on my system. Is that expected/normal?


Thanks,

Paul


[1] 
http://git.0pointer.de/?p=pulseaudio.git;a=blob;f=src/modules/module-console-kit.c


signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


[pulseaudio-discuss] [PATCH] man pages: correct formatting/markup of options

2010-01-31 Thread Paul Menzel

Signed-off-by: Paul Menzel paulepan...@users.sourceforge.net
---
 man/pulse-client.conf.5.xml.in |2 +-
 man/pulse-daemon.conf.5.xml.in |   10 +-
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/man/pulse-client.conf.5.xml.in b/man/pulse-client.conf.5.xml.in
index c2a847f..739fae4 100644
--- a/man/pulse-client.conf.5.xml.in
+++ b/man/pulse-client.conf.5.xml.in
@@ -69,7 +69,7 @@ USA.
 
 option
   poptautospawn=/opt Autospawn a PulseAudio daemon when
-  needed. Takes a boolean value, defaults to yes./p
+  needed. Takes a boolean value, defaults to optyes/opt./p
 /option
 
 option
diff --git a/man/pulse-daemon.conf.5.xml.in b/man/pulse-daemon.conf.5.xml.in
index e6b1e19..b5c9e92 100644
--- a/man/pulse-daemon.conf.5.xml.in
+++ b/man/pulse-daemon.conf.5.xml.in
@@ -53,14 +53,14 @@ USA.
 
 option
   poptdaemonize= /opt Daemonize after startup. Takes a
-  boolean value, defaults to no. The opt--daemonize/opt
+  boolean value, defaults to optno/opt. The opt--daemonize/opt
   command line option takes precedence./p
 /option
 
 option
   poptfail=/opt Fail to start up if any of the directives
   in the configuration script filedefault.pa/file
-  fail. Takes a boolean argument, defaults to yes. The opt--fail/opt 
command line
+  fail. Takes a boolean argument, defaults to optyes/opt. The 
opt--fail/opt command line
   option takes precedence./p
 /option
 
@@ -191,7 +191,7 @@ USA.
   RLIMIT_NICE is used. root is dropped immediately after gaining
   the nice level on startup, thus it is presumably safe. See
   manref section=1 name=pulseaudio/ for more
-  information. Takes a boolean argument, defaults to yes. The 
opt--high-priority/opt
+  information. Takes a boolean argument, defaults to optyes/opt. The 
opt--high-priority/opt
   command line option takes precedence./p
 /option
 
@@ -205,7 +205,7 @@ USA.
   real-time. The controlling thread is left a normally scheduled
   thread. Thus enabling the high-priority option is orthogonal.
   See manref section=1 name=pulseaudio/ for more
-  information. Takes a boolean argument, defaults to yes. The
+  information. Takes a boolean argument, defaults to optyes/opt. The
   opt--realtime/opt command line option takes precedence./p
 /option
 
@@ -215,7 +215,7 @@ USA.
   by default, 9 for clients. Thus it is recommended to choose the
   PulseAudio real-time priorities lower. Some PulseAudio threads
   might choose a priority a little lower or higher than the
-  specified value. Defaults to 5./p
+  specified value. Defaults to opt5/opt./p
 /option
 
 option
-- 
1.6.6.1


signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] Local and remote sink

2010-01-31 Thread Paul Menzel
Dear Samuel,


Am Samstag, den 30.01.2010, 18:57 +0100 schrieb samuel:

[…]

 Actually I would like it to check if tarzanito is available,
 if yes, use it, if not play locally.

$ man pulse-client.conf
[…]
default-server= The default sever to connect to. The environment  vari‐
able $PULSE_SERVER takes precedence.

Maybe you can check if tarzanito is available and set the environment
variable accordingly during startup?

 Also, suppose I would like to play
 locally right now, I'd switch (but not changing the config file, but
 with a quick command, a la pacmd or pactl...) preferrably using a
 keyboard shortcut. (If I have pulseaudio set up, I'd know how to get
 the shortcut)
 
 My question: Do I have to add sinks for tarzanito and my local alsa,
 and if yes, how exactly do I do that? I tried messing with the
 default.pa script, but everytime I change something, it gets messed
 up...

Sorry I have no ideas for this.


Thanks,

Paul


signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


Re: [pulseaudio-discuss] Pulseaudio spinning somewhere - should I report?

2010-01-31 Thread Paul Menzel
Dear Clemens,


Am Sonntag, den 31.01.2010, 17:23 -0500 schrieb Clemens Eisserer:
 After playing Tuxracer I found the pulseaudio-daemon was consuming 100% cpu.
 I did some stack-traces and posted it here: http://pastebin.com/f7ffb6cfd
 
 I am running Fedora-12+latest updates.
 
 Is this a known problem, or should I open a bug-report?

I do not know if this problem is known, but I suggest to open a bug
report at Fedora or upstream, where you have to register [3]. Take a
look at [1] or [2] where you find some information regarding such
problems.


Thanks,

Paul


[1] http://fedoraproject.org/wiki/How_to_debug_PulseAudio_problems
[2] http://pulseaudio.org/wiki/HowToDebugCPULoadBugs
[3] http://pulseaudio.org/


signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


[pulseaudio-discuss] [PATCH] client.conf.in: Typo. s/a/are/

2010-01-30 Thread Paul Menzel
This is the same typo as reported in [1] and fixed in [2] in `client.conf.in`.

I did `git grep values a commented` and did not get anymore hits.


[1] http://pulseaudio.org/ticket/783
[2] 
http://git.0pointer.de/?p=pulseaudio.git;a=commit;h=ea8927f1d47be101a1f64b2f87fd6e527ff76dbe


Signed-off-by: Paul Menzel paulepan...@users.sourceforge.net
---
 src/pulse/client.conf.in |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/pulse/client.conf.in b/src/pulse/client.conf.in
index 436747b..17753b0 100644
--- a/src/pulse/client.conf.in
+++ b/src/pulse/client.conf.in
@@ -16,7 +16,7 @@
 # USA.
 
 ## Configuration file for PulseAudio clients. See pulse-client.conf(5) for
-## more information. Default values a commented out.  Use either ; or # for
+## more information. Default values are commented out.  Use either ; or # for
 ## commenting.
 
 ; default-sink =
-- 
1.6.6.1


signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil
___
pulseaudio-discuss mailing list
pulseaudio-discuss@mail.0pointer.de
https://tango.0pointer.de/mailman/listinfo/pulseaudio-discuss


[pulseaudio-discuss] No or only random connections to PulseAudio server after `pulseaudio -k`.

2010-01-30 Thread Paul Menzel
Dear PulseAudio folks,


I want to debug PulseAudio problems as described in [1].

Therefore I added `autospawn = no` to `~/.pulse/client.conf` and issued
`pulseaudio -k` and `pulseaudio -v`.

But after doing so all my programs are not able to connect to the
PulseAudio server(?) anymore.

Putting `autospawn = yes` to `~/.pulse/client.conf` produces the same
behavior besides with MPlayer, where it is sometimes able to connect to
the server but fails in over 90 % of the attempts doing so.

1. For example running `gstreamer-properties` which has PulseAudio
selected as output device gives an error message.

$ LANG=C gstreamer-properties
gstreamer-properties-Message: Error running pipeline 'PulseAudio Sound 
Server': Failed to connect: Connection refused [pulsesink.c(442): 
gst_pulseringbuffer_open_device (): 
/GstPipeline:pipeline0/GstPulseSink:pulsesink3]

2. Also MPlayer does not work anymore with PulseAudio as output.

AO: [pulse] Init failed: Connection refused
Failed to initialize audio driver 'pulse'
Could not open/initialize audio device - no sound.
Audio: no sound

Killing PulseAudio with `autospawn = yes` and feeding for example 12
audio files into MPlayer, then maybe half of the time running `mplayer
-ao pulse *` is able to play a song at the end of the list (e. g. 9th
song) and fails for the following songs again.

Here is a excerpt from the debug messages. The whole can be found as an
attachment for the ticket I filed [2] because of the size restrictions
for attachments for this list.

$ LANG=C pulseaudio -v 2 20100130--pa-v.log
[…]
D: module-udev-detect.c: /dev/snd/controlC0 is accessible: yes
D: module-console-kit.c: dbus: 
interface=org.freedesktop.ConsoleKit.Seat, 
path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded
D: module-console-kit.c: dbus: 
interface=org.freedesktop.ConsoleKit.Seat, 
path=/org/freedesktop/ConsoleKit/Seat1, member=SessionRemoved
D: alsa-sink.c: Wakeup from ALSA!
I: module-suspend-on-idle.c: Source 
alsa_input.pci-_20_01.0.analog-stereo idle for too long, suspending ...
D: source.c: Suspend cause of source 
alsa_input.pci-_20_01.0.analog-stereo is 0x0004, suspending
I: alsa-source.c: Device suspended...
I: module-suspend-on-idle.c: Sink 
alsa_output.pci-_20_01.0.analog-stereo idle for too long, suspending ...
D: sink.c: Suspend cause of sink 
alsa_output.pci-_20_01.0.analog-stereo is 0x0004, suspending
I: alsa-sink.c: Device suspended...
D: reserve-wrap.c: Device lock status of reserve-monitor-wrap...@audio0 
changed: not busy
D: module-udev-detect.c: /dev/snd/controlC0 is accessible: yes
D: module-console-kit.c: dbus: 
interface=org.freedesktop.ConsoleKit.Seat, 
path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded
D: module-console-kit.c: dbus: 
interface=org.freedesktop.ConsoleKit.Seat, 
path=/org/freedesktop/ConsoleKit/Seat1, member=SessionRemoved
D: module-console-kit.c: dbus: 
interface=org.freedesktop.ConsoleKit.Seat, 
path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded
D: module-console-kit.c: dbus: 
interface=org.freedesktop.ConsoleKit.Seat, 
path=/org/freedesktop/ConsoleKit/Seat1, member=SessionRemoved
D: module-console-kit.c: dbus: 
interface=org.freedesktop.ConsoleKit.Seat, 
path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded
D: module-console-kit.c: dbus: 
interface=org.freedesktop.ConsoleKit.Seat, 
path=/org/freedesktop/ConsoleKit/Seat1, member=SessionRemoved
D: module-console-kit.c: dbus: 
interface=org.freedesktop.ConsoleKit.Seat, 
path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded
D: module-console-kit.c: dbus: 
interface=org.freedesktop.ConsoleKit.Seat, 
path=/org/freedesktop/ConsoleKit/Seat1, member=SessionRemoved
D: module-console-kit.c: dbus: 
interface=org.freedesktop.ConsoleKit.Seat, 
path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded
D: module-console-kit.c: dbus: 
interface=org.freedesktop.ConsoleKit.Seat, 
path=/org/freedesktop/ConsoleKit/Seat1, member=SessionRemoved
D: module-console-kit.c: dbus: 
interface=org.freedesktop.ConsoleKit.Seat, 
path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded
D: module-console-kit.c: dbus: 
interface=org.freedesktop.ConsoleKit.Seat, 
path=/org/freedesktop/ConsoleKit/Seat1, member=SessionRemoved
D: module-console-kit.c: dbus: 
interface=org.freedesktop.ConsoleKit.Seat, 
path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded
D: module-console-kit.c: dbus: 
interface=org.freedesktop.ConsoleKit.Seat, 
path=/org/freedesktop/ConsoleKit/Seat1, member=SessionRemoved
D: module-console-kit.c: dbus: 
interface=org.freedesktop.ConsoleKit.Seat, 
path=/org/freedesktop/ConsoleKit/Seat1, member=SessionAdded
D: module-console-kit.c: dbus: