diff options
author | patacongo <patacongo@42af7a65-404d-4744-a932-0658087f49c3> | 2012-04-25 16:47:28 +0000 |
---|---|---|
committer | patacongo <patacongo@42af7a65-404d-4744-a932-0658087f49c3> | 2012-04-25 16:47:28 +0000 |
commit | ffcaa123f477ac9062398c06b68950203f0c899a (patch) | |
tree | 5ae0d3a4edccd428088dcf117a5241387c6d6f21 /nuttx/drivers | |
parent | 798d0d0debae5b8dc0f40844adf18ce14906cfd7 (diff) | |
download | px4-nuttx-ffcaa123f477ac9062398c06b68950203f0c899a.tar.gz px4-nuttx-ffcaa123f477ac9062398c06b68950203f0c899a.tar.bz2 px4-nuttx-ffcaa123f477ac9062398c06b68950203f0c899a.zip |
Fix infinite loop in CDC/ACM driver
git-svn-id: svn://svn.code.sf.net/p/nuttx/code/trunk@4655 42af7a65-404d-4744-a932-0658087f49c3
Diffstat (limited to 'nuttx/drivers')
-rw-r--r-- | nuttx/drivers/usbdev/cdcacm.c | 18 |
1 files changed, 10 insertions, 8 deletions
diff --git a/nuttx/drivers/usbdev/cdcacm.c b/nuttx/drivers/usbdev/cdcacm.c index 2065b5433..24903b504 100644 --- a/nuttx/drivers/usbdev/cdcacm.c +++ b/nuttx/drivers/usbdev/cdcacm.c @@ -2182,14 +2182,16 @@ void cdcacm_uninitialize(FAR void *handle) usbtrace(TRACE_CLSERROR(USBSER_TRACEERR_UARTUNREGISTER), (uint16_t)-ret); } - /* Unbind the class (if still bound) */ - - if (priv->usbdev) - { - cdcacm_unbind(&drvr->drvr, priv->usbdev); - } - - /* Unregister the driver (unless we are a part of a composite device) */ + /* Unregister the driver (unless we are a part of a composite device). The + * device unregister logic will (1) return all of the requests to us then + * (2) all the unbind method. + * + * The same thing will happen in the composite case except that: (1) the + * composite driver will call usbdev_unregister() which will (2) return the + * requests for all members of the composite, and (3) call the unbind + * method in the composite device which will (4) call the unbind method + * for this device. + */ #ifndef CONFIG_CDCACM_COMPOSITE usbdev_unregister(&drvr->drvr); |