diff --git a/reactos/drivers/usb/cromwell/core/config.c b/reactos/drivers/usb/cromwell/core/config.c index bc182734058..01be0d82391 100644 --- a/reactos/drivers/usb/cromwell/core/config.c +++ b/reactos/drivers/usb/cromwell/core/config.c @@ -453,8 +453,9 @@ int usb_get_configuration(struct usb_device *dev) /* configuration is */ result = usb_get_descriptor(dev, USB_DT_CONFIG, cfgno, buffer, 8); if (result < 8) { - if (result < 0) + if (result < 0) { err("unable to get descriptor"); + } else { err("config descriptor too short (expected %i, got %i)", 8, result); result = -EINVAL; diff --git a/reactos/drivers/usb/cromwell/core/hub.c b/reactos/drivers/usb/cromwell/core/hub.c index dda273f94f1..1fa5853f01a 100644 --- a/reactos/drivers/usb/cromwell/core/hub.c +++ b/reactos/drivers/usb/cromwell/core/hub.c @@ -289,9 +289,10 @@ static int hub_hub_status(struct usb_hub *hub, int ret; ret = get_hub_status(dev, &hub->status->hub); - if (ret < 0) + if (ret < 0) { dev_err (hubdev (dev), "%s failed (err = %d)\n", __FUNCTION__, ret); + } else { *status = le16_to_cpu(hub->status->hub.wHubStatus); *change = le16_to_cpu(hub->status->hub.wHubChange); @@ -703,9 +704,10 @@ static int hub_port_status(struct usb_device *dev, int port, int ret; ret = get_port_status(dev, port + 1, &hub->status->port); - if (ret < 0) + if (ret < 0) { dev_err (hubdev (dev), "%s failed (err = %d)\n", __FUNCTION__, ret); + } else { *status = le16_to_cpu(hub->status->port.wPortStatus); *change = le16_to_cpu(hub->status->port.wPortChange); @@ -1115,8 +1117,9 @@ static void hub_events(void) } /* end for i */ /* deal with hub status changes */ - if (hub_hub_status(hub, &hubstatus, &hubchange) < 0) + if (hub_hub_status(hub, &hubstatus, &hubchange) < 0) { dev_err (&hub->intf->dev, "get_hub_status failed\n"); + } else { if (hubchange & HUB_CHANGE_LOCAL_POWER) { dev_dbg (&hub->intf->dev, "power change\n"); @@ -1321,15 +1324,16 @@ int usb_physical_reset_device(struct usb_device *dev) ret = usb_get_device_descriptor(dev); if (ret < sizeof(dev->descriptor)) { - if (ret < 0) + if (ret < 0) { err("unable to get device %s descriptor " "(error=%d)", dev->devpath, ret); - else + } + else { err("USB device %s descriptor short read " "(expected %Zi, got %i)", dev->devpath, sizeof(dev->descriptor), ret); - + } clear_bit(dev->devnum, dev->bus->devmap.devicemap); dev->devnum = -1; return -EIO; diff --git a/reactos/drivers/usb/cromwell/core/usb.c b/reactos/drivers/usb/cromwell/core/usb.c index 443b7090c0f..f1dcb00eaad 100644 --- a/reactos/drivers/usb/cromwell/core/usb.c +++ b/reactos/drivers/usb/cromwell/core/usb.c @@ -275,12 +275,13 @@ void usb_driver_claim_interface(struct usb_driver *driver, struct usb_interface return; // FIXME change API to report an error in this case - if (iface->driver) + if (iface->driver) { err ("%s driver booted %s off interface %p", driver->name, iface->driver->name, iface); - else + } + else { dbg("%s driver claimed interface %p", driver->name, iface); - + } iface->driver = driver; usb_set_intfdata(iface, priv); } @@ -1139,10 +1140,12 @@ int usb_new_device(struct usb_device *dev, struct device *parent) } if (err < 8) { - if (err < 0) + if (err < 0) { dev_err(&dev->dev, "USB device not responding, giving up (error=%d)\n", err); - else + } + else { dev_err(&dev->dev, "USB device descriptor short read (expected %i, got %i)\n", 8, err); + } clear_bit(dev->devnum, dev->bus->devmap.devicemap); dev->devnum = -1; return 1; @@ -1156,11 +1159,13 @@ int usb_new_device(struct usb_device *dev, struct device *parent) err = usb_get_device_descriptor(dev); if (err < (signed)sizeof(dev->descriptor)) { - if (err < 0) + if (err < 0) { dev_err(&dev->dev, "unable to get device descriptor (error=%d)\n", err); - else + } + else { dev_err(&dev->dev, "USB device descriptor short read (expected %Zi, got %i)\n", sizeof(dev->descriptor), err); + } clear_bit(dev->devnum, dev->bus->devmap.devicemap); dev->devnum = -1;