On Thu, Dec 07, 2017 at 05:14:10PM +0000, Michael Drake wrote: > > > On 07/12/17 15:16, Greg KH wrote: > > On Thu, Dec 07, 2017 at 04:01:23PM +0100, Greg KH wrote: > > > On Thu, Dec 07, 2017 at 04:00:36PM +0100, Greg KH wrote: > > > > On Thu, Dec 07, 2017 at 10:26:21AM +0000, Michael Drake wrote: > > > > > This adds a new way of dumping descriptors, which splits the knowledge > > > > > of how to interpret descriptor data from the actual dumping. This has > > > > > two advantages: > > > > > > > > > > 1. It is easy to add support for new descriptors, since they are now > > > > > simple definitions that resemble the tables in the USB specifications. > > > > > > > > > > 2. The code for dumping descriptors is common, so the output is easy to > > > > > keep consistent. It is also consistent and thorough in its handling > > > > > of insufficient descriptor data buffer, and junk data at the end of > > > > > a descriptor. > > > > > > > > > > UAC1 and UAC2 are converted to use the new mechanism, initial support > > > > > for UAC3 is added. Finally, support for the USB3 BOS Configuration > > > > > Summary Descriptor is added. > > > > > > > > > > This was previously opened as a github pull request here: > > > > > > > > > > https://github.com/gregkh/usbutils/pull/61 > > > > > > > > Thanks for this, all of the patches are now applied. > > > > > > > > There were some intermediate build warnings, but future patches in the > > > > series fixed that up, next time be more careful, each patch should be > > > > "clean". > > > > > > > > However the build now gets the following warnings: > > > > > > > > CC lsusb-lsusb.o > > > > lsusb.c:220:12: warning: ‘get_audioterminal_string’ defined but not used [-Wunused-function] > > > > static int get_audioterminal_string(char *buf, size_t size, u_int16_t termt) > > > > ^~~~~~~~~~~~~~~~~~~~~~~~ > > > > CC lsusb-lsusb-t.o > > > > CC lsusb-desc-defs.o > > > > CC lsusb-desc-dump.o > > > > desc-dump.c: In function ‘desc_bmcontrol_dump’: > > > > desc-dump.c:67:18: warning: comparison between pointer and zero character constant [-Wpointer-compare] > > > > if (strings[0] != '\0') { > > > > ^~ > > > > desc-dump.c:67:7: note: did you mean to dereference the pointer? > > > > if (strings[0] != '\0') { > > > > ^ > > > > CC lsusb-names.o > > > > > > > > > > > > Can you fix this up and send a patch for them? > > > > > > Oops, I should have tested the code, it now crashes for me with the > > > following error: > > > Floating point exception (core dumped) > > > > > > Do you see this as well? > > No, I don't see that. > > > And it's crashing on my USB audio device. Here's the output of it from > > the "old" lsusb output. > > [snip] > > Does it still crash with the warning fixes I posted? If so I'll > look in detail tomorrow. I will check when I get home tonight, I don't have the USB device on me at the moment. 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
