diff options
author | Peter Oh <poh@broadcom.com> | 2013-09-16 17:21:14 -0400 |
---|---|---|
committer | Felipe Balbi <balbi@ti.com> | 2013-09-17 12:07:14 -0400 |
commit | 7f2ccc8cb28cb3ec2f114486a3ccd57deae1560f (patch) | |
tree | 7c04fbc3dd7f4584935c3f8e5b3498b3813232b4 /drivers/usb | |
parent | 660479aad1cd9d862b640c9dd9b91858cbb712b2 (diff) |
usb: gadget: f_mass_storage: reset endpoint driver data when disabled
Gadgets endpoint driver data is a criteria to judge that
whether the endpoints are in use or not. When gadget gets
assigned an endpoint from endpoint list, they check its
driver data if the driver data is NULL.
If the driver data is not NULL then they regard it as in use.
Therefore all of gadgets should reset their endpoints driver
data to NULL as they are disabled. Otherwise it causes a leak
of endpoint resource.
Signed-off-by: Peter Oh <poh@broadcom.com>
Signed-off-by: Felipe Balbi <balbi@ti.com>
Diffstat (limited to 'drivers/usb')
-rw-r--r-- | drivers/usb/gadget/f_mass_storage.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/usb/gadget/f_mass_storage.c b/drivers/usb/gadget/f_mass_storage.c index 313b835eedfd..a01d7d38c016 100644 --- a/drivers/usb/gadget/f_mass_storage.c +++ b/drivers/usb/gadget/f_mass_storage.c | |||
@@ -2260,10 +2260,12 @@ reset: | |||
2260 | /* Disable the endpoints */ | 2260 | /* Disable the endpoints */ |
2261 | if (fsg->bulk_in_enabled) { | 2261 | if (fsg->bulk_in_enabled) { |
2262 | usb_ep_disable(fsg->bulk_in); | 2262 | usb_ep_disable(fsg->bulk_in); |
2263 | fsg->bulk_in->driver_data = NULL; | ||
2263 | fsg->bulk_in_enabled = 0; | 2264 | fsg->bulk_in_enabled = 0; |
2264 | } | 2265 | } |
2265 | if (fsg->bulk_out_enabled) { | 2266 | if (fsg->bulk_out_enabled) { |
2266 | usb_ep_disable(fsg->bulk_out); | 2267 | usb_ep_disable(fsg->bulk_out); |
2268 | fsg->bulk_out->driver_data = NULL; | ||
2267 | fsg->bulk_out_enabled = 0; | 2269 | fsg->bulk_out_enabled = 0; |
2268 | } | 2270 | } |
2269 | 2271 | ||