mirror of
https://gitlab.freedesktop.org/pulseaudio/pulseaudio.git
synced 2025-10-29 05:40:23 -04:00
alsa-ucm: Remove combination port generation logic
A previous commit makes mapping names depend on the UCM device name. Since UCM device names are unique, this means a mapping will at most have one port and thus no combination ports can be generated. This removes the dead code in the pa_alsa_ucm_add_ports_combination() function, unrolls the remaining code in its helper functions that it used, and renames it to pa_alsa_ucm_add_port() to signal that it no longer generates combinations. Signed-off-by: Alper Nebi Yasak <alpernebiyasak@gmail.com> Part-of: <https://gitlab.freedesktop.org/pulseaudio/pulseaudio/-/merge_requests/596>
This commit is contained in:
Alper Nebi Yasak
parent
a50330a4a5
commit
0789a8fb76
3 changed files with 48 additions and 105 deletions
|
|
@ -1045,7 +1045,7 @@ fail:
|
|||
}
|
||||
}
|
||||
|
||||
static char *devset_name(pa_idxset *devices, const char *sep) {
|
||||
PA_UNUSED static char *devset_name(pa_idxset *devices, const char *sep) {
|
||||
int i = 0;
|
||||
int num = pa_idxset_size(devices);
|
||||
pa_alsa_ucm_device *sorted[num], *dev;
|
||||
|
|
@ -1077,7 +1077,7 @@ static char *devset_name(pa_idxset *devices, const char *sep) {
|
|||
return dev_names;
|
||||
}
|
||||
|
||||
static char *devset_description(pa_idxset *devices, const char *sep) {
|
||||
PA_UNUSED static char *devset_description(pa_idxset *devices, const char *sep) {
|
||||
int i = 0;
|
||||
int num = pa_idxset_size(devices);
|
||||
pa_alsa_ucm_device *sorted[num], *dev;
|
||||
|
|
@ -1112,7 +1112,7 @@ static char *devset_description(pa_idxset *devices, const char *sep) {
|
|||
/* If invert is true, uses the formula 1/p = 1/p1 + 1/p2 + ... 1/pn.
|
||||
* This way, the result will always be less than the individual components,
|
||||
* yet higher components will lead to higher result. */
|
||||
static unsigned devset_playback_priority(pa_idxset *devices, bool invert) {
|
||||
PA_UNUSED static unsigned devset_playback_priority(pa_idxset *devices, bool invert) {
|
||||
pa_alsa_ucm_device *dev;
|
||||
uint32_t idx;
|
||||
double priority = 0;
|
||||
|
|
@ -1130,7 +1130,7 @@ static unsigned devset_playback_priority(pa_idxset *devices, bool invert) {
|
|||
return (unsigned) priority;
|
||||
}
|
||||
|
||||
static unsigned devset_capture_priority(pa_idxset *devices, bool invert) {
|
||||
PA_UNUSED static unsigned devset_capture_priority(pa_idxset *devices, bool invert) {
|
||||
pa_alsa_ucm_device *dev;
|
||||
uint32_t idx;
|
||||
double priority = 0;
|
||||
|
|
@ -1148,75 +1148,42 @@ static unsigned devset_capture_priority(pa_idxset *devices, bool invert) {
|
|||
return (unsigned) priority;
|
||||
}
|
||||
|
||||
static void ucm_add_port_combination(
|
||||
void pa_alsa_ucm_add_port(
|
||||
pa_hashmap *hash,
|
||||
pa_alsa_ucm_mapping_context *context,
|
||||
bool is_sink,
|
||||
pa_idxset *devices,
|
||||
pa_hashmap *ports,
|
||||
pa_card_profile *cp,
|
||||
pa_core *core) {
|
||||
|
||||
pa_device_port *port;
|
||||
int i;
|
||||
int num = pa_idxset_size(devices);
|
||||
uint32_t idx;
|
||||
unsigned priority;
|
||||
char *name, *desc, *tmp;
|
||||
char *name, *desc;
|
||||
const char *dev_name;
|
||||
const char *direction;
|
||||
const char *profile;
|
||||
pa_alsa_ucm_device *sorted[num], *dev;
|
||||
pa_alsa_ucm_device *dev;
|
||||
pa_alsa_ucm_port_data *data;
|
||||
pa_alsa_ucm_volume *vol;
|
||||
pa_alsa_jack *jack, *jack2;
|
||||
pa_device_port_type_t type, type2;
|
||||
pa_alsa_jack *jack;
|
||||
pa_device_port_type_t type;
|
||||
uint32_t idx;
|
||||
void *state;
|
||||
|
||||
i = 0;
|
||||
PA_IDXSET_FOREACH(dev, devices, idx) {
|
||||
sorted[i] = dev;
|
||||
i++;
|
||||
}
|
||||
pa_assert(context->ucm_devices);
|
||||
pa_assert(pa_idxset_size(context->ucm_devices) <= 1);
|
||||
|
||||
/* Sort by alphabetical order so as to have a deterministic naming scheme
|
||||
* for combination ports */
|
||||
qsort(&sorted[0], num, sizeof(pa_alsa_ucm_device *), pa_alsa_ucm_device_cmp);
|
||||
dev = pa_idxset_first(context->ucm_devices, &idx);
|
||||
if (!dev)
|
||||
return;
|
||||
|
||||
name = devset_name(devices, "+");
|
||||
tmp = pa_sprintf_malloc("%s%s", is_sink ? PA_UCM_PRE_TAG_OUTPUT : PA_UCM_PRE_TAG_INPUT, name);
|
||||
pa_xfree(name);
|
||||
name = tmp;
|
||||
|
||||
desc = devset_description(devices, ", ");
|
||||
if (pa_idxset_size(devices) > 1) {
|
||||
tmp = pa_sprintf_malloc("Combination port for %s", desc);
|
||||
pa_xfree(desc);
|
||||
desc = tmp;
|
||||
}
|
||||
|
||||
/* Make combination ports always have lower priority */
|
||||
priority = is_sink ? devset_playback_priority(devices, true) : devset_capture_priority(devices, true);
|
||||
|
||||
dev = sorted[0];
|
||||
dev_name = pa_proplist_gets(dev->proplist, PA_ALSA_PROP_UCM_NAME);
|
||||
name = pa_sprintf_malloc("%s%s", is_sink ? PA_UCM_PRE_TAG_OUTPUT : PA_UCM_PRE_TAG_INPUT, dev_name);
|
||||
desc = pa_xstrdup(pa_proplist_gets(dev->proplist, PA_ALSA_PROP_UCM_DESCRIPTION));
|
||||
priority = is_sink ? dev->playback_priority : dev->capture_priority;
|
||||
jack = ucm_get_jack(context->ucm, dev);
|
||||
type = dev->type;
|
||||
|
||||
for (i = 1; i < num; i++) {
|
||||
jack2 = ucm_get_jack(context->ucm, dev);
|
||||
if (jack2) {
|
||||
if (jack && jack != jack2)
|
||||
pa_log_warn("Multiple jacks per combined device '%s': '%s' '%s'", name, jack->name, jack2->name);
|
||||
jack = jack2;
|
||||
}
|
||||
|
||||
type2 = dev->type;
|
||||
if (type2 != PA_DEVICE_PORT_TYPE_UNKNOWN) {
|
||||
if (type != PA_DEVICE_PORT_TYPE_UNKNOWN && type != type2)
|
||||
pa_log_warn("Multiple device types per combined device '%s': %d %d", name, type, type2);
|
||||
type = type2;
|
||||
}
|
||||
}
|
||||
|
||||
port = pa_hashmap_get(ports, name);
|
||||
if (!port) {
|
||||
pa_device_port_new_data port_data;
|
||||
|
|
@ -1233,37 +1200,32 @@ static void ucm_add_port_combination(
|
|||
pa_device_port_new_data_done(&port_data);
|
||||
|
||||
data = PA_DEVICE_PORT_DATA(port);
|
||||
ucm_port_data_init(data, context->ucm, port, sorted, num);
|
||||
ucm_port_data_init(data, context->ucm, port, &dev, 1);
|
||||
port->impl_free = ucm_port_data_free;
|
||||
|
||||
pa_hashmap_put(ports, port->name, port);
|
||||
pa_log_debug("Add port %s: %s", port->name, port->description);
|
||||
|
||||
if (num == 1) {
|
||||
/* To keep things simple and not worry about stacking controls, we only support hardware volumes on non-combination
|
||||
* ports. */
|
||||
PA_HASHMAP_FOREACH_KV(profile, vol, is_sink ? dev->playback_volumes : dev->capture_volumes, state) {
|
||||
pa_alsa_path *path = pa_alsa_path_synthesize(vol->mixer_elem,
|
||||
is_sink ? PA_ALSA_DIRECTION_OUTPUT : PA_ALSA_DIRECTION_INPUT);
|
||||
PA_HASHMAP_FOREACH_KV(profile, vol, is_sink ? dev->playback_volumes : dev->capture_volumes, state) {
|
||||
pa_alsa_path *path = pa_alsa_path_synthesize(vol->mixer_elem,
|
||||
is_sink ? PA_ALSA_DIRECTION_OUTPUT : PA_ALSA_DIRECTION_INPUT);
|
||||
|
||||
if (!path)
|
||||
pa_log_warn("Failed to set up volume control: %s", vol->mixer_elem);
|
||||
else {
|
||||
if (vol->master_elem) {
|
||||
pa_alsa_element *e = pa_alsa_element_get(path, vol->master_elem, false);
|
||||
e->switch_use = PA_ALSA_SWITCH_MUTE;
|
||||
e->volume_use = PA_ALSA_VOLUME_MERGE;
|
||||
}
|
||||
|
||||
pa_hashmap_put(data->paths, pa_xstrdup(profile), path);
|
||||
|
||||
/* Add path also to already created empty path set */
|
||||
dev = sorted[0];
|
||||
if (is_sink)
|
||||
pa_hashmap_put(dev->playback_mapping->output_path_set->paths, pa_xstrdup(vol->mixer_elem), path);
|
||||
else
|
||||
pa_hashmap_put(dev->capture_mapping->input_path_set->paths, pa_xstrdup(vol->mixer_elem), path);
|
||||
if (!path)
|
||||
pa_log_warn("Failed to set up volume control: %s", vol->mixer_elem);
|
||||
else {
|
||||
if (vol->master_elem) {
|
||||
pa_alsa_element *e = pa_alsa_element_get(path, vol->master_elem, false);
|
||||
e->switch_use = PA_ALSA_SWITCH_MUTE;
|
||||
e->volume_use = PA_ALSA_VOLUME_MERGE;
|
||||
}
|
||||
|
||||
pa_hashmap_put(data->paths, pa_xstrdup(profile), path);
|
||||
|
||||
/* Add path also to already created empty path set */
|
||||
if (is_sink)
|
||||
pa_hashmap_put(dev->playback_mapping->output_path_set->paths, pa_xstrdup(vol->mixer_elem), path);
|
||||
else
|
||||
pa_hashmap_put(dev->capture_mapping->input_path_set->paths, pa_xstrdup(vol->mixer_elem), path);
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
@ -1285,6 +1247,9 @@ static void ucm_add_port_combination(
|
|||
pa_hashmap_put(hash, port->name, port);
|
||||
pa_device_port_ref(port);
|
||||
}
|
||||
|
||||
/* ELD devices */
|
||||
set_eld_devices(ports);
|
||||
}
|
||||
|
||||
static int ucm_port_contains(const char *port_name, const char *dev_name, bool is_sink) {
|
||||
|
|
@ -1346,7 +1311,7 @@ static bool devset_supports_device(pa_idxset *devices, pa_alsa_ucm_device *dev)
|
|||
* *state should be NULL. It's not safe to modify the devices argument
|
||||
* until iteration ends. The returned idxsets must be freed by the
|
||||
* caller. */
|
||||
static pa_idxset *iterate_device_subsets(pa_idxset *devices, void **state) {
|
||||
PA_UNUSED static pa_idxset *iterate_device_subsets(pa_idxset *devices, void **state) {
|
||||
uint32_t idx;
|
||||
pa_alsa_ucm_device *dev;
|
||||
|
||||
|
|
@ -1413,28 +1378,6 @@ static char* merge_roles(const char *cur, const char *add) {
|
|||
return ret;
|
||||
}
|
||||
|
||||
void pa_alsa_ucm_add_ports_combination(
|
||||
pa_hashmap *p,
|
||||
pa_alsa_ucm_mapping_context *context,
|
||||
bool is_sink,
|
||||
pa_hashmap *ports,
|
||||
pa_card_profile *cp,
|
||||
pa_core *core) {
|
||||
|
||||
pa_idxset *devices;
|
||||
void *state = NULL;
|
||||
|
||||
pa_assert(context->ucm_devices);
|
||||
|
||||
while ((devices = iterate_device_subsets(context->ucm_devices, &state))) {
|
||||
ucm_add_port_combination(p, context, is_sink, devices, ports, cp, core);
|
||||
pa_idxset_free(devices, NULL);
|
||||
}
|
||||
|
||||
/* ELD devices */
|
||||
set_eld_devices(ports);
|
||||
}
|
||||
|
||||
void pa_alsa_ucm_add_ports(
|
||||
pa_hashmap **p,
|
||||
pa_proplist *proplist,
|
||||
|
|
@ -1455,7 +1398,7 @@ void pa_alsa_ucm_add_ports(
|
|||
pa_assert(*p);
|
||||
|
||||
/* add ports first */
|
||||
pa_alsa_ucm_add_ports_combination(*p, context, is_sink, card->ports, NULL, card->core);
|
||||
pa_alsa_ucm_add_port(*p, context, is_sink, card->ports, NULL, card->core);
|
||||
|
||||
/* now set up volume paths if any */
|
||||
probe_volumes(*p, is_sink, pcm_handle, context->ucm->mixers, ignore_dB);
|
||||
|
|
@ -2534,7 +2477,7 @@ void pa_alsa_ucm_add_ports(
|
|||
bool ignore_dB) {
|
||||
}
|
||||
|
||||
void pa_alsa_ucm_add_ports_combination(
|
||||
void pa_alsa_ucm_add_port(
|
||||
pa_hashmap *hash,
|
||||
pa_alsa_ucm_mapping_context *context,
|
||||
bool is_sink,
|
||||
|
|
|
|||
|
|
@ -158,7 +158,7 @@ void pa_alsa_ucm_add_ports(
|
|||
pa_card *card,
|
||||
snd_pcm_t *pcm_handle,
|
||||
bool ignore_dB);
|
||||
void pa_alsa_ucm_add_ports_combination(
|
||||
void pa_alsa_ucm_add_port(
|
||||
pa_hashmap *hash,
|
||||
pa_alsa_ucm_mapping_context *context,
|
||||
bool is_sink,
|
||||
|
|
|
|||
|
|
@ -162,7 +162,7 @@ static void add_profiles(struct userdata *u, pa_hashmap *h, pa_hashmap *ports) {
|
|||
|
||||
PA_IDXSET_FOREACH(m, ap->output_mappings, idx) {
|
||||
if (u->use_ucm)
|
||||
pa_alsa_ucm_add_ports_combination(NULL, &m->ucm_context, true, ports, cp, u->core);
|
||||
pa_alsa_ucm_add_port(NULL, &m->ucm_context, true, ports, cp, u->core);
|
||||
else
|
||||
pa_alsa_path_set_add_ports(m->output_path_set, cp, ports, NULL, u->core);
|
||||
if (m->channel_map.channels > cp->max_sink_channels)
|
||||
|
|
@ -175,7 +175,7 @@ static void add_profiles(struct userdata *u, pa_hashmap *h, pa_hashmap *ports) {
|
|||
|
||||
PA_IDXSET_FOREACH(m, ap->input_mappings, idx) {
|
||||
if (u->use_ucm)
|
||||
pa_alsa_ucm_add_ports_combination(NULL, &m->ucm_context, false, ports, cp, u->core);
|
||||
pa_alsa_ucm_add_port(NULL, &m->ucm_context, false, ports, cp, u->core);
|
||||
else
|
||||
pa_alsa_path_set_add_ports(m->input_path_set, cp, ports, NULL, u->core);
|
||||
if (m->channel_map.channels > cp->max_source_channels)
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue