mirrors/alsa-lib
1
0
Fork 0
mirror of https://github.com/alsa-project/alsa-lib.git synced 2025-10-29 05:40:25 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

ucm: add existence checks to geti calls

Browse source 
Right now in snd_use_case_geti you cannot tell if the item being queried
exists or not when being checked. This also means the only way to check
for the existence of something in the client of the library is to
iterate over the list of mods/devs even if we know exactly the name we
are looking for. We have functions that do exactly this internally so
lets return this information in a logical fashion through geti.

Also clean up some trailing white space nearby.

Signed-off-by: Curtis Malainey <cujomalainey@chromium.org>
Signed-off-by: Jaroslav Kysela <perex@perex.cz>
This commit is contained in:
Curtis Malainey 2023-01-13 13:21:55 -08:00 • committed by Jaroslav Kysela
parent d25ddafaa7
commit 436cd5b6d0
1 changed files with 67 additions and 16 deletions
Download patch file Download diff file Expand all files Collapse all files

83
src/ucm/main.c
View file

@ -2415,12 +2415,73 @@ int snd_use_case_get(snd_use_case_mgr_t *uc_mgr,
return err;
}
/**
* \brief check device status and existance
* \param uc_mgr Use case manager
* \param str device identifier
* \param value Value pointer
* \return Zero if success, otherwise a negative error code
*/
static long check_device(snd_use_case_mgr_t *uc_mgr,
const char *str,
long *value)
{
struct use_case_device *dev;
int err;
if (!str) {
return -EINVAL;
}
err = device_status(uc_mgr, str);
if (err > 0) {
*value = err;
err = 0;
} else if (err < 0) {
return err;
}
dev = find_device(uc_mgr, uc_mgr->active_verb, str, 0);
if (!dev) {
return -ENOENT;
}
return 0;
}
/**
* \brief check modifier status and existance
* \param uc_mgr Use case manager
* \param str modifier identifier
* \param value Value pointer
* \return Zero if success, otherwise a negative error code
*/
static long check_modifier(snd_use_case_mgr_t *uc_mgr,
const char *str,
long *value)
{
struct use_case_modifier *mod;
long err;
if (!str) {
return -EINVAL;
}
err = modifier_status(uc_mgr, str);
if (err > 0) {
*value = err;
return 0;
} else if (err < 0) {
return err;
}
mod = find_modifier(uc_mgr, uc_mgr->active_verb, str, 0);
if (!mod) {
return -ENOENT;
}
return 0;
}
/**
* \brief Get current - integer
* \param uc_mgr Use case manager
* \param identifier
* \return Value if success, otherwise a negative error code
* \param identifier
* \return Value if success, otherwise a negative error code
*/
int snd_use_case_geti(snd_use_case_mgr_t *uc_mgr,
const char *identifier,
@ -2444,25 +2505,15 @@ int snd_use_case_geti(snd_use_case_mgr_t *uc_mgr,
str = NULL;
}
if (check_identifier(identifier, "_devstatus")) {
if (!str) {
err = -EINVAL;
err = check_device(uc_mgr, str, value);
if (err < 0) {
goto __end;
}
err = device_status(uc_mgr, str);
if (err >= 0) {
*value = err;
err = 0;
}
} else if (check_identifier(identifier, "_modstatus")) {
if (!str) {
err = -EINVAL;
err = check_modifier(uc_mgr, str, value);
if (err < 0) {
goto __end;
}
err = modifier_status(uc_mgr, str);
if (err >= 0) {
*value = err;
err = 0;
}
#if 0
/*
* enable this block if the else clause below is expanded to query