On Tue, Feb 1, 2011 at 8:32 PM, Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> wrote: > On Tue, 1 Feb 2011, Maulik Mankad wrote: > >> On Mon, Jan 31, 2011 at 8:27 PM, Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> wrote: >> >> >> >> Since RNDIS is disabled by default the Configuration descriptor also >> >> shows UMS as the first interface. >> > >> > That sounds like a bug. The gadget should know what its own interfaces >> > are! Is the UMS interface number 0 or number 2? >> >> UMS is assigned interface number 2 from usb_interface_id(). >> >> > What value does the bNumInterfaces field have in the config descriptor? >> >> bNumInterfaces field shows 2 interfaces only which is correct(UMS and >> ADB) since RNDIS is disbaled. >> But RNDIS is also assigned interface id even though its disabled. This >> seems to be creating problems. > > This appears to be a bug in composite.c. If a function fails to bind, > the interface numbers it has allocated are not released. > > Does this patch help? > Thanks Alan. I will test this patch and let you know the result tomorrow. Am curious to know if bind() will fail for rndis if the interface is disabled? Regards, Maulik -- 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
