diff options
| author | Milind Choudhary <milindc@codeaurora.org> | 2012-12-12 17:55:28 -0800 |
|---|---|---|
| committer | Marek Vasut <marex@denx.de> | 2012-12-17 15:38:15 +0100 |
| commit | 359439d2888fdc3797903963607bb0bbd4582e08 (patch) | |
| tree | 53b04bf80b9d3fbc57b7feb318092048e007bb0a /common | |
| parent | b6d7852cf8b7ced786937d3d080956b0c7be9836 (diff) | |
| download | u-boot-imx-359439d2888fdc3797903963607bb0bbd4582e08.zip u-boot-imx-359439d2888fdc3797903963607bb0bbd4582e08.tar.gz u-boot-imx-359439d2888fdc3797903963607bb0bbd4582e08.tar.bz2 | |
usb: Clean up newly allocated device nodes in case of configuration failure
If probe of a newly connected device fails for some reason, clean up
the allocated entry in usb_dev array.
Signed-off-by: Milind Choudhary <milindc@codeaurora.org>
Signed-off-by: Simon Glass <sjg@chromium.org>
Diffstat (limited to 'common')
| -rw-r--r-- | common/usb.c | 12 | ||||
| -rw-r--r-- | common/usb_hub.c | 2 |
2 files changed, 14 insertions, 0 deletions
diff --git a/common/usb.c b/common/usb.c index ac9b4ca..6fc0fc1 100644 --- a/common/usb.c +++ b/common/usb.c @@ -805,6 +805,18 @@ struct usb_device *usb_alloc_new_device(void *controller) return &usb_dev[dev_index - 1]; } +/* + * Free the newly created device node. + * Called in error cases where configuring a newly attached + * device fails for some reason. + */ +void usb_free_device(void) +{ + dev_index--; + USB_PRINTF("Freeing device node: %d\n", dev_index); + memset(&usb_dev[dev_index], 0, sizeof(struct usb_device)); + usb_dev[dev_index].devnum = -1; +} /* * By the time we get here, the device has gotten a new device ID diff --git a/common/usb_hub.c b/common/usb_hub.c index 4d75b90..b5eeb62 100644 --- a/common/usb_hub.c +++ b/common/usb_hub.c @@ -259,6 +259,8 @@ void usb_hub_port_connect_change(struct usb_device *dev, int port) /* Run it through the hoops (find a driver, etc) */ if (usb_new_device(usb)) { /* Woops, disable the port */ + usb_free_device(); + dev->children[port] = NULL; USB_HUB_PRINTF("hub: disabling port %d\n", port + 1); usb_clear_port_feature(dev, port + 1, USB_PORT_FEAT_ENABLE); } |