kselftest: alsa: Factor out check that values meet constraints
To simplify the code a bit and allow future reuse factor the checks that values we read are valid out of test_ctl_get_value() into a separate function which can be reused later. As part of this extend the test to check all the values for the control, not just the first one. Signed-off-by: Mark Brown <broonie@kernel.org> Reviewed-by: Cezary Rojewski <cezary.rojewski@intel.com> Link: https://lore.kernel.org/r/20211217130213.3893415-2-broonie@kernel.org Signed-off-by: Takashi Iwai <tiwai@suse.de>
This commit is contained in:
Mark Brown
Takashi Iwai
parent
0f7e5ee62f
commit
3f48b137d8
1 changed files with 109 additions and 86 deletions
|
|
@ -193,6 +193,113 @@ void find_controls(void)
|
||||||
snd_config_delete(config);
|
snd_config_delete(config);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
bool ctl_value_index_valid(struct ctl_data *ctl, snd_ctl_elem_value_t *val,
|
||||||
|
int index)
|
||||||
|
{
|
||||||
|
long int_val;
|
||||||
|
long long int64_val;
|
||||||
|
|
||||||
|
switch (snd_ctl_elem_info_get_type(ctl->info)) {
|
||||||
|
case SND_CTL_ELEM_TYPE_NONE:
|
||||||
|
ksft_print_msg("%s.%d Invalid control type NONE\n",
|
||||||
|
ctl->name, index);
|
||||||
|
return false;
|
||||||
|
|
||||||
|
case SND_CTL_ELEM_TYPE_BOOLEAN:
|
||||||
|
int_val = snd_ctl_elem_value_get_boolean(val, index);
|
||||||
|
switch (int_val) {
|
||||||
|
case 0:
|
||||||
|
case 1:
|
||||||
|
break;
|
||||||
|
default:
|
||||||
|
ksft_print_msg("%s.%d Invalid boolean value %ld\n",
|
||||||
|
ctl->name, index, int_val);
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
break;
|
||||||
|
|
||||||
|
case SND_CTL_ELEM_TYPE_INTEGER:
|
||||||
|
int_val = snd_ctl_elem_value_get_integer(val, index);
|
||||||
|
|
||||||
|
if (int_val < snd_ctl_elem_info_get_min(ctl->info)) {
|
||||||
|
ksft_print_msg("%s.%d value %ld less than minimum %ld\n",
|
||||||
|
ctl->name, index, int_val,
|
||||||
|
snd_ctl_elem_info_get_min(ctl->info));
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (int_val > snd_ctl_elem_info_get_max(ctl->info)) {
|
||||||
|
ksft_print_msg("%s.%d value %ld more than maximum %ld\n",
|
||||||
|
ctl->name, index, int_val,
|
||||||
|
snd_ctl_elem_info_get_max(ctl->info));
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
/* Only check step size if there is one and we're in bounds */
|
||||||
|
if (snd_ctl_elem_info_get_step(ctl->info) &&
|
||||||
|
(int_val - snd_ctl_elem_info_get_min(ctl->info) %
|
||||||
|
snd_ctl_elem_info_get_step(ctl->info))) {
|
||||||
|
ksft_print_msg("%s.%d value %ld invalid for step %ld minimum %ld\n",
|
||||||
|
ctl->name, index, int_val,
|
||||||
|
snd_ctl_elem_info_get_step(ctl->info),
|
||||||
|
snd_ctl_elem_info_get_min(ctl->info));
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
break;
|
||||||
|
|
||||||
|
case SND_CTL_ELEM_TYPE_INTEGER64:
|
||||||
|
int64_val = snd_ctl_elem_value_get_integer64(val, index);
|
||||||
|
|
||||||
|
if (int64_val < snd_ctl_elem_info_get_min64(ctl->info)) {
|
||||||
|
ksft_print_msg("%s.%d value %lld less than minimum %lld\n",
|
||||||
|
ctl->name, index, int64_val,
|
||||||
|
snd_ctl_elem_info_get_min64(ctl->info));
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (int64_val > snd_ctl_elem_info_get_max64(ctl->info)) {
|
||||||
|
ksft_print_msg("%s.%d value %lld more than maximum %lld\n",
|
||||||
|
ctl->name, index, int64_val,
|
||||||
|
snd_ctl_elem_info_get_max(ctl->info));
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
|
||||||
|
/* Only check step size if there is one and we're in bounds */
|
||||||
|
if (snd_ctl_elem_info_get_step64(ctl->info) &&
|
||||||
|
(int64_val - snd_ctl_elem_info_get_min64(ctl->info)) %
|
||||||
|
snd_ctl_elem_info_get_step64(ctl->info)) {
|
||||||
|
ksft_print_msg("%s.%d value %lld invalid for step %lld minimum %lld\n",
|
||||||
|
ctl->name, index, int64_val,
|
||||||
|
snd_ctl_elem_info_get_step64(ctl->info),
|
||||||
|
snd_ctl_elem_info_get_min64(ctl->info));
|
||||||
|
return false;
|
||||||
|
}
|
||||||
|
break;
|
||||||
|
|
||||||
|
default:
|
||||||
|
/* No tests for other types */
|
||||||
|
break;
|
||||||
|
}
|
||||||
|
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
|
||||||
|
/*
|
||||||
|
* Check that the provided value meets the constraints for the
|
||||||
|
* provided control.
|
||||||
|
*/
|
||||||
|
bool ctl_value_valid(struct ctl_data *ctl, snd_ctl_elem_value_t *val)
|
||||||
|
{
|
||||||
|
int i;
|
||||||
|
bool valid = true;
|
||||||
|
|
||||||
|
for (i = 0; i < snd_ctl_elem_info_get_count(ctl->info); i++)
|
||||||
|
if (!ctl_value_index_valid(ctl, val, i))
|
||||||
|
valid = false;
|
||||||
|
|
||||||
|
return valid;
|
||||||
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* Check that we can read the default value and it is valid. Write
|
* Check that we can read the default value and it is valid. Write
|
||||||
* tests use the read value to restore the default.
|
* tests use the read value to restore the default.
|
||||||
|
|
@ -200,8 +307,6 @@ void find_controls(void)
|
||||||
void test_ctl_get_value(struct ctl_data *ctl)
|
void test_ctl_get_value(struct ctl_data *ctl)
|
||||||
{
|
{
|
||||||
int err;
|
int err;
|
||||||
long int_val;
|
|
||||||
long long int64_val;
|
|
||||||
|
|
||||||
/* If the control is turned off let's be polite */
|
/* If the control is turned off let's be polite */
|
||||||
if (snd_ctl_elem_info_is_inactive(ctl->info)) {
|
if (snd_ctl_elem_info_is_inactive(ctl->info)) {
|
||||||
|
|
@ -226,90 +331,8 @@ void test_ctl_get_value(struct ctl_data *ctl)
|
||||||
goto out;
|
goto out;
|
||||||
}
|
}
|
||||||
|
|
||||||
switch (snd_ctl_elem_info_get_type(ctl->info)) {
|
if (!ctl_value_valid(ctl, ctl->def_val))
|
||||||
case SND_CTL_ELEM_TYPE_NONE:
|
err = -EINVAL;
|
||||||
ksft_print_msg("%s Invalid control type NONE\n", ctl->name);
|
|
||||||
err = -1;
|
|
||||||
break;
|
|
||||||
|
|
||||||
case SND_CTL_ELEM_TYPE_BOOLEAN:
|
|
||||||
int_val = snd_ctl_elem_value_get_boolean(ctl->def_val, 0);
|
|
||||||
switch (int_val) {
|
|
||||||
case 0:
|
|
||||||
case 1:
|
|
||||||
break;
|
|
||||||
default:
|
|
||||||
ksft_print_msg("%s Invalid boolean value %ld\n",
|
|
||||||
ctl->name, int_val);
|
|
||||||
err = -1;
|
|
||||||
break;
|
|
||||||
}
|
|
||||||
break;
|
|
||||||
|
|
||||||
case SND_CTL_ELEM_TYPE_INTEGER:
|
|
||||||
int_val = snd_ctl_elem_value_get_integer(ctl->def_val, 0);
|
|
||||||
|
|
||||||
if (int_val < snd_ctl_elem_info_get_min(ctl->info)) {
|
|
||||||
ksft_print_msg("%s value %ld less than minimum %ld\n",
|
|
||||||
ctl->name, int_val,
|
|
||||||
snd_ctl_elem_info_get_min(ctl->info));
|
|
||||||
err = -1;
|
|
||||||
}
|
|
||||||
|
|
||||||
if (int_val > snd_ctl_elem_info_get_max(ctl->info)) {
|
|
||||||
ksft_print_msg("%s value %ld more than maximum %ld\n",
|
|
||||||
ctl->name, int_val,
|
|
||||||
snd_ctl_elem_info_get_max(ctl->info));
|
|
||||||
err = -1;
|
|
||||||
}
|
|
||||||
|
|
||||||
/* Only check step size if there is one and we're in bounds */
|
|
||||||
if (err >= 0 && snd_ctl_elem_info_get_step(ctl->info) &&
|
|
||||||
(int_val - snd_ctl_elem_info_get_min(ctl->info) %
|
|
||||||
snd_ctl_elem_info_get_step(ctl->info))) {
|
|
||||||
ksft_print_msg("%s value %ld invalid for step %ld minimum %ld\n",
|
|
||||||
ctl->name, int_val,
|
|
||||||
snd_ctl_elem_info_get_step(ctl->info),
|
|
||||||
snd_ctl_elem_info_get_min(ctl->info));
|
|
||||||
err = -1;
|
|
||||||
}
|
|
||||||
break;
|
|
||||||
|
|
||||||
case SND_CTL_ELEM_TYPE_INTEGER64:
|
|
||||||
int64_val = snd_ctl_elem_value_get_integer64(ctl->def_val, 0);
|
|
||||||
|
|
||||||
if (int64_val < snd_ctl_elem_info_get_min64(ctl->info)) {
|
|
||||||
ksft_print_msg("%s value %lld less than minimum %lld\n",
|
|
||||||
ctl->name, int64_val,
|
|
||||||
snd_ctl_elem_info_get_min64(ctl->info));
|
|
||||||
err = -1;
|
|
||||||
}
|
|
||||||
|
|
||||||
if (int64_val > snd_ctl_elem_info_get_max64(ctl->info)) {
|
|
||||||
ksft_print_msg("%s value %lld more than maximum %lld\n",
|
|
||||||
ctl->name, int64_val,
|
|
||||||
snd_ctl_elem_info_get_max(ctl->info));
|
|
||||||
err = -1;
|
|
||||||
}
|
|
||||||
|
|
||||||
/* Only check step size if there is one and we're in bounds */
|
|
||||||
if (err >= 0 && snd_ctl_elem_info_get_step64(ctl->info) &&
|
|
||||||
(int64_val - snd_ctl_elem_info_get_min64(ctl->info)) %
|
|
||||||
snd_ctl_elem_info_get_step64(ctl->info)) {
|
|
||||||
ksft_print_msg("%s value %lld invalid for step %lld minimum %lld\n",
|
|
||||||
ctl->name, int64_val,
|
|
||||||
snd_ctl_elem_info_get_step64(ctl->info),
|
|
||||||
snd_ctl_elem_info_get_min64(ctl->info));
|
|
||||||
err = -1;
|
|
||||||
}
|
|
||||||
break;
|
|
||||||
|
|
||||||
default:
|
|
||||||
/* No tests for other types */
|
|
||||||
ksft_test_result_skip("get_value.%d.%d\n",
|
|
||||||
ctl->card->card, ctl->elem);
|
|
||||||
return;
|
|
||||||
}
|
|
||||||
|
|
||||||
out:
|
out:
|
||||||
ksft_test_result(err >= 0, "get_value.%d.%d\n",
|
ksft_test_result(err >= 0, "get_value.%d.%d\n",
|
||||||
|
|
|
||||||
Loading…
Add table
Reference in a new issue