diff options
| author | Fabian Godehardt <fg@emlix.com> | 2011-09-01 08:15:46 -0400 |
|---|---|---|
| committer | Herton Ronaldo Krzesinski <herton.krzesinski@canonical.com> | 2011-11-21 12:54:54 -0500 |
| commit | e34c4c52f88e0102bd3c4dac24fcbb96ed7bc369 (patch) | |
| tree | 71931dd911da97171d6ff09a967497df8e97e5cf /drivers/usb | |
| parent | bef06df5a3e7ed8d89f11ed2bf4ec3a5fcabe437 (diff) | |
USB: g_printer: fix bug in unregistration
BugLink: http://bugs.launchpad.net/bugs/890952
commit 8582d86143c690c68cc42f996def466a035bee34 upstream.
The allocated chardevice region range is only 1 device but on
unregister it currently tries to deregister 2.
Found this while doing a insmod/rmmod/insmod/rm... of the module
which seemed to eat major numbers.
Signed-off-by: Fabian Godehardt <fg@emlix.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Signed-off-by: Tim Gardner <tim.gardner@canonical.com>
Diffstat (limited to 'drivers/usb')
| -rw-r--r-- | drivers/usb/gadget/printer.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/usb/gadget/printer.c b/drivers/usb/gadget/printer.c index 271ef94668e..88a464cc96c 100644 --- a/drivers/usb/gadget/printer.c +++ b/drivers/usb/gadget/printer.c | |||
| @@ -1602,7 +1602,7 @@ cleanup(void) | |||
| 1602 | if (status) | 1602 | if (status) |
| 1603 | ERROR(dev, "usb_gadget_unregister_driver %x\n", status); | 1603 | ERROR(dev, "usb_gadget_unregister_driver %x\n", status); |
| 1604 | 1604 | ||
| 1605 | unregister_chrdev_region(g_printer_devno, 2); | 1605 | unregister_chrdev_region(g_printer_devno, 1); |
| 1606 | class_destroy(usb_gadget_class); | 1606 | class_destroy(usb_gadget_class); |
| 1607 | mutex_unlock(&usb_printer_gadget.lock_printer_io); | 1607 | mutex_unlock(&usb_printer_gadget.lock_printer_io); |
| 1608 | } | 1608 | } |