From patchwork Sat Apr 24 16:26:22 2010 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "David S. Ahern" X-Patchwork-Id: 50913 Return-Path: X-Original-To: incoming@patchwork.ozlabs.org Delivered-To: patchwork-incoming@bilbo.ozlabs.org Received: from lists.gnu.org (lists.gnu.org [199.232.76.165]) (using TLSv1 with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by ozlabs.org (Postfix) with ESMTPS id 87415B7D1E for ; Sun, 25 Apr 2010 02:34:17 +1000 (EST) Received: from localhost ([127.0.0.1]:35280 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1O5iD6-00040Z-05 for incoming@patchwork.ozlabs.org; Sat, 24 Apr 2010 12:28:04 -0400 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1O5iBp-00040P-Jv for qemu-devel@nongnu.org; Sat, 24 Apr 2010 12:26:45 -0400 Received: from [140.186.70.92] (port=53527 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1O5iBn-00040A-8p for qemu-devel@nongnu.org; Sat, 24 Apr 2010 12:26:45 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1O5iBi-0003xi-8I for qemu-devel@nongnu.org; Sat, 24 Apr 2010 12:26:43 -0400 Received: from sj-iport-1.cisco.com ([171.71.176.70]:18104) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1O5iBh-0003wu-Dw for qemu-devel@nongnu.org; Sat, 24 Apr 2010 12:26:38 -0400 Authentication-Results: sj-iport-1.cisco.com; dkim=neutral (message not signed) header.i=none X-IronPort-AV: E=Sophos;i="4.52,267,1270425600"; d="scan'208";a="320136547" Received: from sj-core-2.cisco.com ([171.71.177.254]) by sj-iport-1.cisco.com with ESMTP; 24 Apr 2010 16:26:28 +0000 Received: from localhost.localdomain ([10.89.14.73]) by sj-core-2.cisco.com (8.13.8/8.14.3) with ESMTP id o3OGQRG4026212; Sat, 24 Apr 2010 16:26:27 GMT From: David Ahern To: qemu-devel@nongnu.org Date: Sat, 24 Apr 2010 10:26:22 -0600 Message-Id: <1272126382-20643-1-git-send-email-daahern@cisco.com> X-Mailer: git-send-email 1.6.6.1 X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. Cc: jan.kiszka@web.de, David Ahern Subject: [Qemu-devel] [PATCH] Changes to usb-linux to conform to coding style X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Errors-To: qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org Signed-off-by: David Ahern --- usb-linux.c | 374 +++++++++++++++++++++++++++++++++++------------------------ 1 files changed, 224 insertions(+), 150 deletions(-) diff --git a/usb-linux.c b/usb-linux.c index d0d7cff..b3d6b28 100644 --- a/usb-linux.c +++ b/usb-linux.c @@ -105,9 +105,9 @@ enum { /* * Control transfer state. - * Note that 'buffer' _must_ follow 'req' field because + * Note that 'buffer' _must_ follow 'req' field because * we need contigious buffer when we submit control URB. - */ + */ struct ctrl_struct { uint16_t len; uint16_t offset; @@ -170,10 +170,10 @@ static void set_halt(USBHostDevice *s, int ep) s->endp_table[ep - 1].halted = 1; } -/* +/* * Async URB state. * We always allocate one isoc descriptor even for bulk transfers - * to simplify allocation and casts. + * to simplify allocation and casts. */ typedef struct AsyncURB { @@ -220,15 +220,16 @@ static void async_complete(void *opaque) AsyncURB *aurb; while (1) { - USBPacket *p; + USBPacket *p; - int r = ioctl(s->fd, USBDEVFS_REAPURBNDELAY, &aurb); + int r = ioctl(s->fd, USBDEVFS_REAPURBNDELAY, &aurb); if (r < 0) { - if (errno == EAGAIN) + if (errno == EAGAIN) { return; - + } if (errno == ENODEV && !s->closing) { - printf("husb: device %d.%d disconnected\n", s->bus_num, s->addr); + printf("husb: device %d.%d disconnected\n", + s->bus_num, s->addr); usb_host_close(s); usb_host_auto_check(NULL); return; @@ -240,21 +241,22 @@ static void async_complete(void *opaque) p = aurb->packet; - DPRINTF("husb: async completed. aurb %p status %d alen %d\n", + DPRINTF("husb: async completed. aurb %p status %d alen %d\n", aurb, aurb->urb.status, aurb->urb.actual_length); - if (p) { + if (p) { switch (aurb->urb.status) { case 0: p->len = aurb->urb.actual_length; - if (aurb->urb.type == USBDEVFS_URB_TYPE_CONTROL) + if (aurb->urb.type == USBDEVFS_URB_TYPE_CONTROL) { async_complete_ctrl(s, p); + } break; case -EPIPE: set_halt(s, p->devep); - p->len = USB_RET_STALL; - break; + p->len = USB_RET_STALL; + break; default: p->len = USB_RET_NAK; @@ -262,7 +264,7 @@ static void async_complete(void *opaque) } usb_packet_complete(p); - } + } async_free(aurb); } @@ -297,14 +299,15 @@ static int usb_host_claim_interfaces(USBHostDevice *dev, int configuration) i = 0; dev_descr_len = dev->descr[0]; - if (dev_descr_len > dev->descr_len) + if (dev_descr_len > dev->descr_len) { goto fail; + } nb_configurations = dev->descr[17]; i += dev_descr_len; while (i < dev->descr_len) { - DPRINTF("husb: i is %d, descr_len is %d, dl %d, dt %d\n", i, dev->descr_len, - dev->descr[i], dev->descr[i+1]); + DPRINTF("husb: i is %d, descr_len is %d, dl %d, dt %d\n", + i, dev->descr_len, dev->descr[i], dev->descr[i+1]); if (dev->descr[i+1] != USB_DT_CONFIG) { i += dev->descr[i]; @@ -312,7 +315,7 @@ static int usb_host_claim_interfaces(USBHostDevice *dev, int configuration) } config_descr_len = dev->descr[i]; - printf("husb: config #%d need %d\n", dev->descr[i + 5], configuration); + printf("husb: config #%d need %d\n", dev->descr[i + 5], configuration); if (configuration < 0 || configuration == dev->descr[i + 5]) { configuration = dev->descr[i + 5]; @@ -323,7 +326,8 @@ static int usb_host_claim_interfaces(USBHostDevice *dev, int configuration) } if (i >= dev->descr_len) { - fprintf(stderr, "husb: update iface failed. no matching configuration\n"); + fprintf(stderr, + "husb: update iface failed. no matching configuration\n"); goto fail; } nb_interfaces = dev->descr[i + 4]; @@ -416,15 +420,16 @@ static int usb_host_handle_data(USBHostDevice *s, USBPacket *p) urb = &aurb->urb; - if (p->pid == USB_TOKEN_IN) - urb->endpoint = p->devep | 0x80; - else - urb->endpoint = p->devep; + if (p->pid == USB_TOKEN_IN) { + urb->endpoint = p->devep | 0x80; + } else { + urb->endpoint = p->devep; + } if (is_halted(s, p->devep)) { - ret = ioctl(s->fd, USBDEVFS_CLEAR_HALT, &urb->endpoint); + ret = ioctl(s->fd, USBDEVFS_CLEAR_HALT, &urb->endpoint); if (ret < 0) { - DPRINTF("husb: failed to clear halt. ep 0x%x errno %d\n", + DPRINTF("husb: failed to clear halt. ep 0x%x errno %d\n", urb->endpoint, errno); return USB_RET_NAK; } @@ -449,7 +454,8 @@ static int usb_host_handle_data(USBHostDevice *s, USBPacket *p) ret = ioctl(s->fd, USBDEVFS_SUBMITURB, urb); - DPRINTF("husb: data submit. ep 0x%x len %u aurb %p\n", urb->endpoint, p->len, aurb); + DPRINTF("husb: data submit. ep 0x%x len %u aurb %p\n", + urb->endpoint, p->len, aurb); if (ret < 0) { DPRINTF("husb: submit failed. errno %d\n", errno); @@ -470,10 +476,11 @@ static int usb_host_handle_data(USBHostDevice *s, USBPacket *p) static int ctrl_error(void) { - if (errno == ETIMEDOUT) + if (errno == ETIMEDOUT) { return USB_RET_NAK; - else + } else { return USB_RET_STALL; + } } static int usb_host_set_address(USBHostDevice *s, int addr) @@ -488,12 +495,12 @@ static int usb_host_set_config(USBHostDevice *s, int config) usb_host_release_interfaces(s); int ret = ioctl(s->fd, USBDEVFS_SETCONFIGURATION, &config); - + DPRINTF("husb: ctrl set config %d ret %d errno %d\n", config, ret, errno); - - if (ret < 0) + + if (ret < 0) { return ctrl_error(); - + } usb_host_claim_interfaces(s, config); return 0; } @@ -506,13 +513,13 @@ static int usb_host_set_interface(USBHostDevice *s, int iface, int alt) si.interface = iface; si.altsetting = alt; ret = ioctl(s->fd, USBDEVFS_SETINTERFACE, &si); - - DPRINTF("husb: ctrl set iface %d altset %d ret %d errno %d\n", - iface, alt, ret, errno); - - if (ret < 0) - return ctrl_error(); + DPRINTF("husb: ctrl set iface %d altset %d ret %d errno %d\n", + iface, alt, ret, errno); + + if (ret < 0) { + return ctrl_error(); + } usb_linux_update_endp_table(s); return 0; } @@ -524,7 +531,7 @@ static int usb_host_handle_control(USBHostDevice *s, USBPacket *p) int ret, value, index; int buffer_len; - /* + /* * Process certain standard device requests. * These are infrequent and are processed synchronously. */ @@ -532,8 +539,8 @@ static int usb_host_handle_control(USBHostDevice *s, USBPacket *p) index = le16_to_cpu(s->ctrl.req.wIndex); DPRINTF("husb: ctrl type 0x%x req 0x%x val 0x%x index %u len %u\n", - s->ctrl.req.bRequestType, s->ctrl.req.bRequest, value, index, - s->ctrl.len); + s->ctrl.req.bRequestType, s->ctrl.req.bRequest, value, index, + s->ctrl.len); if (s->ctrl.req.bRequestType == 0) { switch (s->ctrl.req.bRequest) { @@ -546,8 +553,9 @@ static int usb_host_handle_control(USBHostDevice *s, USBPacket *p) } if (s->ctrl.req.bRequestType == 1 && - s->ctrl.req.bRequest == USB_REQ_SET_INTERFACE) + s->ctrl.req.bRequest == USB_REQ_SET_INTERFACE) { return usb_host_set_interface(s, index, value); + } /* The rest are asynchronous */ @@ -562,12 +570,12 @@ static int usb_host_handle_control(USBHostDevice *s, USBPacket *p) aurb->hdev = s; aurb->packet = p; - /* + /* * Setup ctrl transfer. * * s->ctrl is layed out such that data buffer immediately follows * 'req' struct which is exactly what usbdevfs expects. - */ + */ urb = &aurb->urb; urb->type = USBDEVFS_URB_TYPE_CONTROL; @@ -604,9 +612,10 @@ static int do_token_setup(USBDevice *dev, USBPacket *p) USBHostDevice *s = (USBHostDevice *) dev; int ret = 0; - if (p->len != 8) + if (p->len != 8) { return USB_RET_STALL; - + } + memcpy(&s->ctrl.req, p->data, 8); s->ctrl.len = le16_to_cpu(s->ctrl.req.wLength); s->ctrl.offset = 0; @@ -614,17 +623,20 @@ static int do_token_setup(USBDevice *dev, USBPacket *p) if (s->ctrl.req.bRequestType & USB_DIR_IN) { ret = usb_host_handle_control(s, p); - if (ret < 0) + if (ret < 0) { return ret; + } - if (ret < s->ctrl.len) + if (ret < s->ctrl.len) { s->ctrl.len = ret; + } s->ctrl.state = CTRL_STATE_DATA; } else { - if (s->ctrl.len == 0) + if (s->ctrl.len == 0) { s->ctrl.state = CTRL_STATE_ACK; - else + } else { s->ctrl.state = CTRL_STATE_DATA; + } } return ret; @@ -635,16 +647,17 @@ static int do_token_in(USBDevice *dev, USBPacket *p) USBHostDevice *s = (USBHostDevice *) dev; int ret = 0; - if (p->devep != 0) + if (p->devep != 0) { return usb_host_handle_data(s, p); + } switch(s->ctrl.state) { case CTRL_STATE_ACK: if (!(s->ctrl.req.bRequestType & USB_DIR_IN)) { ret = usb_host_handle_control(s, p); - if (ret == USB_RET_ASYNC) + if (ret == USB_RET_ASYNC) { return USB_RET_ASYNC; - + } s->ctrl.state = CTRL_STATE_IDLE; return ret > 0 ? 0 : ret; } @@ -654,12 +667,14 @@ static int do_token_in(USBDevice *dev, USBPacket *p) case CTRL_STATE_DATA: if (s->ctrl.req.bRequestType & USB_DIR_IN) { int len = s->ctrl.len - s->ctrl.offset; - if (len > p->len) + if (len > p->len) { len = p->len; + } memcpy(p->data, s->ctrl.buffer + s->ctrl.offset, len); s->ctrl.offset += len; - if (s->ctrl.offset >= s->ctrl.len) + if (s->ctrl.offset >= s->ctrl.len) { s->ctrl.state = CTRL_STATE_ACK; + } return len; } @@ -675,8 +690,9 @@ static int do_token_out(USBDevice *dev, USBPacket *p) { USBHostDevice *s = (USBHostDevice *) dev; - if (p->devep != 0) + if (p->devep != 0) { return usb_host_handle_data(s, p); + } switch(s->ctrl.state) { case CTRL_STATE_ACK: @@ -691,12 +707,14 @@ static int do_token_out(USBDevice *dev, USBPacket *p) case CTRL_STATE_DATA: if (!(s->ctrl.req.bRequestType & USB_DIR_IN)) { int len = s->ctrl.len - s->ctrl.offset; - if (len > p->len) + if (len > p->len) { len = p->len; + } memcpy(s->ctrl.buffer + s->ctrl.offset, p->data, len); s->ctrl.offset += len; - if (s->ctrl.offset >= s->ctrl.len) + if (s->ctrl.offset >= s->ctrl.len) { s->ctrl.state = CTRL_STATE_ACK; + } return len; } @@ -734,8 +752,9 @@ static int usb_host_handle_packet(USBDevice *s, USBPacket *p) } /* Rest of the PIDs must match our address */ - if (s->state < USB_STATE_DEFAULT || p->devaddr != s->addr) + if (s->state < USB_STATE_DEFAULT || p->devaddr != s->addr) { return USB_RET_NODEV; + } switch (p->pid) { case USB_TOKEN_SETUP: @@ -746,7 +765,7 @@ static int usb_host_handle_packet(USBDevice *s, USBPacket *p) case USB_TOKEN_OUT: return do_token_out(s, p); - + default: return USB_RET_STALL; } @@ -775,8 +794,9 @@ static int usb_linux_update_endp_table(USBHostDevice *s) } /* in address state */ - if (configuration == 0) + if (configuration == 0) { return 1; + } /* get the desired configuration, interface, and endpoint descriptors * from device description */ @@ -822,15 +842,17 @@ static int usb_linux_update_endp_table(USBHostDevice *s) } /* advance to the endpoints */ - while (i < length && descriptors[i +1] != USB_DT_ENDPOINT) + while (i < length && descriptors[i +1] != USB_DT_ENDPOINT) { i += descriptors[i]; + } if (i >= length) break; while (i < length) { - if (descriptors[i + 1] != USB_DT_ENDPOINT) + if (descriptors[i + 1] != USB_DT_ENDPOINT) { break; + } devep = descriptors[i + 2]; switch (descriptors[i + 3] & 0x3) { @@ -866,9 +888,9 @@ static int usb_host_open(USBHostDevice *dev, int bus_num, struct usbdevfs_connectinfo ci; char buf[1024]; - if (dev->fd != -1) + if (dev->fd != -1) { goto fail; - + } printf("husb: open device %d.%d\n", bus_num, addr); if (!usb_host_device_path) { @@ -899,21 +921,23 @@ static int usb_host_open(USBHostDevice *dev, int bus_num, { int x; printf("=== begin dumping device descriptor data ===\n"); - for (x = 0; x < dev->descr_len; x++) + for (x = 0; x < dev->descr_len; x++) { printf("%02x ", dev->descr[x]); + } printf("\n=== end dumping device descriptor data ===\n"); } #endif - /* - * Initial configuration is -1 which makes us claim first + /* + * Initial configuration is -1 which makes us claim first * available config. We used to start with 1, which does not - * always work. I've seen devices where first config starts + * always work. I've seen devices where first config starts * with 2. */ - if (!usb_host_claim_interfaces(dev, -1)) + if (!usb_host_claim_interfaces(dev, -1)) { goto fail; + } ret = ioctl(fd, USBDEVFS_CONNECTINFO, &ci); if (ret < 0) { @@ -924,20 +948,23 @@ static int usb_host_open(USBHostDevice *dev, int bus_num, printf("husb: grabbed usb device %d.%d\n", bus_num, addr); ret = usb_linux_update_endp_table(dev); - if (ret) + if (ret) { goto fail; + } - if (ci.slow) + if (ci.slow) { dev->dev.speed = USB_SPEED_LOW; - else + } else { dev->dev.speed = USB_SPEED_HIGH; + } - if (!prod_name || prod_name[0] == '\0') + if (!prod_name || prod_name[0] == '\0') { snprintf(dev->dev.product_desc, sizeof(dev->dev.product_desc), "host:%d.%d", bus_num, addr); - else + } else { pstrcpy(dev->dev.product_desc, sizeof(dev->dev.product_desc), prod_name); + } /* USB devio uses 'write' flag to check for async completions */ qemu_set_fd_handler(dev->fd, NULL, async_complete, dev); @@ -947,15 +974,17 @@ static int usb_host_open(USBHostDevice *dev, int bus_num, fail: dev->fd = -1; - if (fd != -1) + if (fd != -1) { close(fd); + } return -1; } static int usb_host_close(USBHostDevice *dev) { - if (dev->fd == -1) + if (dev->fd == -1) { return -1; + } qemu_set_fd_handler(dev->fd, NULL, NULL, NULL); dev->closing = 1; @@ -1012,8 +1041,9 @@ USBDevice *usb_host_device_open(const char *devname) dev = usb_create(NULL /* FIXME */, "usb-host"); if (strstr(devname, "auto:")) { - if (parse_filter(devname, &filter) < 0) + if (parse_filter(devname, &filter) < 0) { goto fail; + } } else { if ((p = strchr(devname, '.'))) { filter.bus_num = strtoul(devname, NULL, 0); @@ -1049,13 +1079,13 @@ int usb_host_device_close(const char *devname) int bus_num, addr; USBHostDevice *s; - if (strstr(devname, "auto:")) + if (strstr(devname, "auto:")) { return usb_host_auto_del(devname); - - if (usb_host_find_device(&bus_num, &addr, product_name, sizeof(product_name), - devname) < 0) + } + if (usb_host_find_device(&bus_num, &addr, product_name, + sizeof(product_name), devname) < 0) { return -1; - + } s = hostdev_find(bus_num, addr); if (s) { usb_device_delete_addr(s->bus_num, s->dev.addr); @@ -1073,15 +1103,18 @@ static int get_tag_value(char *buf, int buf_size, const char *p; char *q; p = strstr(str, tag); - if (!p) + if (!p) { return -1; + } p += strlen(tag); - while (qemu_isspace(*p)) + while (qemu_isspace(*p)) { p++; + } q = buf; while (*p != '\0' && !strchr(stopchars, *p)) { - if ((q - buf) < (buf_size - 1)) + if ((q - buf) < (buf_size - 1)) { *q++ = *p; + } p++; } *q = '\0'; @@ -1116,51 +1149,62 @@ static int usb_host_scan_dev(void *opaque, USBScanFunc *func) device_count = 0; bus_num = addr = speed = class_id = product_id = vendor_id = 0; for(;;) { - if (fgets(line, sizeof(line), f) == NULL) + if (fgets(line, sizeof(line), f) == NULL) { break; - if (strlen(line) > 0) + } + if (strlen(line) > 0) { line[strlen(line) - 1] = '\0'; + } if (line[0] == 'T' && line[1] == ':') { if (device_count && (vendor_id || product_id)) { /* New device. Add the previously discovered device. */ ret = func(opaque, bus_num, addr, class_id, vendor_id, product_id, product_name, speed); - if (ret) + if (ret) { goto the_end; + } } - if (get_tag_value(buf, sizeof(buf), line, "Bus=", " ") < 0) + if (get_tag_value(buf, sizeof(buf), line, "Bus=", " ") < 0) { goto fail; + } bus_num = atoi(buf); - if (get_tag_value(buf, sizeof(buf), line, "Dev#=", " ") < 0) + if (get_tag_value(buf, sizeof(buf), line, "Dev#=", " ") < 0) { goto fail; + } addr = atoi(buf); - if (get_tag_value(buf, sizeof(buf), line, "Spd=", " ") < 0) + if (get_tag_value(buf, sizeof(buf), line, "Spd=", " ") < 0) { goto fail; - if (!strcmp(buf, "480")) + } + if (!strcmp(buf, "480")) { speed = USB_SPEED_HIGH; - else if (!strcmp(buf, "1.5")) + } else if (!strcmp(buf, "1.5")) { speed = USB_SPEED_LOW; - else + } else { speed = USB_SPEED_FULL; + } product_name[0] = '\0'; class_id = 0xff; device_count++; product_id = 0; vendor_id = 0; } else if (line[0] == 'P' && line[1] == ':') { - if (get_tag_value(buf, sizeof(buf), line, "Vendor=", " ") < 0) + if (get_tag_value(buf, sizeof(buf), line, "Vendor=", " ") < 0) { goto fail; + } vendor_id = strtoul(buf, NULL, 16); - if (get_tag_value(buf, sizeof(buf), line, "ProdID=", " ") < 0) + if (get_tag_value(buf, sizeof(buf), line, "ProdID=", " ") < 0) { goto fail; + } product_id = strtoul(buf, NULL, 16); } else if (line[0] == 'S' && line[1] == ':') { - if (get_tag_value(buf, sizeof(buf), line, "Product=", "") < 0) + if (get_tag_value(buf, sizeof(buf), line, "Product=", "") < 0) { goto fail; + } pstrcpy(product_name, sizeof(product_name), buf); } else if (line[0] == 'D' && line[1] == ':') { - if (get_tag_value(buf, sizeof(buf), line, "Cls=", " (") < 0) + if (get_tag_value(buf, sizeof(buf), line, "Cls=", " (") < 0) { goto fail; + } class_id = strtoul(buf, NULL, 16); } fail: ; @@ -1171,8 +1215,9 @@ static int usb_host_scan_dev(void *opaque, USBScanFunc *func) product_id, product_name, speed); } the_end: - if (f) + if (f) { fclose(f); + } return ret; } @@ -1186,7 +1231,8 @@ static int usb_host_scan_dev(void *opaque, USBScanFunc *func) * * @return 0 failed, 1 succeeded ('line' contains data) */ -static int usb_host_read_file(char *line, size_t line_size, const char *device_file, const char *device_name) +static int usb_host_read_file(char *line, size_t line_size, + const char *device_file, const char *device_name) { FILE *f; int ret = 0; @@ -1228,59 +1274,71 @@ static int usb_host_scan_sys(void *opaque, USBScanFunc *func) while ((de = readdir(dir))) { if (de->d_name[0] != '.' && !strchr(de->d_name, ':')) { char *tmpstr = de->d_name; - if (!strncmp(de->d_name, "usb", 3)) + if (!strncmp(de->d_name, "usb", 3)) { tmpstr += 3; + } bus_num = atoi(tmpstr); - if (!usb_host_read_file(line, sizeof(line), "devnum", de->d_name)) + if (!usb_host_read_file(line, sizeof(line), "devnum", de->d_name)) { goto the_end; - if (sscanf(line, "%d", &addr) != 1) + } + if (sscanf(line, "%d", &addr) != 1) { goto the_end; - + } if (!usb_host_read_file(line, sizeof(line), "bDeviceClass", - de->d_name)) + de->d_name)) { goto the_end; - if (sscanf(line, "%x", &class_id) != 1) + } + if (sscanf(line, "%x", &class_id) != 1) { goto the_end; + } - if (!usb_host_read_file(line, sizeof(line), "idVendor", de->d_name)) + if (!usb_host_read_file(line, sizeof(line), "idVendor", + de->d_name)) { goto the_end; - if (sscanf(line, "%x", &vendor_id) != 1) + } + if (sscanf(line, "%x", &vendor_id) != 1) { goto the_end; - + } if (!usb_host_read_file(line, sizeof(line), "idProduct", - de->d_name)) + de->d_name)) { goto the_end; - if (sscanf(line, "%x", &product_id) != 1) + } + if (sscanf(line, "%x", &product_id) != 1) { goto the_end; - + } if (!usb_host_read_file(line, sizeof(line), "product", de->d_name)) { *product_name = 0; } else { - if (strlen(line) > 0) + if (strlen(line) > 0) { line[strlen(line) - 1] = '\0'; + } pstrcpy(product_name, sizeof(product_name), line); } - if (!usb_host_read_file(line, sizeof(line), "speed", de->d_name)) + if (!usb_host_read_file(line, sizeof(line), "speed", de->d_name)) { goto the_end; - if (!strcmp(line, "480\n")) + } + if (!strcmp(line, "480\n")) { speed = USB_SPEED_HIGH; - else if (!strcmp(line, "1.5\n")) + } else if (!strcmp(line, "1.5\n")) { speed = USB_SPEED_LOW; - else + } else { speed = USB_SPEED_FULL; + } ret = func(opaque, bus_num, addr, class_id, vendor_id, product_id, product_name, speed); - if (ret) + if (ret) { goto the_end; + } } } the_end: - if (dir) + if (dir) { closedir(dir); + } return ret; } @@ -1329,17 +1387,19 @@ static int usb_host_scan(void *opaque, USBScanFunc *func) } found_devices: if (!usb_fs_type) { - if (mon) + if (mon) { monitor_printf(mon, "husb: unable to access USB devices\n"); + } return -ENOENT; } /* the module setting (used later for opening devices) */ usb_host_device_path = qemu_mallocz(strlen(devpath)+1); strcpy(usb_host_device_path, devpath); - if (mon) + if (mon) { monitor_printf(mon, "husb: using %s file-system with %s\n", fs_type[usb_fs_type], usb_host_device_path); + } } switch (usb_fs_type) { @@ -1373,27 +1433,29 @@ static int usb_host_auto_scan(void *opaque, int bus_num, int addr, QTAILQ_FOREACH(s, &hostdevs, next) { f = &s->match; - if (f->bus_num > 0 && f->bus_num != bus_num) + if (f->bus_num > 0 && f->bus_num != bus_num) { continue; - - if (f->addr > 0 && f->addr != addr) + } + if (f->addr > 0 && f->addr != addr) { continue; + } - if (f->vendor_id > 0 && f->vendor_id != vendor_id) + if (f->vendor_id > 0 && f->vendor_id != vendor_id) { continue; + } - if (f->product_id > 0 && f->product_id != product_id) + if (f->product_id > 0 && f->product_id != product_id) { continue; - + } /* We got a match */ /* Already attached ? */ - if (s->fd != -1) + if (s->fd != -1) { return 0; - + } DPRINTF("husb: auto open: bus_num %d addr %d\n", bus_num, addr); - usb_host_open(s, bus_num, addr, product_name); + usb_host_open(s, bus_num, addr, product_name); } return 0; @@ -1407,21 +1469,24 @@ static void usb_host_auto_check(void *unused) usb_host_scan(NULL, usb_host_auto_scan); QTAILQ_FOREACH(s, &hostdevs, next) { - if (s->fd == -1) + if (s->fd == -1) { unconnected++; + } } if (unconnected == 0) { /* nothing to watch */ - if (usb_auto_timer) + if (usb_auto_timer) { qemu_del_timer(usb_auto_timer); + } return; } if (!usb_auto_timer) { usb_auto_timer = qemu_new_timer(rt_clock, usb_host_auto_check, NULL); - if (!usb_auto_timer) + if (!usb_auto_timer) { return; + } } qemu_mod_timer(usb_auto_timer, qemu_get_clock(rt_clock) + 2000); } @@ -1451,13 +1516,15 @@ static int parse_filter(const char *spec, struct USBAutoFilter *f) f->product_id = 0; for (i = BUS; i < DONE; i++) { - p = strpbrk(p, ":."); - if (!p) break; + p = strpbrk(p, ":."); + if (!p) { + break; + } p++; - - if (*p == '*') - continue; + if (*p == '*') { + continue; + } switch(i) { case BUS: f->bus_num = strtol(p, NULL, 10); break; case DEV: f->addr = strtol(p, NULL, 10); break; @@ -1503,8 +1570,9 @@ static const char *usb_class_str(uint8_t class) { const struct usb_class_info *p; for(p = usb_class_info; p->class != -1; p++) { - if (p->class == class) + if (p->class == class) { break; + } } return p->class_name; } @@ -1534,13 +1602,15 @@ static void usb_info_device(Monitor *mon, int bus_num, int addr, int class_id, monitor_printf(mon, " Device %d.%d, speed %s Mb/s\n", bus_num, addr, speed_str); class_str = usb_class_str(class_id); - if (class_str) + if (class_str) { monitor_printf(mon, " %s:", class_str); - else + } else { monitor_printf(mon, " Class %02x:", class_id); + } monitor_printf(mon, " USB device %04x:%04x", vendor_id, product_id); - if (product_name[0] != '\0') + if (product_name[0] != '\0') { monitor_printf(mon, ", %s", product_name); + } monitor_printf(mon, "\n"); } @@ -1559,18 +1629,20 @@ static int usb_host_info_device(void *opaque, int bus_num, int addr, static void dec2str(int val, char *str, size_t size) { - if (val == 0) + if (val == 0) { snprintf(str, size, "*"); - else - snprintf(str, size, "%d", val); + } else { + snprintf(str, size, "%d", val); + } } static void hex2str(int val, char *str, size_t size) { - if (val == 0) + if (val == 0) { snprintf(str, size, "*"); - else + } else { snprintf(str, size, "%04x", val); + } } void usb_host_info(Monitor *mon) @@ -1580,8 +1652,10 @@ void usb_host_info(Monitor *mon) usb_host_scan(mon, usb_host_info_device); - if (QTAILQ_EMPTY(&hostdevs)) + if (QTAILQ_EMPTY(&hostdevs)) { return; + } + monitor_printf(mon, " Auto filters:\n"); QTAILQ_FOREACH(s, &hostdevs, next) { char bus[10], addr[10], vid[10], pid[10];