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.
Cheers,
--
Michael Drake http://www.codethink.co.uk/
--
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
