On Fri, Sep 10, 2010 at 04:08:46PM -0400, Alan Stern wrote: > On Fri, 10 Sep 2010, Greg KH wrote: > > > On Fri, Sep 10, 2010 at 03:14:44PM -0400, Alan Stern wrote: > > > The /sys/kernel/debug/usb/devices file doesn't know about Wireless or > > > SuperSpeed USB. This patch (as1416) teaches it. > > > > > > Signed-off-by: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> > > > CC: David Vrabel <david.vrabel@xxxxxxx> > > > CC: Sarah Sharp <sarah.a.sharp@xxxxxxxxxxxxxxx> > > > > > > --- > > > > > > I wasn't sure what to put in the Wireless case. The string I chose has > > > the disadvantage of not being a number, so it might confuse programs > > > that try to parse it. If anyone has a better suggestion, let me know. > > > > There are tools that parse this value, so we can't make it a string :( > > Do these tools assume particular field widths? If they do, how will > they cope with a 4-digit SuperSpeed entry? I looked at usbview, and it can handle if the field gets bigger. I think that's the only tool that parses this file. Actually, usbview is pretty "dead" these days, and I can easily change it to handle this modification. vmware used to have some logic that worried about this file, but I don't think it relies on it anymore, but I could be wrong. It might be nice to test a windows vmware instance on a Linux host with this change. thanks, greg k-h -- 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
