On Fri, Jun 14, 2024 at 06:08:12PM +0200, Takashi Iwai wrote: > Mark Brown wrote: > +++ b/tools/testing/selftests/alsa/mixer-test.c > @@ -668,6 +668,10 @@ static int write_and_verify(struct ctl_data *ctl, > ksft_print_msg("%s read and written values differ\n", > ctl->name); > > + /* Allow difference for volatile controls */ > + if (snd_ctl_elem_info_is_volatile(ctl->info)) > + return 0; > + > return -1; > } That'll still print the warnings about the values differing and won't check the values are in range... I'll send a patch.
Attachment:
signature.asc
Description: PGP signature
- References:
- [PATCH v2 0/6] ALSA: some driver fixes for control input validations
- From: Takashi Iwai
- [PATCH v2 4/6] kselftest/alsa: mixer-test: Skip write verification for volatile controls
- From: Takashi Iwai
- Re: [PATCH v2 4/6] kselftest/alsa: mixer-test: Skip write verification for volatile controls
- From: Mark Brown
- Re: [PATCH v2 4/6] kselftest/alsa: mixer-test: Skip write verification for volatile controls
- From: Takashi Iwai
- [PATCH v2 0/6] ALSA: some driver fixes for control input validations
- Prev by Date: [PATCH RFC 3/3] arm64: dts: amlogic: add clock and clock-names to sound cards
- Next by Date: [PATCH v3 0/4] Add audio support for LPC32XX CPUs
- Previous by thread: Re: [PATCH v2 4/6] kselftest/alsa: mixer-test: Skip write verification for volatile controls
- Next by thread: [PATCH v2 3/6] ALSA: control: Apply sanity check of input values for user elements
- Index(es):