On Tue, 13 Jul 2021 22:46:43 +0200,
Pavel Machek wrote:
>
> Hi!
>
> > This patch corrects those issues. It introduces a new value type,
> > USB_MIXER_BESPOKEN, which indicates a non-standard mixer element, and
> > use this type for all scarlett2 mixer elements, as well as
> > initializing the fixed unit id 0 for avoiding the overflow.
>
> New mixer value is introduced, but printing code in mixer.c is not
> updated.
>
> Is something like this needed?
Currently BESPOKEN type doesn't use the standard dump callback, hence
this won't hit, but such a change wouldn't hurt.
thanks,
Takashi
>
>
> > +++ b/sound/usb/mixer.h
> > @@ -55,6 +55,7 @@ enum {
> > USB_MIXER_U16,
> > USB_MIXER_S32,
> > USB_MIXER_U32,
> > + USB_MIXER_BESPOKEN, /* non-standard type */
> > };
> >
>
> Best regards,
> Pavel
>
> diff --git a/sound/usb/mixer.c b/sound/usb/mixer.c
> index 2b5281ef8fca..83d5e4d19128 100644
> --- a/sound/usb/mixer.c
> +++ b/sound/usb/mixer.c
> @@ -3294,7 +3294,7 @@ static void snd_usb_mixer_dump_cval(struct snd_info_buffer *buffer,
> {
> struct usb_mixer_elem_info *cval = mixer_elem_list_to_info(list);
> static const char * const val_types[] = {
> - "BOOLEAN", "INV_BOOLEAN", "S8", "U8", "S16", "U16", "S32", "U32",
> + "BOOLEAN", "INV_BOOLEAN", "S8", "U8", "S16", "U16", "S32", "U32", "BESPOKEN",
> };
>
> --
> DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> [2 Digital signature <application/pgp-signature (7bit)>]
>
