[pulseaudio-commits] 2 commits - configure.ac PROTOCOL src/pulse src/pulsecore src/utils
Tanu Kaskinen
tanuk at kemper.freedesktop.org
Mon Jul 15 09:52:09 PDT 2013
PROTOCOL | 9 +++++++++
configure.ac | 2 +-
src/pulse/introspect.c | 33 ++++++++++++++++++---------------
src/pulse/introspect.h | 6 ++++++
src/pulsecore/protocol-native.c | 3 +++
src/utils/pactl.c | 3 ++-
6 files changed, 39 insertions(+), 17 deletions(-)
New commits:
commit 16886bbf87cb02c13f4ebe89150c55003e80cac7
Author: Luiz Augusto von Dentz <luiz.von.dentz at intel.com>
Date: Mon Jul 15 18:21:26 2013 +0300
pactl: Print available flag of card profiles
diff --git a/src/utils/pactl.c b/src/utils/pactl.c
index b6d943e..df47caa 100644
--- a/src/utils/pactl.c
+++ b/src/utils/pactl.c
@@ -575,7 +575,8 @@ static void get_card_info_callback(pa_context *c, const pa_card_info *i, int is_
printf(_("\tProfiles:\n"));
for (p = i->profiles; p->name; p++)
- printf("\t\t%s: %s (sinks: %u, sources: %u, priority: %u)\n", p->name, p->description, p->n_sinks, p->n_sources, p->priority);
+ printf("\t\t%s: %s (sinks: %u, sources: %u, priority: %u, available: %s)\n", p->name,
+ p->description, p->n_sinks, p->n_sources, p->priority, pa_yes_no(p->available));
}
if (i->active_profile)
commit b98a2e158f2969b7af8522a5632f3848cf93dd85
Author: Luiz Augusto von Dentz <luiz.von.dentz at intel.com>
Date: Mon Jul 15 18:21:25 2013 +0300
introspect: Expose card profile available flag
Available flag can be used by clients to disable the selection of
profiles unavailable.
Update protocol to 29
diff --git a/PROTOCOL b/PROTOCOL
index 185b91a..0673d86 100644
--- a/PROTOCOL
+++ b/PROTOCOL
@@ -342,6 +342,15 @@ SUBCOMMAND_SAVE_FORMATS, in reply from SUBCOMMAND_READ_FORMATS[_ALL]
(uint8_t ) PA_ENCODING_MPEG2_AAC_IEC61937 := 6
+## v29, implemented by >= 5.0
+#
+New field in all commands that send/receive profile introspection data
+(PA_COMMAND_GET_CARD_INFO)
+
+ uint32 available
+
+The field is added once for every profile.
+
#### If you just changed the protocol, read this
## module-tunnel depends on the sink/source/sink-input/source-input protocol
## internals, so if you changed these, you might have broken module-tunnel.
diff --git a/configure.ac b/configure.ac
index 0ef0645..616a990 100644
--- a/configure.ac
+++ b/configure.ac
@@ -41,7 +41,7 @@ AC_SUBST(PA_MINOR, pa_minor)
AC_SUBST(PA_MAJORMINOR, pa_major.pa_minor)
AC_SUBST(PA_API_VERSION, 12)
-AC_SUBST(PA_PROTOCOL_VERSION, 28)
+AC_SUBST(PA_PROTOCOL_VERSION, 29)
# The stable ABI for client applications, for the version info x:y:z
# always will hold y=z
diff --git a/src/pulse/introspect.c b/src/pulse/introspect.c
index 334ffd9..45e0115 100644
--- a/src/pulse/introspect.c
+++ b/src/pulse/introspect.c
@@ -886,12 +886,8 @@ static void context_get_card_info_callback(pa_pdispatch *pd, uint32_t command, u
pa_tagstruct_gets(t, &i.name) < 0 ||
pa_tagstruct_getu32(t, &i.owner_module) < 0 ||
pa_tagstruct_gets(t, &i.driver) < 0 ||
- pa_tagstruct_getu32(t, &i.n_profiles) < 0) {
-
- pa_context_fail(o->context, PA_ERR_PROTOCOL);
- card_info_free(&i);
- goto finish;
- }
+ pa_tagstruct_getu32(t, &i.n_profiles) < 0)
+ goto fail;
if (i.n_profiles > 0) {
i.profiles = pa_xnew0(pa_card_profile_info, i.n_profiles+1);
@@ -902,11 +898,15 @@ static void context_get_card_info_callback(pa_pdispatch *pd, uint32_t command, u
pa_tagstruct_gets(t, &i.profiles[j].description) < 0 ||
pa_tagstruct_getu32(t, &i.profiles[j].n_sinks) < 0 ||
pa_tagstruct_getu32(t, &i.profiles[j].n_sources) < 0 ||
- pa_tagstruct_getu32(t, &i.profiles[j].priority) < 0) {
+ pa_tagstruct_getu32(t, &i.profiles[j].priority) < 0)
+ goto fail;
- pa_context_fail(o->context, PA_ERR_PROTOCOL);
- card_info_free(&i);
- goto finish;
+ i.profiles[j].available = 1;
+ if (o->context->version >= 29) {
+ uint32_t av;
+ if (pa_tagstruct_getu32(t, &av) < 0)
+ goto fail;
+ i.profiles[j].available = av;
}
}
@@ -934,11 +934,8 @@ static void context_get_card_info_callback(pa_pdispatch *pd, uint32_t command, u
}
if (o->context->version >= 26) {
- if (fill_card_port_info(o->context, t, &i) < 0) {
- pa_context_fail(o->context, PA_ERR_PROTOCOL);
- card_info_free(&i);
- goto finish;
- }
+ if (fill_card_port_info(o->context, t, &i) < 0)
+ goto fail;
}
if (o->callback) {
@@ -958,6 +955,12 @@ static void context_get_card_info_callback(pa_pdispatch *pd, uint32_t command, u
finish:
pa_operation_done(o);
pa_operation_unref(o);
+ return;
+
+fail:
+ pa_context_fail(o->context, PA_ERR_PROTOCOL);
+ card_info_free(&i);
+ goto finish;
}
pa_operation* pa_context_get_card_info_by_index(pa_context *c, uint32_t idx, pa_card_info_cb_t cb, void *userdata) {
diff --git a/src/pulse/introspect.h b/src/pulse/introspect.h
index a833471..f199a18 100644
--- a/src/pulse/introspect.h
+++ b/src/pulse/introspect.h
@@ -452,6 +452,12 @@ typedef struct pa_card_profile_info {
uint32_t n_sinks; /**< Number of sinks this profile would create */
uint32_t n_sources; /**< Number of sources this profile would create */
uint32_t priority; /**< The higher this value is, the more useful this profile is as a default. */
+ int available;
+ /**< Is this profile available? If this is zero, meaning "unavailable",
+ * then it makes no sense to try to activate this profile. If this is
+ * non-zero, it's still not a guarantee that activating the profile will
+ * result in anything useful, it just means that the server isn't aware of
+ * any reason why the profile would definitely be useless. \since 5.0 */
} pa_card_profile_info;
/** Stores information about a specific port of a card. Please
diff --git a/src/pulsecore/protocol-native.c b/src/pulsecore/protocol-native.c
index ba5aabc..9342c1a 100644
--- a/src/pulsecore/protocol-native.c
+++ b/src/pulsecore/protocol-native.c
@@ -3287,6 +3287,9 @@ static void card_fill_tagstruct(pa_native_connection *c, pa_tagstruct *t, pa_car
pa_tagstruct_putu32(t, p->n_sinks);
pa_tagstruct_putu32(t, p->n_sources);
pa_tagstruct_putu32(t, p->priority);
+
+ if (c->version >= 29)
+ pa_tagstruct_putu32(t, (p->available != PA_AVAILABLE_NO));
}
pa_tagstruct_puts(t, card->active_profile->name);
More information about the pulseaudio-commits
mailing list