aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChris Dickens <christopher.a.dickens@gmail.com>2020-08-28 15:42:41 -0700
committerChris Dickens <christopher.a.dickens@gmail.com>2020-09-13 00:09:32 -0700
commita6bfd45deab1e9550a7e7f8bf618b484da7e2816 (patch)
tree512a9d287ec4d6f656280bdc163e6bb16d7f2ac5
parentccba488d11e373078aba7b086e3824417b31f67e (diff)
downloadplatform_external_libusb-upstream-master.tar.gz
platform_external_libusb-upstream-master.tar.bz2
platform_external_libusb-upstream-master.zip
Doxygen: Fix warnings about undocumented compoundsupstream-master
Commit fa3f91e7c4 ("descriptor: Miscellaneous improvements to the overall source") introduced some union types, however since descriptor.c is parsed by Doxygen it will complain with the following warnings: libusb/descriptor.c:49: warning: Compound string_desc_buf is not documented. libusb/descriptor.c:55: warning: Compound bos_desc_buf is not documented. libusb/descriptor.c:43: warning: Compound config_desc_buf is not documented. Fix this by moving the union definitions into libusbi.h. Signed-off-by: Chris Dickens <christopher.a.dickens@gmail.com>
-rw-r--r--libusb/descriptor.c28
-rw-r--r--libusb/libusbi.h20
-rw-r--r--libusb/version_nano.h2
3 files changed, 25 insertions, 25 deletions
diff --git a/libusb/descriptor.c b/libusb/descriptor.c
index cda3afd..ecd9441 100644
--- a/libusb/descriptor.c
+++ b/libusb/descriptor.c
@@ -40,24 +40,6 @@
((uint32_t)((p)[1]) << 8) | \
((uint32_t)((p)[0]))))
-union config_desc_buf {
- struct usbi_configuration_descriptor desc;
- uint8_t buf[LIBUSB_DT_CONFIG_SIZE];
- uint16_t dummy; /* Force 2-byte alignment */
-};
-
-union string_desc_buf {
- struct usbi_string_descriptor desc;
- uint8_t buf[255]; /* Some devices choke on size > 255 */
- uint16_t dummy; /* Force 2-byte alignment */
-};
-
-union bos_desc_buf {
- struct usbi_bos_descriptor desc;
- uint8_t buf[LIBUSB_DT_BOS_SIZE];
- uint16_t dummy; /* Force 2-byte alignment */
-};
-
static void parse_descriptor(const void *source, const char *descriptor, void *dest)
{
const uint8_t *sp = source;
@@ -573,7 +555,7 @@ int API_EXPORTED libusb_get_device_descriptor(libusb_device *dev,
int API_EXPORTED libusb_get_active_config_descriptor(libusb_device *dev,
struct libusb_config_descriptor **config)
{
- union config_desc_buf _config;
+ union usbi_config_desc_buf _config;
uint16_t config_len;
uint8_t *buf;
int r;
@@ -614,7 +596,7 @@ int API_EXPORTED libusb_get_active_config_descriptor(libusb_device *dev,
int API_EXPORTED libusb_get_config_descriptor(libusb_device *dev,
uint8_t config_index, struct libusb_config_descriptor **config)
{
- union config_desc_buf _config;
+ union usbi_config_desc_buf _config;
uint16_t config_len;
uint8_t *buf;
int r;
@@ -676,7 +658,7 @@ int API_EXPORTED libusb_get_config_descriptor_by_value(libusb_device *dev,
usbi_dbg("value %u", bConfigurationValue);
for (idx = 0; idx < dev->device_descriptor.bNumConfigurations; idx++) {
- union config_desc_buf _config;
+ union usbi_config_desc_buf _config;
r = get_config_descriptor(dev, idx, _config.buf, sizeof(_config.buf));
if (r < 0)
@@ -864,7 +846,7 @@ static int parse_bos(struct libusb_context *ctx,
int API_EXPORTED libusb_get_bos_descriptor(libusb_device_handle *dev_handle,
struct libusb_bos_descriptor **bos)
{
- union bos_desc_buf _bos;
+ union usbi_bos_desc_buf _bos;
uint16_t bos_len;
uint8_t *bos_data;
int r;
@@ -1103,7 +1085,7 @@ void API_EXPORTED libusb_free_container_id_descriptor(
int API_EXPORTED libusb_get_string_descriptor_ascii(libusb_device_handle *dev_handle,
uint8_t desc_index, unsigned char *data, int length)
{
- union string_desc_buf str;
+ union usbi_string_desc_buf str;
int r, si, di;
uint16_t langid, wdata;
diff --git a/libusb/libusbi.h b/libusb/libusbi.h
index 1208274..c4e705f 100644
--- a/libusb/libusbi.h
+++ b/libusb/libusbi.h
@@ -630,7 +630,7 @@ struct usbi_interface_descriptor {
struct usbi_string_descriptor {
uint8_t bLength;
uint8_t bDescriptorType;
- uint16_t wData[];
+ uint16_t wData[ZERO_SIZED_ARRAY];
} LIBUSB_PACKED;
struct usbi_bos_descriptor {
@@ -644,6 +644,24 @@ struct usbi_bos_descriptor {
#pragma pack(pop)
#endif
+union usbi_config_desc_buf {
+ struct usbi_configuration_descriptor desc;
+ uint8_t buf[LIBUSB_DT_CONFIG_SIZE];
+ uint16_t align; /* Force 2-byte alignment */
+};
+
+union usbi_string_desc_buf {
+ struct usbi_string_descriptor desc;
+ uint8_t buf[255]; /* Some devices choke on size > 255 */
+ uint16_t align; /* Force 2-byte alignment */
+};
+
+union usbi_bos_desc_buf {
+ struct usbi_bos_descriptor desc;
+ uint8_t buf[LIBUSB_DT_BOS_SIZE];
+ uint16_t align; /* Force 2-byte alignment */
+};
+
/* shared data and functions */
int usbi_io_init(struct libusb_context *ctx);
diff --git a/libusb/version_nano.h b/libusb/version_nano.h
index c859d43..4b21d4e 100644
--- a/libusb/version_nano.h
+++ b/libusb/version_nano.h
@@ -1 +1 @@
-#define LIBUSB_NANO 11559
+#define LIBUSB_NANO 11560