On Mon, 24 Jan 2022 16:14:10 +0100, Mark Brown wrote: > +bool test_ctl_write_invalid_integer64(struct ctl_data *ctl) > +{ .... > + /* Minimum representable value */ > + snd_ctl_elem_value_copy(val, ctl->def_val); > + snd_ctl_elem_value_set_integer(val, i, LLONG_MIN); This should be snd_ctl_elem_value_set_integer64()? > + /* Maximum representable value */ > + snd_ctl_elem_value_copy(val, ctl->def_val); > + snd_ctl_elem_value_set_integer(val, i, LLONG_MAX); Ditto. thanks, Takashi