mirrors/pipewire
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/pipewire/pipewire.git synced 2025-11-04 13:30:12 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

alsa-ucm: Always create device conflicting/supported device idxsets

Browse source 
This is intended to make the current and upcoming code a bit clearer, as
we won't need to constantly check for the existence of these idxsets
before using or operating on them.

Link: d8c89de24d
Signed-off-by: Alper Nebi Yasak <alpernebiyasak@gmail.com>
This commit is contained in:
Alper Nebi Yasak 2023-11-27 20:19:31 +03:00 • committed by Wim Taymans
parent 9c868740c2
commit 36af5f074a
1 changed files with 27 additions and 37 deletions
Download patch file Download diff file Expand all files Collapse all files

32
spa/plugins/alsa/acp/alsa-ucm.c
View file

@ -506,10 +506,10 @@ static int ucm_get_device_property(
n_confdev = snd_use_case_get_list(uc_mgr, id, &devices); n_confdev = snd_use_case_get_list(uc_mgr, id, &devices);
pa_xfree(id); pa_xfree(id);
device->conflicting_devices = pa_idxset_new(pa_idxset_trivial_hash_func, pa_idxset_trivial_compare_func);
if (n_confdev <= 0) if (n_confdev <= 0)
pa_log_debug("No %s for device %s", "_conflictingdevs", device_name); pa_log_debug("No %s for device %s", "_conflictingdevs", device_name);
else { else {
device->conflicting_devices = pa_idxset_new(pa_idxset_trivial_hash_func, pa_idxset_trivial_compare_func);
ucm_add_devices_to_idxset(device->conflicting_devices, device, verb->devices, devices, n_confdev); ucm_add_devices_to_idxset(device->conflicting_devices, device, verb->devices, devices, n_confdev);
snd_use_case_free_list(devices, n_confdev); snd_use_case_free_list(devices, n_confdev);
} }
@ -518,10 +518,10 @@ static int ucm_get_device_property(
n_suppdev = snd_use_case_get_list(uc_mgr, id, &devices); n_suppdev = snd_use_case_get_list(uc_mgr, id, &devices);
pa_xfree(id); pa_xfree(id);
device->supported_devices = pa_idxset_new(pa_idxset_trivial_hash_func, pa_idxset_trivial_compare_func);
if (n_suppdev <= 0) if (n_suppdev <= 0)
pa_log_debug("No %s for device %s", "_supporteddevs", device_name); pa_log_debug("No %s for device %s", "_supporteddevs", device_name);
else { else {
device->supported_devices = pa_idxset_new(pa_idxset_trivial_hash_func, pa_idxset_trivial_compare_func);
ucm_add_devices_to_idxset(device->supported_devices, device, verb->devices, devices, n_suppdev); ucm_add_devices_to_idxset(device->supported_devices, device, verb->devices, devices, n_suppdev);
snd_use_case_free_list(devices, n_suppdev); snd_use_case_free_list(devices, n_suppdev);
} }
@ -718,30 +718,18 @@ static void append_lost_relationship(pa_alsa_ucm_device *dev) {
uint32_t idx; uint32_t idx;
pa_alsa_ucm_device *d; pa_alsa_ucm_device *d;
if (dev->conflicting_devices) { PA_IDXSET_FOREACH(d, dev->conflicting_devices, idx)
PA_IDXSET_FOREACH(d, dev->conflicting_devices, idx) {
if (!d->conflicting_devices)
d->conflicting_devices = pa_idxset_new(pa_idxset_trivial_hash_func, pa_idxset_trivial_compare_func);
if (pa_idxset_put(d->conflicting_devices, dev, NULL) == 0) if (pa_idxset_put(d->conflicting_devices, dev, NULL) == 0)
pa_log_warn("Add lost conflicting device %s to %s", pa_log_warn("Add lost conflicting device %s to %s",
pa_proplist_gets(dev->proplist, PA_ALSA_PROP_UCM_NAME), pa_proplist_gets(dev->proplist, PA_ALSA_PROP_UCM_NAME),
pa_proplist_gets(d->proplist, PA_ALSA_PROP_UCM_NAME)); pa_proplist_gets(d->proplist, PA_ALSA_PROP_UCM_NAME));
}
}
if (dev->supported_devices) {
PA_IDXSET_FOREACH(d, dev->supported_devices, idx) {
if (!d->supported_devices)
d->supported_devices = pa_idxset_new(pa_idxset_trivial_hash_func, pa_idxset_trivial_compare_func);
PA_IDXSET_FOREACH(d, dev->supported_devices, idx)
if (pa_idxset_put(d->supported_devices, dev, NULL) == 0) if (pa_idxset_put(d->supported_devices, dev, NULL) == 0)
pa_log_warn("Add lost supported device %s to %s", pa_log_warn("Add lost supported device %s to %s",
pa_proplist_gets(dev->proplist, PA_ALSA_PROP_UCM_NAME), pa_proplist_gets(dev->proplist, PA_ALSA_PROP_UCM_NAME),
pa_proplist_gets(d->proplist, PA_ALSA_PROP_UCM_NAME)); pa_proplist_gets(d->proplist, PA_ALSA_PROP_UCM_NAME));
} }
}
}
int pa_alsa_ucm_query_profiles(pa_alsa_ucm_config *ucm, int card_index) { int pa_alsa_ucm_query_profiles(pa_alsa_ucm_config *ucm, int card_index) {
char *card_name; char *card_name;
@ -1225,8 +1213,8 @@ static int ucm_check_conformance(
return 1; return 1;
} }
if (dev->conflicting_devices) { /* the device defines conflicting devices */
PA_IDXSET_FOREACH(d, dev->conflicting_devices, idx) { PA_IDXSET_FOREACH(d, dev->conflicting_devices, idx) {
/* No conflicting device must already be selected */
for (i = 0; i < dev_num; i++) { for (i = 0; i < dev_num; i++) {
if (pdevices[i] == d) { if (pdevices[i] == d) {
pa_log_debug("Conflicting device found"); pa_log_debug("Conflicting device found");
@ -1234,14 +1222,18 @@ static int ucm_check_conformance(
} }
} }
} }
} else if (dev->supported_devices) { /* the device defines supported devices */
if (!pa_idxset_isempty(dev->supported_devices)) {
/* No already selected device must be unsupported */
for (i = 0; i < dev_num; i++) { for (i = 0; i < dev_num; i++) {
if (!ucm_device_exists(dev->supported_devices, pdevices[i])) { if (!ucm_device_exists(dev->supported_devices, pdevices[i])) {
pa_log_debug("Supported device not found"); pa_log_debug("Supported device not found");
return 0; return 0;
} }
} }
} else { /* not support any other devices */ }
if (pa_idxset_isempty(dev->conflicting_devices) && pa_idxset_isempty(dev->supported_devices)) {
pa_log_debug("Not support any other devices"); pa_log_debug("Not support any other devices");
return 0; return 0;
} }
@ -2145,9 +2137,7 @@ static void free_verb(pa_alsa_ucm_verb *verb) {
pa_proplist_free(di->proplist); pa_proplist_free(di->proplist);
if (di->conflicting_devices)
pa_idxset_free(di->conflicting_devices, NULL); pa_idxset_free(di->conflicting_devices, NULL);
if (di->supported_devices)
pa_idxset_free(di->supported_devices, NULL); pa_idxset_free(di->supported_devices, NULL);
pa_xfree(di->eld_mixer_device_name); pa_xfree(di->eld_mixer_device_name);