On Sat, Sep 8, 2012 at 3:34 AM, Linus Walleij <linus.walleij@xxxxxxxxxx> wrote: > > On Fri, Sep 7, 2012 at 8:51 PM, Sebastian Andrzej Siewior > <sebastian@xxxxxxxxxxxxx> wrote: > > On Wed, Jan 25, 2012 at 11:02:03AM +0100, Linus Walleij wrote: > > >> static struct usb_descriptor_header *ecm_fs_function[] = { > >> /* CDC ECM control descriptors */ > >> + (struct usb_descriptor_header *) &ecm_iad_descriptor, > >> (struct usb_descriptor_header *) &ecm_control_intf, > >> (struct usb_descriptor_header *) &ecm_header_desc, > >> (struct usb_descriptor_header *) &ecm_union_desc, > >> @@ -247,6 +262,7 @@ static struct usb_endpoint_descriptor > >> hs_ecm_out_desc = { > >> > >> static struct usb_descriptor_header *ecm_hs_function[] = { > >> /* CDC ECM control descriptors */ > >> + (struct usb_descriptor_header *) &ecm_iad_descriptor, > >> (struct usb_descriptor_header *) &ecm_control_intf, > >> (struct usb_descriptor_header *) &ecm_header_desc, > >> (struct usb_descriptor_header *) &ecm_union_desc, > > > > Why did you add this IAD thingy to FS and HS and not SS descriptors? I > > don't > > see a reason so I would guess you just forgot :) > > Argh, Praveena, save me here ... I was being patch secretary > this time, but to my untrained USB-eyes it looks like you're right, > interested in patching it? This patch was generated based on Kernel 3.0 and then Super Speed was not fully into gadget framework! > > Yours, > Linus Walleij > -- > 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 -- 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