On Tue, Dec 05, 2017 at 04:14:26PM +0000, Michael Drake wrote: > This adds a new way of dumping descriptors. It takes the descriptor > data to be dumped, and a descriptor definition as input. > > The descriptor definition takes the form of a NULL terminated array > of descriptor field definitions. > > These definitions describe how the raw descriptor data buffer should > be interpreted. > > Thus the knowledge of how to interpret a descriptor buffer is separate > from the shared code that renders the descriptor dump. This has two > advantages: > > 1. 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. > > 2. It is easy to add support for new descriptors, since they are now > simple definitions that resemble the tables in the USB specifications. > --- > desc-dump.c | 550 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ > desc-dump.h | 64 +++++++ > 2 files changed, 614 insertions(+) > create mode 100644 desc-dump.c > create mode 100644 desc-dump.h Same meta-comments as on patch 2 apply here :) 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
