Re: [PATCH v2 0/8] lsusb: Add initial support for USB Audio Class 3

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





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



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux