Re: [PATCH 1/5] ALSA: usbaudio: parse USB descriptors with structs

[Clemens Ladisch <clemens@ladisch.de>]

Daniel Mack wrote:
> In preparation of support for v2.0 audio class, use the structs from
> linux/usb/audio.h and add some new ones to describe the fields that are
> actually parsed by the descriptor decoders.
> ...
> --- a/include/linux/usb/audio.h
> +++ b/include/linux/usb/audio.h
> @@ -81,7 +81,7 @@
>  
>  /* Terminal Control Selectors */
>  /* 4.3.2  Class-Specific AC Interface Descriptor */
> -struct uac_ac_header_descriptor {
> +struct uac_ac_header_descriptor_v1 {

This header is already used by drivers in drivers/usb/gadget/.

If there isn't too much overlap between v1 and v2 descriptors, it might
make sense to put them in separate headers (audio2.h) and to put the
parsers for v1 and v2 into separate files, too.


Regards,
Clemens