diff options
| author | Pete Zaitcev <zaitcev@redhat.com> | 2007-10-31 18:59:30 -0400 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2007-11-28 16:58:34 -0500 |
| commit | f08812d5eb8f8cd1a5bd5f5c26a96eb93d97ab69 (patch) | |
| tree | 1e71713fca245258755a0f99f58d0b9343424ee4 /drivers/usb/misc/berry_charge.c | |
| parent | 3c886c50486cfbef2a2382b99bf4083a465c970a (diff) | |
USB: FIx locks and urb->status in adutux (updated)
Two main issues fixed here are:
- An improper use of in-struct lock to protect an open count
- Use of urb status for -EINPROGRESS
Also, along the way:
- Change usb_unlink_urb to usb_kill_urb. Apparently there's no need
to use usb_unlink_urb whatsoever in this driver, and the old use of
usb_kill_urb was outright racy (it unlinked and immediately freed).
- Fix indentation in adu_write. Looks like it was damaged by a script.
- Vitaly wants -EBUSY on multiply opens.
- bInterval was taken from a wrong endpoint.
Signed-off-by: Pete Zaitcev <zaitcev@redhat.com>
Signed-off-by: Vitaliy Ivanov <vitalivanov@gmail.com>
Tested-by: Vitaliy Ivanov <vitalivanov@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/usb/misc/berry_charge.c')
0 files changed, 0 insertions, 0 deletions