Message ID | 1536689190-24989-1-git-send-email-saranya.gopal@intel.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | usbcore: Select UAC3 configuration for audio if present | expand |
On Di, 2018-09-11 at 23:36 +0530, saranya.gopal@intel.com wrote: > @@ -121,6 +132,23 @@ int usb_choose_configuration(struct usb_device *udev) > #endif > } > > + /* > + * Select first configuration as default for audio so that > + * devices that don't comply with UAC3 protocol are supported. > + * But, still iterate through other configurations and > + * select UAC3 compliant config if present. > + */ > + if (i == 0 && num_configs > 1 && desc && is_audio(desc)) { > + best = c; > + continue; > + } > + > + if (i > 0 && desc && is_audio(desc)) { > + if (is_uac3_config(desc)) > + best = c; > + break; > + } Hi, that looks like a special case. Couldn't we do for (all configuration) { 1. take first 2. if this configuration is generic and the current is specific change current 3. if if this configuration is the same interface class as current and protocol is higher, change current } Regards Oliver
Hi Oliver, > -----Original Message----- > From: linux-usb-owner@vger.kernel.org [mailto:linux-usb- > owner@vger.kernel.org] On Behalf Of Oliver Neukum > Sent: Thursday, September 13, 2018 4:37 PM > To: Gopal, Saranya <saranya.gopal@intel.com>; linux-usb@vger.kernel.org > Cc: K V, Abhilash <abhilash.k.v@intel.com>; Balaji, M <m.balaji@intel.com>; > Regupathy, Rajaram <rajaram.regupathy@intel.com>; > felipe.balbi@linux.intel.com > Subject: Re: [PATCH] usbcore: Select UAC3 configuration for audio if present > > On Di, 2018-09-11 at 23:36 +0530, saranya.gopal@intel.com wrote: > > @@ -121,6 +132,23 @@ int usb_choose_configuration(struct usb_device > *udev) > > #endif > > } > > > > + /* > > + * Select first configuration as default for audio so that > > + * devices that don't comply with UAC3 protocol are supported. > > + * But, still iterate through other configurations and > > + * select UAC3 compliant config if present. > > + */ > > + if (i == 0 && num_configs > 1 && desc && is_audio(desc)) { > > + best = c; > > + continue; > > + } > > + > > + if (i > 0 && desc && is_audio(desc)) { > > + if (is_uac3_config(desc)) > > + best = c; > > + break; > > + } > > Hi, > > that looks like a special case. Couldn't we do > > for (all configuration) { > > 1. take first > 2. if this configuration is generic and the current is specific change > current > 3. if if this configuration is the same interface class as current and > protocol is higher, change current > I have followed the same style used for Ethernet case in this function. Do you foresee benefits with your approach over the existing one? Thanks, Saranya > } > Regards > Oliver
diff --git a/drivers/usb/core/generic.c b/drivers/usb/core/generic.c index bc8242b..dc6eb24 100644 --- a/drivers/usb/core/generic.c +++ b/drivers/usb/core/generic.c @@ -21,6 +21,7 @@ #include <linux/usb.h> #include <linux/usb/hcd.h> +#include <uapi/linux/usb/audio.h> #include "usb.h" static inline const char *plural(int n) @@ -42,6 +43,16 @@ static int is_activesync(struct usb_interface_descriptor *desc) && desc->bInterfaceProtocol == 1; } +static int is_audio(struct usb_interface_descriptor *desc) +{ + return desc->bInterfaceClass == USB_CLASS_AUDIO; +} + +static int is_uac3_config(struct usb_interface_descriptor *desc) +{ + return desc->bInterfaceProtocol == UAC_VERSION_3; +} + int usb_choose_configuration(struct usb_device *udev) { int i; @@ -121,6 +132,23 @@ int usb_choose_configuration(struct usb_device *udev) #endif } + /* + * Select first configuration as default for audio so that + * devices that don't comply with UAC3 protocol are supported. + * But, still iterate through other configurations and + * select UAC3 compliant config if present. + */ + if (i == 0 && num_configs > 1 && desc && is_audio(desc)) { + best = c; + continue; + } + + if (i > 0 && desc && is_audio(desc)) { + if (is_uac3_config(desc)) + best = c; + break; + } + /* From the remaining configs, choose the first one whose * first interface is for a non-vendor-specific class. * Reason: Linux is more likely to have a class driver