On Fri, Jun 14, 2024 at 02:47:26PM +0200, Takashi Iwai wrote:

> The control elements with volatile flag don't guarantee that the
> written values are actually saved for the next reads, hence they
> aren't suitable for the standard mixer tests.  Skip the write tests
> for those volatile controls for avoiding confusion.

We should still verify that you can actually write I think...

> +     if (!snd_ctl_elem_info_is_volatile(ctl->info)) {
> +             ksft_print_msg("%s is volatile\n", ctl->name);
> +             ksft_test_result_skip("write_invalid.%d.%d\n",
> +                                   ctl->card->card, ctl->elem);
> +             return;
> +     }

...and that you don't read back invalid values after a write like this
for example.  I think any change for this should be in the validation of
the read but we should still try the writes we think we can do.

Attachment: signature.asc
Description: PGP signature

Reply via email to