From dadb3a884de2a226a242e861b52630da41d1f903 Mon Sep 17 00:00:00 2001 From: Wim Taymans Date: Mon, 17 Aug 2020 11:08:26 +0200 Subject: [PATCH] profile: refactor code to make EnumProfile and Profile params Use the same code to enumerate the profiles and the current profile, this makes sure we have all info in the Profile param as well and don't need to do a lookup in the EnumProfile. --- spa/plugins/alsa/alsa-pcm-device.c | 74 ++++++++----- spa/plugins/bluez5/bluez5-device.c | 161 +++++++++++++++-------------- spa/plugins/jack/jack-device.c | 43 +++++--- 3 files changed, 159 insertions(+), 119 deletions(-) diff --git a/spa/plugins/alsa/alsa-pcm-device.c b/spa/plugins/alsa/alsa-pcm-device.c index 2852001b7..99ec29818 100644 --- a/spa/plugins/alsa/alsa-pcm-device.c +++ b/spa/plugins/alsa/alsa-pcm-device.c @@ -339,6 +339,50 @@ static int impl_sync(void *object, int seq) return 0; } +static struct spa_pod *build_profile(struct impl *this, struct spa_pod_builder *b, + uint32_t id, uint32_t index) +{ + struct spa_pod_frame f[2]; + const char *name, *desc; + + switch (index) { + case 0: + name = "off"; + desc = "Off"; + break; + case 1: + name = "on"; + desc = "On"; + break; + default: + errno = -EINVAL; + return NULL; + } + + spa_pod_builder_push_object(b, &f[0], SPA_TYPE_OBJECT_ParamProfile, id); + spa_pod_builder_add(b, + SPA_PARAM_PROFILE_index, SPA_POD_Int(index), + SPA_PARAM_PROFILE_name, SPA_POD_String(name), + SPA_PARAM_PROFILE_description, SPA_POD_String(desc), + 0); + if (index == 1) { + spa_pod_builder_prop(b, SPA_PARAM_PROFILE_classes, 0); + spa_pod_builder_push_struct(b, &f[1]); + if (this->n_capture) { + spa_pod_builder_add_struct(b, + SPA_POD_String("Audio/Source"), + SPA_POD_Int(this->n_capture)); + } + if (this->n_playback) { + spa_pod_builder_add_struct(b, + SPA_POD_String("Audio/Sink"), + SPA_POD_Int(this->n_playback)); + } + spa_pod_builder_pop(b, &f[1]); + } + return spa_pod_builder_pop(b, &f[0]); + +} static int impl_enum_params(void *object, int seq, uint32_t id, uint32_t start, uint32_t num, const struct spa_pod *filter) @@ -363,34 +407,10 @@ static int impl_enum_params(void *object, int seq, switch (id) { case SPA_PARAM_EnumProfile: { - struct spa_pod_frame f[2]; switch (result.index) { case 0: - param = spa_pod_builder_add_object(&b, - SPA_TYPE_OBJECT_ParamProfile, id, - SPA_PARAM_PROFILE_index, SPA_POD_Int(0), - SPA_PARAM_PROFILE_name, SPA_POD_String("Off")); - break; case 1: - spa_pod_builder_push_object(&b, &f[0], SPA_TYPE_OBJECT_ParamProfile, id); - spa_pod_builder_add(&b, - SPA_PARAM_PROFILE_index, SPA_POD_Int(1), - SPA_PARAM_PROFILE_name, SPA_POD_String("On"), - 0); - spa_pod_builder_prop(&b, SPA_PARAM_PROFILE_classes, 0); - spa_pod_builder_push_struct(&b, &f[1]); - if (this->n_capture) { - spa_pod_builder_add_struct(&b, - SPA_POD_String("Audio/Source"), - SPA_POD_Int(this->n_capture)); - } - if (this->n_playback) { - spa_pod_builder_add_struct(&b, - SPA_POD_String("Audio/Sink"), - SPA_POD_Int(this->n_playback)); - } - spa_pod_builder_pop(&b, &f[1]); - param = spa_pod_builder_pop(&b, &f[0]); + param = build_profile(this, &b, id, result.index); break; default: return 0; @@ -401,9 +421,7 @@ static int impl_enum_params(void *object, int seq, { switch (result.index) { case 0: - param = spa_pod_builder_add_object(&b, - SPA_TYPE_OBJECT_ParamProfile, id, - SPA_PARAM_PROFILE_index, SPA_POD_Int(this->profile)); + param = build_profile(this, &b, id, this->profile); break; default: return 0; diff --git a/spa/plugins/bluez5/bluez5-device.c b/spa/plugins/bluez5/bluez5-device.c index 15c319a31..288217954 100644 --- a/spa/plugins/bluez5/bluez5-device.c +++ b/spa/plugins/bluez5/bluez5-device.c @@ -224,6 +224,84 @@ static int impl_sync(void *object, int seq) return 0; } +static struct spa_pod *build_profile(struct impl *this, struct spa_pod_builder *b, + uint32_t id, uint32_t index) +{ + struct spa_bt_device *device = this->bt_dev; + struct spa_pod_frame f[2]; + const char *name, *desc; + uint32_t n_source = 0, n_sink = 0; + + switch (index) { + case 0: + name = "off"; + desc = "Off"; + break; + case 1: + { + uint32_t profile = device->connected_profiles & + (SPA_BT_PROFILE_A2DP_SINK | SPA_BT_PROFILE_A2DP_SOURCE); + name = "A2DP"; + if (profile == 0) + return NULL; + else if (profile == SPA_BT_PROFILE_A2DP_SINK) + desc = "High Fidelity Playback (A2DP Sink)"; + else if (profile == SPA_BT_PROFILE_A2DP_SOURCE) + desc = "High Fidelity Capture (A2DP Source)"; + else + desc = "High Fidelity Duplex (A2DP Source/Sink)"; + if (profile & SPA_BT_PROFILE_A2DP_SOURCE) + n_source++; + if (profile & SPA_BT_PROFILE_A2DP_SINK) + n_sink++; + break; + } + case 2: + { + uint32_t profile = device->connected_profiles & + (SPA_BT_PROFILE_HEADSET_HEAD_UNIT | SPA_BT_PROFILE_HEADSET_AUDIO_GATEWAY); + name = "HSP/HFP"; + if (profile == 0) + return NULL; + else if (profile == SPA_BT_PROFILE_HEADSET_HEAD_UNIT) + desc = "Headset Head Unit (HSP/HFP)"; + else if (profile == SPA_BT_PROFILE_HEADSET_AUDIO_GATEWAY) + desc = "Headset Audio Gateway (HSP/HFP)"; + else + desc = "Headset Audio (HSP/HFP)"; + n_source++; + n_sink++; + break; + } + default: + errno = -EINVAL; + return NULL; + } + + spa_pod_builder_push_object(b, &f[0], SPA_TYPE_OBJECT_ParamProfile, id); + spa_pod_builder_add(b, + SPA_PARAM_PROFILE_index, SPA_POD_Int(index), + SPA_PARAM_PROFILE_name, SPA_POD_String(name), + SPA_PARAM_PROFILE_description, SPA_POD_String(desc), + 0); + if (n_source > 0 || n_sink > 0) { + spa_pod_builder_prop(b, SPA_PARAM_PROFILE_classes, 0); + spa_pod_builder_push_struct(b, &f[1]); + if (n_source > 0) { + spa_pod_builder_add_struct(b, + SPA_POD_String("Audio/Source"), + SPA_POD_Int(n_source)); + } + if (n_sink > 0) { + spa_pod_builder_add_struct(b, + SPA_POD_String("Audio/Sink"), + SPA_POD_Int(n_sink)); + } + spa_pod_builder_pop(b, &f[1]); + } + return spa_pod_builder_pop(b, &f[0]); +} + static int impl_enum_params(void *object, int seq, uint32_t id, uint32_t start, uint32_t num, const struct spa_pod *filter) @@ -234,7 +312,6 @@ static int impl_enum_params(void *object, int seq, uint8_t buffer[1024]; struct spa_result_device_params result; uint32_t count = 0; - struct spa_bt_device *device = this->bt_dev; spa_return_val_if_fail(this != NULL, -EINVAL); spa_return_val_if_fail(num != 0, -EINVAL); @@ -249,84 +326,12 @@ static int impl_enum_params(void *object, int seq, switch (id) { case SPA_PARAM_EnumProfile: { - struct spa_pod_frame f[2]; switch (result.index) { - case 0: - param = spa_pod_builder_add_object(&b, - SPA_TYPE_OBJECT_ParamProfile, id, - SPA_PARAM_PROFILE_index, SPA_POD_Int(0), - SPA_PARAM_PROFILE_name, SPA_POD_String("Off")); - break; - case 1: - { - uint32_t profile = device->connected_profiles & - (SPA_BT_PROFILE_A2DP_SINK | SPA_BT_PROFILE_A2DP_SOURCE); - const char *description; - - if (profile == 0) + case 0: case 1: case 2: + param = build_profile(this, &b, id, result.index); + if (param == NULL) goto next; - else if (profile == SPA_BT_PROFILE_A2DP_SINK) - description = "High Fidelity Playback (A2DP Sink)"; - else if (profile == SPA_BT_PROFILE_A2DP_SOURCE) - description = "High Fidelity Capture (A2DP Source)"; - else - description = "High Fidelity Duplex (A2DP Source/Sink)"; - - spa_pod_builder_push_object(&b, &f[0], SPA_TYPE_OBJECT_ParamProfile, id); - spa_pod_builder_add(&b, - SPA_PARAM_PROFILE_index, SPA_POD_Int(1), - SPA_PARAM_PROFILE_name, SPA_POD_String("A2DP"), - SPA_PARAM_PROFILE_description, SPA_POD_String(description), - 0); - spa_pod_builder_prop(&b, SPA_PARAM_PROFILE_classes, 0); - spa_pod_builder_push_struct(&b, &f[1]); - if (profile & SPA_BT_PROFILE_A2DP_SOURCE) { - spa_pod_builder_add_struct(&b, - SPA_POD_String("Audio/Source"), - SPA_POD_Int(1)); - } - if (profile & SPA_BT_PROFILE_A2DP_SINK) { - spa_pod_builder_add_struct(&b, - SPA_POD_String("Audio/Sink"), - SPA_POD_Int(1)); - } - spa_pod_builder_pop(&b, &f[1]); - param = spa_pod_builder_pop(&b, &f[0]); break; - } - case 2: - { - uint32_t profile = device->connected_profiles & - (SPA_BT_PROFILE_HEADSET_HEAD_UNIT | SPA_BT_PROFILE_HEADSET_AUDIO_GATEWAY); - const char *description; - - if (profile == 0) - goto next; - else if (profile == SPA_BT_PROFILE_HEADSET_HEAD_UNIT) - description = "Headset Head Unit (HSP/HFP)"; - else if (profile == SPA_BT_PROFILE_HEADSET_AUDIO_GATEWAY) - description = "Headset Audio Gateway (HSP/HFP)"; - else - description = "Headset Audio (HSP/HFP)"; - - spa_pod_builder_push_object(&b, &f[0], SPA_TYPE_OBJECT_ParamProfile, id); - spa_pod_builder_add(&b, - SPA_PARAM_PROFILE_index, SPA_POD_Int(2), - SPA_PARAM_PROFILE_name, SPA_POD_String("HSP/HFP"), - SPA_PARAM_PROFILE_description, SPA_POD_String(description), - 0); - spa_pod_builder_prop(&b, SPA_PARAM_PROFILE_classes, 0); - spa_pod_builder_push_struct(&b, &f[1]); - spa_pod_builder_add_struct(&b, - SPA_POD_String("Audio/Source"), - SPA_POD_Int(1)); - spa_pod_builder_add_struct(&b, - SPA_POD_String("Audio/Sink"), - SPA_POD_Int(1)); - spa_pod_builder_pop(&b, &f[1]); - param = spa_pod_builder_pop(&b, &f[0]); - break; - } default: return 0; } @@ -336,9 +341,7 @@ static int impl_enum_params(void *object, int seq, { switch (result.index) { case 0: - param = spa_pod_builder_add_object(&b, - SPA_TYPE_OBJECT_ParamProfile, id, - SPA_PARAM_PROFILE_index, SPA_POD_Int(this->profile)); + param = build_profile(this, &b, id, this->profile); break; default: return 0; diff --git a/spa/plugins/jack/jack-device.c b/spa/plugins/jack/jack-device.c index fec471c02..973cd8db3 100644 --- a/spa/plugins/jack/jack-device.c +++ b/spa/plugins/jack/jack-device.c @@ -227,6 +227,35 @@ static int impl_sync(void *object, int seq) return 0; } +static struct spa_pod *build_profile(struct impl *this, struct spa_pod_builder *b, + uint32_t id, uint32_t index) +{ + struct spa_pod_frame f[2]; + const char *name, *desc; + + switch (index) { + case 0: + name = "off"; + desc = "Off"; + break; + case 1: + name = "on"; + desc = "On"; + break; + default: + errno = -EINVAL; + return NULL; + } + + spa_pod_builder_push_object(b, &f[0], SPA_TYPE_OBJECT_ParamProfile, id); + spa_pod_builder_add(b, + SPA_PARAM_PROFILE_index, SPA_POD_Int(index), + SPA_PARAM_PROFILE_name, SPA_POD_String(name), + SPA_PARAM_PROFILE_description, SPA_POD_String(desc), + 0); + return spa_pod_builder_pop(b, &f[0]); +} + static int impl_enum_params(void *object, int seq, uint32_t id, uint32_t start, uint32_t num, const struct spa_pod *filter) @@ -253,16 +282,8 @@ static int impl_enum_params(void *object, int seq, { switch (result.index) { case 0: - param = spa_pod_builder_add_object(&b, - SPA_TYPE_OBJECT_ParamProfile, id, - SPA_PARAM_PROFILE_index, SPA_POD_Int(0), - SPA_PARAM_PROFILE_name, SPA_POD_String("Off")); - break; case 1: - param = spa_pod_builder_add_object(&b, - SPA_TYPE_OBJECT_ParamProfile, id, - SPA_PARAM_PROFILE_index, SPA_POD_Int(1), - SPA_PARAM_PROFILE_name, SPA_POD_String("On")); + param = build_profile(this, &b, id, result.index); break; default: return 0; @@ -273,9 +294,7 @@ static int impl_enum_params(void *object, int seq, { switch (result.index) { case 0: - param = spa_pod_builder_add_object(&b, - SPA_TYPE_OBJECT_ParamProfile, id, - SPA_PARAM_PROFILE_index, SPA_POD_Int(this->profile)); + param = build_profile(this, &b, id, this->profile); break; default: return 0;