On Wed, Jun 3, 2009 at 05:35, Mike Frysinger wrote: > On Wed, Jun 3, 2009 at 04:52, Sergei Shtylyov wrote: >> Mike Frysinger wrote: >>> +#ifdef CONFIG_BLACKFIN >>> + } else if (gadget_is_musbhsfc(gadget) || gadget_is_musbhdrc(gadget)) { >>> + if ((USB_ENDPOINT_XFER_BULK == type) >>> + || (USB_ENDPOINT_XFER_ISOC == type)) { >>> + if (USB_DIR_IN & desc->bEndpointAddress) >>> + ep = find_ep (gadget, "ep5in"); >>> + else >>> + ep = find_ep (gadget, "ep6out"); >>> + } >>> + if (ep && ep_matches (gadget, ep, desc)) >>> + return ep; >>> +#endif >> >> Have you run the path thru checkpatch.pl? It would give errors about >> space between function name and left paren... > > the original commit did, but not the new one. i'll fix that up, thanks. actually, this was done on purpose. this change preserves the style used in that driver/function. -mike -- To unsubscribe from this list: send the line "unsubscribe linux-usb" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html