[PATCH - amixer] listen to D-Bus for volume-up, volume-down and mute signals.

[PATCH - amixer] listen to D-Bus for volume-up, volume-down and mute signals.

[Łukasz Stelmach]

The patch introduces D-Bus support in amixer. With --dbus option given amixer
works continiously (like with the --stdin) waiting for
org.freedesktop.Hal.Device.Condition with the first argument "ButtonPressed"
and the second being one of: "volume-up", "volume-down" and "mute". These
are emmited by the HAL input helper. A single amixer process can controll
one mixer item. There are options to choose a specific item to controll,
an input device to listen and a single adjustment step size.

Signed-off-by: Łukasz Stelmach <stlman@poczta.fm>
---
 amixer/Makefile.am |    3 +-
 amixer/amixer.1    |   18 ++++++
 amixer/amixer.c    |  171 ++++++++++++++++++++++++++++++++++++++++++++++++++++
 configure.in       |   12 ++++
 4 files changed, 203 insertions(+), 1 deletions(-)

diff --git a/amixer/Makefile.am b/amixer/Makefile.am
index fcd0e81..fa951a6 100644
--- a/amixer/Makefile.am
+++ b/amixer/Makefile.am
@@ -1,5 +1,6 @@
 INCLUDES = -I$(top_srcdir)/include
-LDADD = -lm
+LDADD = -lm @DBUS_LIBS@
+CFLAGS += @DBUS_CFLAGS@
 # LDFLAGS = -static
 # CFLAGS += -g -Wall
 
diff --git a/amixer/amixer.1 b/amixer/amixer.1
index b1ac323..bc160be 100644
--- a/amixer/amixer.1
+++ b/amixer/amixer.1
@@ -88,6 +88,24 @@ Select the card number to control. The device name created from this
 parameter has syntax 'hw:N' where N is specified card number.
 
 .TP
+\fI\-\-dbus[=\fISCONTROL\fP]
+
+Connect to system D-Bus and wait for \fIorg.freedesktop.Hal.Device.Condition\fP
+signals with ButtonPressed as the first argument and one of: \fIvolume-up\fP,
+\fIvolume-down\fP or \fImute\fP as the second one. Control the selected
+\fISCONTROL\fP item.  The default item is 'Master,0'.
+
+.TP
+\fI\-\-dbus-path=<\fIUDI\fP>
+
+Choose a particular input device that emits control events.
+
+.TP
+\fI\-\-dbus-step=<\fIN\fP>
+
+Set an amount by wich to increase or decrease volume upon event.
+
+.TP
 \fI\-D\fP device
 
 Select the device name to control. The default control name is 'default'.
diff --git a/amixer/amixer.c b/amixer/amixer.c
index 9620721..31de674 100644
--- a/amixer/amixer.c
+++ b/amixer/amixer.c
@@ -18,6 +18,8 @@
  *
  */
 
+#include "aconfig.h"
+
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
@@ -31,6 +33,10 @@
 #include <sys/poll.h>
 #include "amixer.h"
 
+#ifdef HAVE_DBUS
+#include <dbus/dbus.h>
+#endif /* HAVE_DBUS */
+
 #define LEVEL_BASIC		(1<<0)
 #define LEVEL_INACTIVE		(1<<1)
 #define LEVEL_ID		(1<<2)
@@ -43,6 +49,12 @@ static int ignore_error = 0;
 static struct snd_mixer_selem_regopt smixer_options;
 static char card[64] = "default";
 
+#ifdef HAVE_DBUS
+static char dbus_sctl[64] = "Master,0";
+static char *dbus_path = NULL;
+static char dbus_step[8] = "1";
+#endif /* HAVE_DBUS */
+
 static void error(const char *fmt,...)
 {
 	va_list va;
@@ -60,6 +72,11 @@ static int help(void)
 	printf("\nAvailable options:\n");
 	printf("  -h,--help       this help\n");
 	printf("  -c,--card N     select the card\n");
+#ifdef HAVE_DBUS
+	printf("  --dbus[=sID]    Connect to the system D-Bus and wait for volume messages\n");
+	printf("  --dbus-path UDI Choose a particular input device (udi)\n");
+	printf("  --dbus-step N   Set the step size (default: 1)\n");
+#endif /* HAVE_DBUS */
 	printf("  -D,--device N   select the device, default '%s'\n", card);
 	printf("  -d,--debug      debug mode\n");
 	printf("  -n,--nocheck    do not perform range checking\n");
@@ -1887,15 +1904,147 @@ static int exec_stdin(void)
 	return 0;
 }
 
+#ifdef HAVE_DBUS
+
+#define OPT_DBUS 0x81
+#define OPT_DBUS_PATH 0x82
+#define OPT_DBUS_STEP 0x83
+
+void* get_dbus_arg(DBusMessageIter* args, int type, void** out) {
+	if (type != dbus_message_iter_get_arg_type(args))
+		return NULL;
+	dbus_message_iter_get_basic(args, out);
+	dbus_message_iter_next(args);
+	return *out;
+}
+
+int exec_dbus(void)
+{
+	char *sargv[MAX_ARGS];
+	int sargc;
+	const char* s;
+	int stepl;
+	DBusError error;
+	DBusMessage* msg;
+	DBusConnection *conn;
+	DBusMessageIter args;
+
+	stepl = strlen(dbus_step);
+	dbus_step[stepl+1] = '\0';
+
+	dbus_error_init (&error);
+	conn = dbus_bus_get (DBUS_BUS_SYSTEM, &error);
+
+	if(!conn) {
+		fprintf(stderr, "dbus:%s: %s\n",
+			error.name, error.message);
+		return 1;
+	}
+
+	dbus_bus_add_match(conn,
+		"type='signal',"
+		"interface='org.freedesktop.Hal.Device',"
+		"member='Condition'",
+		&error);
+	/* The first message is a NameOwnerChanged signal, ignore it. */
+	msg = dbus_connection_pop_message(conn);
+	dbus_message_unref(msg);
+
+	if(dbus_error_is_set(&error)) {
+		fprintf(stderr, "dbus:match error:%s\n",
+			error.message);
+		return 1;
+	}
+
+	while (1) {
+		sargv[0] = dbus_sctl; sargc = 1;
+
+		if (!(msg = dbus_connection_pop_message(conn))) {
+			dbus_connection_read_write(conn, -1);
+			continue;
+		}
+
+		s = dbus_message_get_interface(msg);
+		if(strncmp("org.freedesktop.Hal.Device", s, 26)) {
+			if(debugflag)
+				fprintf(stderr, "dbus:unsupported interface:%s\n", s);
+			goto badmessage;
+		}
+
+		if(dbus_path &&
+		    (s=dbus_message_get_path(msg)) &&
+		    strncmp(dbus_path, s, strlen(s))) {
+			if(debugflag)
+				fprintf(stderr, "dbus:not the selected udi:%s\n", s);
+			goto badmessage;
+		}
+
+		if(!dbus_message_iter_init(msg, &args)) {
+			if(debugflag)
+				fprintf(stderr, "dbus:message has no arguments\n");
+			goto badmessage;
+		}
+
+		if (strncmp(get_dbus_arg(&args, DBUS_TYPE_STRING, (void*)&s), "ButtonPressed", 13)) {
+			if(debugflag)
+				fprintf(stderr, "dbus:not a ButtonPressed event\n");
+			goto badmessage;
+		}
+
+		if (!get_dbus_arg(&args, DBUS_TYPE_STRING, (void*)&s)) {
+			if(debugflag)
+				fprintf(stderr, "dbus:invalid argument type\n");
+			goto badmessage;
+		}
+
+		/* static int sset(unsigned int argc, char *argv[], int roflag, int keep_handle) */
+		if (!strncmp(s, "volume-up", 9)) {
+			if(debugflag)
+				fprintf(stderr, "dbus:volume-up\n");
+			dbus_step[stepl]='+';
+			sargv[1] = dbus_step;
+		} else if (!strncmp(s, "volume-down", 11)) {
+			if(debugflag)
+				fprintf(stderr, "dbus:volume-down\n");
+			dbus_step[stepl]='-';
+			sargv[1] = dbus_step;
+		} else if (!strncmp(s, "mute", 4)) {
+			if(debugflag)
+				fprintf(stderr, "dbus:mute (toggle)\n");
+			sargv[1] = "toggle";
+		} else {
+			if(debugflag)
+				fprintf(stderr, "dbus:invalid argument value\n");
+			goto badmessage;
+		}
+		sset(++sargc, sargv, 0, 1);
+badmessage:
+		dbus_message_unref(msg);
+	}
+
+	if(conn)
+		dbus_connection_unref(conn);
+	return 0;
+}
+#endif /* HAVE_DBUS */
 
 int main(int argc, char *argv[])
 {
 	int morehelp, level = 0;
 	int read_stdin = 0;
+#ifdef HAVE_DBUS
+	int read_dbus = 0;
+	int t;
+#endif /* HAVE_DBUS */
 	static const struct option long_option[] =
 	{
 		{"help", 0, NULL, 'h'},
 		{"card", 1, NULL, 'c'},
+#ifdef HAVE_DBUS
+		{"dbus", optional_argument, NULL, OPT_DBUS},
+		{"dbus-path", 1, NULL, OPT_DBUS_PATH},
+		{"dbus-step", 1, NULL, OPT_DBUS_STEP},
+#endif /* HAVE_DBUS */
 		{"device", 1, NULL, 'D'},
 		{"quiet", 0, NULL, 'q'},
 		{"inactive", 0, NULL, 'i'},
@@ -1964,6 +2113,24 @@ int main(int argc, char *argv[])
 		case 's':
 			read_stdin = 1;
 			break;
+#ifdef HAVE_DBUS
+		case OPT_DBUS:
+			if(optarg) {
+				strncpy(dbus_sctl, optarg, sizeof(dbus_sctl)-1);
+				dbus_sctl[sizeof(dbus_sctl)-1] = '\0';
+			}
+			read_dbus = 1;
+			break;
+		case OPT_DBUS_PATH:
+			dbus_path=optarg;
+			break;
+		case OPT_DBUS_STEP:
+			t=atoi(optarg);
+			if(t > 0 && t <= 999999) {
+				snprintf(dbus_step, sizeof(dbus_step), "%d", atoi(optarg));
+			}
+			break;
+#endif /* HAVE_DBUS */
 		default:
 			fprintf(stderr, "Invalid switch or option needs an argument.\n");
 			morehelp++;
@@ -1978,6 +2145,10 @@ int main(int argc, char *argv[])
 	if (read_stdin)
 		return exec_stdin();
 
+#ifdef HAVE_DBUS
+	if(read_dbus)
+		return exec_dbus();
+#endif /* HAVE_DBUS */
 	if (argc - optind <= 0) {
 		return selems(LEVEL_BASIC | level) ? 1 : 0;
 	}
diff --git a/configure.in b/configure.in
index 1349ff3..717d6c5 100644
--- a/configure.in
+++ b/configure.in
@@ -7,6 +7,9 @@ AM_INIT_AUTOMAKE(alsa-utils, 1.0.20)
 AM_GNU_GETTEXT([external])
 AM_GNU_GETTEXT_VERSION([0.15])
 
+dnl required versions of other packages
+m4_define([dbus_required_version], [1.2.0])
+
 dnl Checks for programs.
 
 dnl try to gues cross-compiler if not set
@@ -74,6 +77,15 @@ AC_ARG_ENABLE(alsamixer,
        no)  alsamixer=false ;;
        *) AC_MSG_ERROR(bad value ${enableval} for --enable-alsamixer) ;;
      esac],[alsamixer=true])
+AC_ARG_WITH(dbus, [  --with-dbus             Enable D-Bus support in amixer])
+  if test "x$with_dbus" != xno; then
+    PKG_CHECK_MODULES(DBUS, dbus-1 >= dbus_required_version,
+      have_dbus=yes,
+      have_dbus=no)
+  fi
+  if test "x$have_dbus" = xyes; then
+    AC_DEFINE(HAVE_DBUS, 1, [Define to 1 if amixer is to support D-Bus])
+  fi
 fi
 AM_CONDITIONAL(ALSAMIXER, test x$alsamixer = xtrue)
 
-- 
1.6.3.3


----------------------------------------------------------------------
Zostan Dziewczyna lub Chlopakiem lata!
Wygraj skuter >> http://link.interia.pl/f22a7 

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

Re: [PATCH - amixer] listen to D-Bus for volume-up, volume-down and mute signals.

[Takashi Iwai]

At Mon,  3 Aug 2009 12:43:21 +0200,
Łukasz Stelmach wrote:
> 
> 
> The patch introduces D-Bus support in amixer. With --dbus option given amixer
> works continiously (like with the --stdin) waiting for
> org.freedesktop.Hal.Device.Condition with the first argument "ButtonPressed"
> and the second being one of: "volume-up", "volume-down" and "mute". These
> are emmited by the HAL input helper. A single amixer process can controll
> one mixer item. There are options to choose a specific item to controll,
> an input device to listen and a single adjustment step size.
> 
> Signed-off-by: Łukasz Stelmach <stlman@poczta.fm>

Thanks for the patch.

The approach is interesting, but I'm not sure whether we should put
all into amixer in this way since we don't want extra library
dependency.  (Note: amixer could be used in initrd.)
I think it's better to split the code and make it as an individual
daemon instead of additional options of amixer.


Just my $0.02.

thanks,

Takashi


> ---
>  amixer/Makefile.am |    3 +-
>  amixer/amixer.1    |   18 ++++++
>  amixer/amixer.c    |  171 ++++++++++++++++++++++++++++++++++++++++++++++++++++
>  configure.in       |   12 ++++
>  4 files changed, 203 insertions(+), 1 deletions(-)
> 
> diff --git a/amixer/Makefile.am b/amixer/Makefile.am
> index fcd0e81..fa951a6 100644
> --- a/amixer/Makefile.am
> +++ b/amixer/Makefile.am
> @@ -1,5 +1,6 @@
>  INCLUDES = -I$(top_srcdir)/include
> -LDADD = -lm
> +LDADD = -lm @DBUS_LIBS@
> +CFLAGS += @DBUS_CFLAGS@
>  # LDFLAGS = -static
>  # CFLAGS += -g -Wall
>  
> diff --git a/amixer/amixer.1 b/amixer/amixer.1
> index b1ac323..bc160be 100644
> --- a/amixer/amixer.1
> +++ b/amixer/amixer.1
> @@ -88,6 +88,24 @@ Select the card number to control. The device name created from this
>  parameter has syntax 'hw:N' where N is specified card number.
>  
>  .TP
> +\fI\-\-dbus[=\fISCONTROL\fP]
> +
> +Connect to system D-Bus and wait for \fIorg.freedesktop.Hal.Device.Condition\fP
> +signals with ButtonPressed as the first argument and one of: \fIvolume-up\fP,
> +\fIvolume-down\fP or \fImute\fP as the second one. Control the selected
> +\fISCONTROL\fP item.  The default item is 'Master,0'.
> +
> +.TP
> +\fI\-\-dbus-path=<\fIUDI\fP>
> +
> +Choose a particular input device that emits control events.
> +
> +.TP
> +\fI\-\-dbus-step=<\fIN\fP>
> +
> +Set an amount by wich to increase or decrease volume upon event.
> +
> +.TP
>  \fI\-D\fP device
>  
>  Select the device name to control. The default control name is 'default'.
> diff --git a/amixer/amixer.c b/amixer/amixer.c
> index 9620721..31de674 100644
> --- a/amixer/amixer.c
> +++ b/amixer/amixer.c
> @@ -18,6 +18,8 @@
>   *
>   */
>  
> +#include "aconfig.h"
> +
>  #include <stdio.h>
>  #include <stdlib.h>
>  #include <string.h>
> @@ -31,6 +33,10 @@
>  #include <sys/poll.h>
>  #include "amixer.h"
>  
> +#ifdef HAVE_DBUS
> +#include <dbus/dbus.h>
> +#endif /* HAVE_DBUS */
> +
>  #define LEVEL_BASIC		(1<<0)
>  #define LEVEL_INACTIVE		(1<<1)
>  #define LEVEL_ID		(1<<2)
> @@ -43,6 +49,12 @@ static int ignore_error = 0;
>  static struct snd_mixer_selem_regopt smixer_options;
>  static char card[64] = "default";
>  
> +#ifdef HAVE_DBUS
> +static char dbus_sctl[64] = "Master,0";
> +static char *dbus_path = NULL;
> +static char dbus_step[8] = "1";
> +#endif /* HAVE_DBUS */
> +
>  static void error(const char *fmt,...)
>  {
>  	va_list va;
> @@ -60,6 +72,11 @@ static int help(void)
>  	printf("\nAvailable options:\n");
>  	printf("  -h,--help       this help\n");
>  	printf("  -c,--card N     select the card\n");
> +#ifdef HAVE_DBUS
> +	printf("  --dbus[=sID]    Connect to the system D-Bus and wait for volume messages\n");
> +	printf("  --dbus-path UDI Choose a particular input device (udi)\n");
> +	printf("  --dbus-step N   Set the step size (default: 1)\n");
> +#endif /* HAVE_DBUS */
>  	printf("  -D,--device N   select the device, default '%s'\n", card);
>  	printf("  -d,--debug      debug mode\n");
>  	printf("  -n,--nocheck    do not perform range checking\n");
> @@ -1887,15 +1904,147 @@ static int exec_stdin(void)
>  	return 0;
>  }
>  
> +#ifdef HAVE_DBUS
> +
> +#define OPT_DBUS 0x81
> +#define OPT_DBUS_PATH 0x82
> +#define OPT_DBUS_STEP 0x83
> +
> +void* get_dbus_arg(DBusMessageIter* args, int type, void** out) {
> +	if (type != dbus_message_iter_get_arg_type(args))
> +		return NULL;
> +	dbus_message_iter_get_basic(args, out);
> +	dbus_message_iter_next(args);
> +	return *out;
> +}
> +
> +int exec_dbus(void)
> +{
> +	char *sargv[MAX_ARGS];
> +	int sargc;
> +	const char* s;
> +	int stepl;
> +	DBusError error;
> +	DBusMessage* msg;
> +	DBusConnection *conn;
> +	DBusMessageIter args;
> +
> +	stepl = strlen(dbus_step);
> +	dbus_step[stepl+1] = '\0';
> +
> +	dbus_error_init (&error);
> +	conn = dbus_bus_get (DBUS_BUS_SYSTEM, &error);
> +
> +	if(!conn) {
> +		fprintf(stderr, "dbus:%s: %s\n",
> +			error.name, error.message);
> +		return 1;
> +	}
> +
> +	dbus_bus_add_match(conn,
> +		"type='signal',"
> +		"interface='org.freedesktop.Hal.Device',"
> +		"member='Condition'",
> +		&error);
> +	/* The first message is a NameOwnerChanged signal, ignore it. */
> +	msg = dbus_connection_pop_message(conn);
> +	dbus_message_unref(msg);
> +
> +	if(dbus_error_is_set(&error)) {
> +		fprintf(stderr, "dbus:match error:%s\n",
> +			error.message);
> +		return 1;
> +	}
> +
> +	while (1) {
> +		sargv[0] = dbus_sctl; sargc = 1;
> +
> +		if (!(msg = dbus_connection_pop_message(conn))) {
> +			dbus_connection_read_write(conn, -1);
> +			continue;
> +		}
> +
> +		s = dbus_message_get_interface(msg);
> +		if(strncmp("org.freedesktop.Hal.Device", s, 26)) {
> +			if(debugflag)
> +				fprintf(stderr, "dbus:unsupported interface:%s\n", s);
> +			goto badmessage;
> +		}
> +
> +		if(dbus_path &&
> +		    (s=dbus_message_get_path(msg)) &&
> +		    strncmp(dbus_path, s, strlen(s))) {
> +			if(debugflag)
> +				fprintf(stderr, "dbus:not the selected udi:%s\n", s);
> +			goto badmessage;
> +		}
> +
> +		if(!dbus_message_iter_init(msg, &args)) {
> +			if(debugflag)
> +				fprintf(stderr, "dbus:message has no arguments\n");
> +			goto badmessage;
> +		}
> +
> +		if (strncmp(get_dbus_arg(&args, DBUS_TYPE_STRING, (void*)&s), "ButtonPressed", 13)) {
> +			if(debugflag)
> +				fprintf(stderr, "dbus:not a ButtonPressed event\n");
> +			goto badmessage;
> +		}
> +
> +		if (!get_dbus_arg(&args, DBUS_TYPE_STRING, (void*)&s)) {
> +			if(debugflag)
> +				fprintf(stderr, "dbus:invalid argument type\n");
> +			goto badmessage;
> +		}
> +
> +		/* static int sset(unsigned int argc, char *argv[], int roflag, int keep_handle) */
> +		if (!strncmp(s, "volume-up", 9)) {
> +			if(debugflag)
> +				fprintf(stderr, "dbus:volume-up\n");
> +			dbus_step[stepl]='+';
> +			sargv[1] = dbus_step;
> +		} else if (!strncmp(s, "volume-down", 11)) {
> +			if(debugflag)
> +				fprintf(stderr, "dbus:volume-down\n");
> +			dbus_step[stepl]='-';
> +			sargv[1] = dbus_step;
> +		} else if (!strncmp(s, "mute", 4)) {
> +			if(debugflag)
> +				fprintf(stderr, "dbus:mute (toggle)\n");
> +			sargv[1] = "toggle";
> +		} else {
> +			if(debugflag)
> +				fprintf(stderr, "dbus:invalid argument value\n");
> +			goto badmessage;
> +		}
> +		sset(++sargc, sargv, 0, 1);
> +badmessage:
> +		dbus_message_unref(msg);
> +	}
> +
> +	if(conn)
> +		dbus_connection_unref(conn);
> +	return 0;
> +}
> +#endif /* HAVE_DBUS */
>  
>  int main(int argc, char *argv[])
>  {
>  	int morehelp, level = 0;
>  	int read_stdin = 0;
> +#ifdef HAVE_DBUS
> +	int read_dbus = 0;
> +	int t;
> +#endif /* HAVE_DBUS */
>  	static const struct option long_option[] =
>  	{
>  		{"help", 0, NULL, 'h'},
>  		{"card", 1, NULL, 'c'},
> +#ifdef HAVE_DBUS
> +		{"dbus", optional_argument, NULL, OPT_DBUS},
> +		{"dbus-path", 1, NULL, OPT_DBUS_PATH},
> +		{"dbus-step", 1, NULL, OPT_DBUS_STEP},
> +#endif /* HAVE_DBUS */
>  		{"device", 1, NULL, 'D'},
>  		{"quiet", 0, NULL, 'q'},
>  		{"inactive", 0, NULL, 'i'},
> @@ -1964,6 +2113,24 @@ int main(int argc, char *argv[])
>  		case 's':
>  			read_stdin = 1;
>  			break;
> +#ifdef HAVE_DBUS
> +		case OPT_DBUS:
> +			if(optarg) {
> +				strncpy(dbus_sctl, optarg, sizeof(dbus_sctl)-1);
> +				dbus_sctl[sizeof(dbus_sctl)-1] = '\0';
> +			}
> +			read_dbus = 1;
> +			break;
> +		case OPT_DBUS_PATH:
> +			dbus_path=optarg;
> +			break;
> +		case OPT_DBUS_STEP:
> +			t=atoi(optarg);
> +			if(t > 0 && t <= 999999) {
> +				snprintf(dbus_step, sizeof(dbus_step), "%d", atoi(optarg));
> +			}
> +			break;
> +#endif /* HAVE_DBUS */
>  		default:
>  			fprintf(stderr, "Invalid switch or option needs an argument.\n");
>  			morehelp++;
> @@ -1978,6 +2145,10 @@ int main(int argc, char *argv[])
>  	if (read_stdin)
>  		return exec_stdin();
>  
> +#ifdef HAVE_DBUS
> +	if(read_dbus)
> +		return exec_dbus();
> +#endif /* HAVE_DBUS */
>  	if (argc - optind <= 0) {
>  		return selems(LEVEL_BASIC | level) ? 1 : 0;
>  	}
> diff --git a/configure.in b/configure.in
> index 1349ff3..717d6c5 100644
> --- a/configure.in
> +++ b/configure.in
> @@ -7,6 +7,9 @@ AM_INIT_AUTOMAKE(alsa-utils, 1.0.20)
>  AM_GNU_GETTEXT([external])
>  AM_GNU_GETTEXT_VERSION([0.15])
>  
> +dnl required versions of other packages
> +m4_define([dbus_required_version], [1.2.0])
> +
>  dnl Checks for programs.
>  
>  dnl try to gues cross-compiler if not set
> @@ -74,6 +77,15 @@ AC_ARG_ENABLE(alsamixer,
>         no)  alsamixer=false ;;
>         *) AC_MSG_ERROR(bad value ${enableval} for --enable-alsamixer) ;;
>       esac],[alsamixer=true])
> +AC_ARG_WITH(dbus, [  --with-dbus             Enable D-Bus support in amixer])
> +  if test "x$with_dbus" != xno; then
> +    PKG_CHECK_MODULES(DBUS, dbus-1 >= dbus_required_version,
> +      have_dbus=yes,
> +      have_dbus=no)
> +  fi
> +  if test "x$have_dbus" = xyes; then
> +    AC_DEFINE(HAVE_DBUS, 1, [Define to 1 if amixer is to support D-Bus])
> +  fi
>  fi
>  AM_CONDITIONAL(ALSAMIXER, test x$alsamixer = xtrue)
>  
> -- 
> 1.6.3.3
> 
> 
> ----------------------------------------------------------------------
> Zostan Dziewczyna lub Chlopakiem lata!
> Wygraj skuter >> http://link.interia.pl/f22a7 
> 
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

Re: [PATCH - amixer] listen to D-Bus for volume-up, volume-down and mute signals.

[Lukasz Stelmach]

[-- Attachment #1: Type: text/plain, Size: 1534 bytes --]

Takashi Iwai pisze:
> At Mon,  3 Aug 2009 12:43:21 +0200,
> Łukasz Stelmach wrote:
>>
>> The patch introduces D-Bus support in amixer. With --dbus option given amixer
>> works continiously (like with the --stdin) waiting for
>> org.freedesktop.Hal.Device.Condition with the first argument "ButtonPressed"
>> and the second being one of: "volume-up", "volume-down" and "mute". These
>> are emmited by the HAL input helper. A single amixer process can controll
>> one mixer item. There are options to choose a specific item to controll,
>> an input device to listen and a single adjustment step size.
>>
>> Signed-off-by: Łukasz Stelmach <stlman@poczta.fm>
> 
> Thanks for the patch.
> 
> The approach is interesting, but I'm not sure whether we should put
> all into amixer in this way since we don't want extra library
> dependency.  (Note: amixer could be used in initrd.)
> I think it's better to split the code and make it as an individual
> daemon instead of additional options of amixer.

No worries ;) there is --with(out)--dbus option for configure to
disable all this code which is enough for initrd because most often
(AFAIK) initrd binaries are not the same that in the live system.

-- 
Było mi bardzo miło.               Czwarta pospolita klęska, [...]
>Łukasz<                 Już nie katolicka lecz złodziejska.  (c)PP


----------------------------------------------------------------------
Zostan Dziewczyna lub Chlopakiem lata!
Wygraj skuter >> http://link.interia.pl/f22a7 

[-- Attachment #2: stlman.vcf --]
[-- Type: text/x-vcard, Size: 177 bytes --]

begin:vcard
fn;quoted-printable:=C5=81ukasz Stelmach
n;quoted-printable:Stelmach;=C5=81ukasz
email;internet:stlman@poczta.fm
x-mozilla-html:FALSE
version:2.1
end:vcard


[-- Attachment #3: Type: text/plain, Size: 160 bytes --]

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

Re: [PATCH - amixer] listen to D-Bus for volume-up, volume-down and mute signals.

[Takashi Iwai]

At Mon, 03 Aug 2009 14:04:09 +0200,
Lukasz Stelmach wrote:
> 
> Takashi Iwai pisze:
> > At Mon,  3 Aug 2009 12:43:21 +0200,
> > Łukasz Stelmach wrote:
> >>
> >> The patch introduces D-Bus support in amixer. With --dbus option given amixer
> >> works continiously (like with the --stdin) waiting for
> >> org.freedesktop.Hal.Device.Condition with the first argument "ButtonPressed"
> >> and the second being one of: "volume-up", "volume-down" and "mute". These
> >> are emmited by the HAL input helper. A single amixer process can controll
> >> one mixer item. There are options to choose a specific item to controll,
> >> an input device to listen and a single adjustment step size.
> >>
> >> Signed-off-by: Łukasz Stelmach <stlman@poczta.fm>
> > 
> > Thanks for the patch.
> > 
> > The approach is interesting, but I'm not sure whether we should put
> > all into amixer in this way since we don't want extra library
> > dependency.  (Note: amixer could be used in initrd.)
> > I think it's better to split the code and make it as an individual
> > daemon instead of additional options of amixer.
> 
> No worries ;) there is --with(out)--dbus option for configure to
> disable all this code which is enough for initrd because most often
> (AFAIK) initrd binaries are not the same that in the live system.

Usually same binaries are used for initrd nowadays in many distros.
So, configure option doesn't help.

And, I really prefer this as a separate daemon, so it be as small as
possible.  Seeing amixer as a daemon is somewhat strange...


Takashi
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

Re: [PATCH - amixer] listen to D-Bus for volume-up, volume-down and mute signals.

[Mark Brown]

On Mon, Aug 03, 2009 at 02:13:16PM +0200, Takashi Iwai wrote:
> Lukasz Stelmach wrote:

> > No worries ;) there is --with(out)--dbus option for configure to
> > disable all this code which is enough for initrd because most often
> > (AFAIK) initrd binaries are not the same that in the live system.

> Usually same binaries are used for initrd nowadays in many distros.
> So, configure option doesn't help.

> And, I really prefer this as a separate daemon, so it be as small as
> possible.  Seeing amixer as a daemon is somewhat strange...

I tend to agree, merging it into amixer seems kind of arbatrary.  Such a
daemon (and its dbus interface) may also be useful for handling things
like scenario management so I can see it growing over time.

Re: [PATCH - amixer] listen to D-Bus for volume-up, volume-down and mute signals.

[Lennart Poettering]

On Mon, 03.08.09 14:13, Takashi Iwai (tiwai@suse.de) wrote:

> > >> The patch introduces D-Bus support in amixer. With --dbus option given amixer
> > >> works continiously (like with the --stdin) waiting for
> > >> org.freedesktop.Hal.Device.Condition with the first argument "ButtonPressed"
> > >> and the second being one of: "volume-up", "volume-down" and "mute". These
> > >> are emmited by the HAL input helper. A single amixer process can controll
> > >> one mixer item. There are options to choose a specific item to controll,
> > >> an input device to listen and a single adjustment step size.
> > >>
> > >> Signed-off-by: Łukasz Stelmach <stlman@poczta.fm>
> > > 
> > > Thanks for the patch.
> > > 
> > > The approach is interesting, but I'm not sure whether we should put
> > > all into amixer in this way since we don't want extra library
> > > dependency.  (Note: amixer could be used in initrd.)
> > > I think it's better to split the code and make it as an individual
> > > daemon instead of additional options of amixer.
> > 
> > No worries ;) there is --with(out)--dbus option for configure to
> > disable all this code which is enough for initrd because most often
> > (AFAIK) initrd binaries are not the same that in the live system.
> 
> Usually same binaries are used for initrd nowadays in many distros.
> So, configure option doesn't help.
> 
> And, I really prefer this as a separate daemon, so it be as small as
> possible.  Seeing amixer as a daemon is somewhat strange...

Also, HAL is obsoleted by udev these days. Adding new HAL support to
software at this point in time is certainly a bad idea. On Fedora 12
we have now considerably dumbed down HAL. It is only shipped as compat
kludge for user applications. And with other distros this isn't much diffrent:

https://wiki.ubuntu.com/Halsectomy

Finally, stuff like this really belongs in the user sessions so that
OSD feedback and so on can be shown. In fact most desktops do include
a tool for this anyway.

Lennart

-- 
Lennart Poettering                        Red Hat, Inc.
lennart [at] poettering [dot] net
http://0pointer.net/lennart/           GnuPG 0x1A015CC4
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

Re: [PATCH - amixer] listen to D-Bus for volume-up, volume-down and mute signals.

[Lukasz Stelmach]

[-- Attachment #1: Type: text/plain, Size: 1542 bytes --]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Mark Brown pisze:
> On Mon, Aug 03, 2009 at 02:13:16PM +0200, Takashi Iwai wrote:
>> Lukasz Stelmach wrote:
> 
>>> No worries ;) there is --with(out)--dbus option for configure to
>>> disable all this code which is enough for initrd because most often
>>> (AFAIK) initrd binaries are not the same that in the live system.
> 
>> Usually same binaries are used for initrd nowadays in many distros.
>> So, configure option doesn't help.
> 
>> And, I really prefer this as a separate daemon, so it be as small as
>> possible.  Seeing amixer as a daemon is somewhat strange...
> 
> I tend to agree, merging it into amixer seems kind of arbatrary.  Such a
> daemon (and its dbus interface) may also be useful for handling things
> like scenario management so I can see it growing over time.

Scenario management? Could you please say something more what I
have not been missing for so long ;-)

- --
Było mi bardzo miło.               Czwarta pospolita klęska, [...]
>Łukasz<                 Już nie katolicka lecz złodziejska.  (c)PP
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkp3F3QACgkQNdzY8sm9K9wLpgCcC46pzInBHGBIHXmB8fCWqkel
QDQAmwTvtJHJuHMkqQv4GvEfKGzfUFhR
=J5+H
-----END PGP SIGNATURE-----


----------------------------------------------------------------------
Zostan Dziewczyna lub Chlopakiem lata!
Wygraj skuter >> http://link.interia.pl/f22a7 

[-- Attachment #2: stlman.vcf --]
[-- Type: text/x-vcard, Size: 177 bytes --]

begin:vcard
fn;quoted-printable:=C5=81ukasz Stelmach
n;quoted-printable:Stelmach;=C5=81ukasz
email;internet:stlman@poczta.fm
x-mozilla-html:FALSE
version:2.1
end:vcard


[-- Attachment #3: Type: text/plain, Size: 160 bytes --]

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

Re: [PATCH - amixer] listen to D-Bus for volume-up, volume-down and mute signals.

[Mark Brown]

On Mon, Aug 03, 2009 at 06:59:32PM +0200, Lukasz Stelmach wrote:
> Mark Brown pisze:

> > I tend to agree, merging it into amixer seems kind of arbatrary.  Such a
> > daemon (and its dbus interface) may also be useful for handling things
> > like scenario management so I can see it growing over time.

> Scenario management? Could you please say something more what I
> have not been missing for so long ;-)

See http://www.slimlogic.co.uk/?p=40

Re: [PATCH - amixer] listen to D-Bus for volume-up, volume-down and mute signals.

[Lukasz Stelmach]

[-- Attachment #1: Type: text/plain, Size: 2415 bytes --]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Takashi Iwai pisze:
> At Mon, 03 Aug 2009 14:04:09 +0200,
> Lukasz Stelmach wrote:
>> Takashi Iwai pisze:
>>> At Mon,  3 Aug 2009 12:43:21 +0200,
>>> Łukasz Stelmach wrote:
>>>> The patch introduces D-Bus support in amixer. With --dbus option given amixer
>>>> works continiously (like with the --stdin) waiting for
>>>> org.freedesktop.Hal.Device.Condition with the first argument "ButtonPressed"
>>>> and the second being one of: "volume-up", "volume-down" and "mute".
[...]
>>>> Signed-off-by: Łukasz Stelmach <stlman@poczta.fm>
>>> Thanks for the patch.
>>>
>>> The approach is interesting, but I'm not sure whether we should put
>>> all into amixer in this way since we don't want extra library
>>> dependency.  (Note: amixer could be used in initrd.)
>>> I think it's better to split the code and make it as an individual
>>> daemon instead of additional options of amixer.
>> No worries ;) there is --with(out)--dbus option for configure to
>> disable all this code which is enough for initrd because most often
>> (AFAIK) initrd binaries are not the same that in the live system.
> 
> Usually same binaries are used for initrd nowadays in many distros.
> So, configure option doesn't help.

Makefile rules to build two different executable files?

> And, I really prefer this as a separate daemon, so it be as small as
> possible.

I've done some testing with all the code unused in dbus scenario
ifdefed out and it was not any smaller than full a mixer with
the patch. In fact I haven't tried to hard to understand all the
string parsing and control code, however, there doesn't seem
to be much to throw out if one wants a flexible D-Bus only tool.

>  Seeing amixer as a daemon is somewhat strange...

What about --stdin option?

- --
Było mi bardzo miło.               Czwarta pospolita klęska, [...]
>Łukasz<                 Już nie katolicka lecz złodziejska.  (c)PP
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkp3GTkACgkQNdzY8sm9K9xvdgCgh4oQzK7hbkPIbe+pLPbz89ha
ohwAnifsxiAZ1LraJzjdFPjdtR6XWL5k
=fjE8
-----END PGP SIGNATURE-----


----------------------------------------------------------------------
Zostan Dziewczyna lub Chlopakiem lata!
Wygraj skuter >> http://link.interia.pl/f22a7 

[-- Attachment #2: stlman.vcf --]
[-- Type: text/x-vcard, Size: 177 bytes --]

begin:vcard
fn;quoted-printable:=C5=81ukasz Stelmach
n;quoted-printable:Stelmach;=C5=81ukasz
email;internet:stlman@poczta.fm
x-mozilla-html:FALSE
version:2.1
end:vcard


[-- Attachment #3: Type: text/plain, Size: 160 bytes --]

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

Re: [PATCH - amixer] listen to D-Bus for volume-up, volume-down and mute signals.

[Lukasz Stelmach]

[-- Attachment #1: Type: text/plain, Size: 2066 bytes --]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Lennart Poettering pisze:
> On Mon, 03.08.09 14:13, Takashi Iwai (tiwai@suse.de) wrote:
> 
>>>>> The patch introduces D-Bus support in amixer. With --dbus option given amixer
>>>>> works continiously (like with the --stdin) waiting for
>>>>> org.freedesktop.Hal.Device.Condition with the first argument "ButtonPressed"
>>>>> and the second being one of: "volume-up", "volume-down" and "mute". These
>>>>> are emmited by the HAL input helper. 
[...]
> Also, HAL is obsoleted by udev these days. Adding new HAL support to
> software at this point in time is certainly a bad idea. On Fedora 12
> we have now considerably dumbed down HAL. It is only shipped as compat
> kludge for user applications. And with other distros this isn't much diffrent:
> 
> https://wiki.ubuntu.com/Halsectomy

I must have been living under a stone. OK I see this is rather strong
point. In fact it seems reasonable to do that stuff in udev, though,
I liked HAL D-Bus interfaces.

> Finally, stuff like this really belongs in the user sessions so that
> OSD feedback and so on can be shown.

I knew I've forgotten something. XOSD!

> In fact most desktops do include a tool for this anyway.

I don't use any of the bloated desktop environments and I couldn't
find a tool that does the job. Can you name one and I promise
I won't trouble any one here with my ideas. If you don't... I won't
be bothering you too much either ;-)

- --
Było mi bardzo miło.               Czwarta pospolita klęska, [...]
>Łukasz<                 Już nie katolicka lecz złodziejska.  (c)PP
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkp3Hm0ACgkQNdzY8sm9K9ySCwCeJy4oxvK3ACoe7/ABNTm/rl3O
jbYAnAjluQcKpa1UC5toLgzF2jnLt4qL
=Il+R
-----END PGP SIGNATURE-----


----------------------------------------------------------------------
Zostan Dziewczyna lub Chlopakiem lata!
Wygraj skuter >> http://link.interia.pl/f22a7 

[-- Attachment #2: stlman.vcf --]
[-- Type: text/x-vcard, Size: 177 bytes --]

begin:vcard
fn;quoted-printable:=C5=81ukasz Stelmach
n;quoted-printable:Stelmach;=C5=81ukasz
email;internet:stlman@poczta.fm
x-mozilla-html:FALSE
version:2.1
end:vcard


[-- Attachment #3: Type: text/plain, Size: 160 bytes --]

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

Re: [PATCH - amixer] listen to D-Bus for volume-up, volume-down and mute signals.

[Lukasz Stelmach]

[-- Attachment #1: Type: text/plain, Size: 1198 bytes --]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Mark Brown pisze:
> On Mon, Aug 03, 2009 at 06:59:32PM +0200, Lukasz Stelmach wrote:
>> Mark Brown pisze:
> 
>>> I tend to agree, merging it into amixer seems kind of arbatrary.  Such a
>>> daemon (and its dbus interface) may also be useful for handling things
>>> like scenario management so I can see it growing over time.
> 
>> Scenario management? Could you please say something more what I
>> have not been missing for so long ;-)
> 
> See http://www.slimlogic.co.uk/?p=40

Thanks. That's definitely not anything I had in mind doing that patch.

- --
Było mi bardzo miło.               Czwarta pospolita klęska, [...]
>Łukasz<                 Już nie katolicka lecz złodziejska.  (c)PP
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkp3H4gACgkQNdzY8sm9K9xFYwCeM5d8v2VohuO3zSBJ6oAHYuA+
iXMAn3QTazTV/QDtDfveCaHr+wwSeB9G
=el5A
-----END PGP SIGNATURE-----


----------------------------------------------------------------------
Zostan Dziewczyna lub Chlopakiem lata!
Wygraj skuter >> http://link.interia.pl/f22a7 

[-- Attachment #2: stlman.vcf --]
[-- Type: text/x-vcard, Size: 177 bytes --]

begin:vcard
fn;quoted-printable:=C5=81ukasz Stelmach
n;quoted-printable:Stelmach;=C5=81ukasz
email;internet:stlman@poczta.fm
x-mozilla-html:FALSE
version:2.1
end:vcard


[-- Attachment #3: Type: text/plain, Size: 160 bytes --]

_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
http://mailman.alsa-project.org/mailman/listinfo/alsa-devel

Re: [PATCH - amixer] listen to D-Bus for volume-up, volume-down and mute signals.

[Liam Girdwood]

On Mon, 2009-08-03 at 19:34 +0200, Lukasz Stelmach wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> Mark Brown pisze:
> > On Mon, Aug 03, 2009 at 06:59:32PM +0200, Lukasz Stelmach wrote:
> >> Mark Brown pisze:
> > 
> >>> I tend to agree, merging it into amixer seems kind of arbatrary.  Such a
> >>> daemon (and its dbus interface) may also be useful for handling things
> >>> like scenario management so I can see it growing over time.
> > 
> >> Scenario management? Could you please say something more what I
> >> have not been missing for so long ;-)
> > 
> > See http://www.slimlogic.co.uk/?p=40
> 
> Thanks. That's definitely not anything I had in mind doing that patch.
> 

We are about to start another sprint on the scenario manager in
September. Eventual aim would be to merge in into alsa-lib and salsa for
audio hardware control abstraction. i.e. abstract per device/card
hardware audio mixers/muxes/gains/attenuations/switches settings into a
generic and meaningful scenario (or use case) e.g. "Headset Phone Call".

dbus integration is something that interests me. Please take a look at
the scenario lib public header and let me know if the API fits into your
design.

http://git.slimlogic.co.uk/?p=alsa-scenario.git;a=blob;f=include/ascenario.h;h=fd341133a601e121978054d5b2064a7a1b2d78b7;hb=8bd7f83d847424eb44704cf1649974b2fda8d8f1


Thanks

Liam 

Re: [PATCH - amixer] listen to D-Bus for volume-up, volume-down and mute signals.

[Lennart Poettering]

On Mon, 03.08.09 19:29, Lukasz Stelmach (stlman@poczta.fm) wrote:

> > In fact most desktops do include a tool for this anyway.
> 
> I don't use any of the bloated desktop environments and I couldn't
> find a tool that does the job. Can you name one and I promise
> I won't trouble any one here with my ideas. If you don't... I won't
> be bothering you too much either ;-)

Hmm, you consider GNOME/KDE bloat but then use that as an excuse to
add bloat to amixer?

Lennart

-- 
Lennart Poettering                        Red Hat, Inc.
lennart [at] poettering [dot] net
http://0pointer.net/lennart/           GnuPG 0x1A015CC4

Re: [PATCH - amixer] listen to D-Bus for volume-up, volume-down and mute signals.

[Colin Guthrie]

'Twas brillig, and Lukasz Stelmach at 03/08/09 18:29 did gyre and gimble:
> I knew I've forgotten something. XOSD!

Hehe, not quite!

> I don't use any of the bloated desktop environments and I couldn't
> find a tool that does the job. Can you name one and I promise
> I won't trouble any one here with my ideas. If you don't... I won't
> be bothering you too much either ;-)

Well I'd have a play with Gnome (whose current interface is much nicer 
than the KDE one), which pops up a nice translucent display whenever I 
press the buttons on my lappy's volume control panel.

Not sure which process actually does this UI, but I'm sure it wouldn't 
take long to see it.

Just google for various screenshots (including some Ubuntu patches to 
add it to notification daemon instead of standalone).

Col

-- 

Colin Guthrie
gmane(at)colin.guthr.ie
http://colin.guthr.ie/

Day Job:
   Tribalogic Limited [http://www.tribalogic.net/]
Open Source:
   Mandriva Linux Contributor [http://www.mandriva.com/]
   PulseAudio Hacker [http://www.pulseaudio.org/]
   Trac Hacker [http://trac.edgewall.org/]