5 Oct
2021
5 Oct
'21
10:58 a.m.
On 04. 10. 21 16:32, Pavel Hofman wrote:
+static int openctl_play_rateshift(struct loopback_handle *lhandle,
char *ascii_name) {
I would create 'openctl_elem_ascii' function which will accept the ascii id and the snd_ctl_elem_value_t pointer like 'openctl_elem' does. The the common code may be moved to the 'openctl_elem_id' function from the 'openctl_elem' function.
But it's just nitpicking, the rest of patch looks fine and follows my suggestion. Thank you.
Jaroslav
--
Jaroslav Kysela perex@perex.cz
Linux Sound Maintainer; ALSA Project; Red Hat, Inc.