diff options
author | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2012-09-18 11:05:17 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2012-09-18 11:05:17 -0400 |
commit | 3033bc8d74c1a9d9104b9f231283e780e6dd8de7 (patch) | |
tree | a8a319dd600364ef66ce565415d9efce229bef7e | |
parent | d3d6c328e5030da560dcbaaa9ced063d5ba93642 (diff) |
USB: Serial: usb-serial: remove debug module parameter
Now that all usb-serial modules are only using dev_dbg()
the debug module parameter does not do anything at all, so
remove it to reduce any confusion if someone were to try
to use it.
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | drivers/usb/serial/console.c | 6 | ||||
-rw-r--r-- | drivers/usb/serial/generic.c | 5 | ||||
-rw-r--r-- | drivers/usb/serial/usb-serial.c | 8 | ||||
-rw-r--r-- | include/linux/usb/serial.h | 6 |
4 files changed, 7 insertions, 18 deletions
diff --git a/drivers/usb/serial/console.c b/drivers/usb/serial/console.c index 335f98d092cc..6ea18a1ea1a7 100644 --- a/drivers/usb/serial/console.c +++ b/drivers/usb/serial/console.c | |||
@@ -22,8 +22,6 @@ | |||
22 | #include <linux/usb.h> | 22 | #include <linux/usb.h> |
23 | #include <linux/usb/serial.h> | 23 | #include <linux/usb/serial.h> |
24 | 24 | ||
25 | static int debug; | ||
26 | |||
27 | struct usbcons_info { | 25 | struct usbcons_info { |
28 | int magic; | 26 | int magic; |
29 | int break_flag; | 27 | int break_flag; |
@@ -283,10 +281,8 @@ void usb_serial_console_disconnect(struct usb_serial *serial) | |||
283 | } | 281 | } |
284 | } | 282 | } |
285 | 283 | ||
286 | void usb_serial_console_init(int serial_debug, int minor) | 284 | void usb_serial_console_init(int minor) |
287 | { | 285 | { |
288 | debug = serial_debug; | ||
289 | |||
290 | if (minor == 0) { | 286 | if (minor == 0) { |
291 | /* | 287 | /* |
292 | * Call register_console() if this is the first device plugged | 288 | * Call register_console() if this is the first device plugged |
diff --git a/drivers/usb/serial/generic.c b/drivers/usb/serial/generic.c index 47e228030922..296612153ea2 100644 --- a/drivers/usb/serial/generic.c +++ b/drivers/usb/serial/generic.c | |||
@@ -24,8 +24,6 @@ | |||
24 | #include <linux/kfifo.h> | 24 | #include <linux/kfifo.h> |
25 | #include <linux/serial.h> | 25 | #include <linux/serial.h> |
26 | 26 | ||
27 | static int debug; | ||
28 | |||
29 | #ifdef CONFIG_USB_SERIAL_GENERIC | 27 | #ifdef CONFIG_USB_SERIAL_GENERIC |
30 | 28 | ||
31 | static __u16 vendor = 0x05f9; | 29 | static __u16 vendor = 0x05f9; |
@@ -60,11 +58,10 @@ static struct usb_serial_driver * const serial_drivers[] = { | |||
60 | 58 | ||
61 | #endif | 59 | #endif |
62 | 60 | ||
63 | int usb_serial_generic_register(int _debug) | 61 | int usb_serial_generic_register(void) |
64 | { | 62 | { |
65 | int retval = 0; | 63 | int retval = 0; |
66 | 64 | ||
67 | debug = _debug; | ||
68 | #ifdef CONFIG_USB_SERIAL_GENERIC | 65 | #ifdef CONFIG_USB_SERIAL_GENERIC |
69 | generic_device_ids[0].idVendor = vendor; | 66 | generic_device_ids[0].idVendor = vendor; |
70 | generic_device_ids[0].idProduct = product; | 67 | generic_device_ids[0].idProduct = product; |
diff --git a/drivers/usb/serial/usb-serial.c b/drivers/usb/serial/usb-serial.c index 8a9352b2c26f..001185dd127c 100644 --- a/drivers/usb/serial/usb-serial.c +++ b/drivers/usb/serial/usb-serial.c | |||
@@ -52,7 +52,6 @@ | |||
52 | drivers depend on it. | 52 | drivers depend on it. |
53 | */ | 53 | */ |
54 | 54 | ||
55 | static bool debug; | ||
56 | /* initially all NULL */ | 55 | /* initially all NULL */ |
57 | static struct usb_serial *serial_table[SERIAL_TTY_MINORS]; | 56 | static struct usb_serial *serial_table[SERIAL_TTY_MINORS]; |
58 | static DEFINE_MUTEX(table_lock); | 57 | static DEFINE_MUTEX(table_lock); |
@@ -1074,7 +1073,7 @@ static int usb_serial_probe(struct usb_interface *interface, | |||
1074 | 1073 | ||
1075 | serial->disconnected = 0; | 1074 | serial->disconnected = 0; |
1076 | 1075 | ||
1077 | usb_serial_console_init(debug, minor); | 1076 | usb_serial_console_init(minor); |
1078 | exit: | 1077 | exit: |
1079 | module_put(type->driver.owner); | 1078 | module_put(type->driver.owner); |
1080 | return 0; | 1079 | return 0; |
@@ -1259,7 +1258,7 @@ static int __init usb_serial_init(void) | |||
1259 | } | 1258 | } |
1260 | 1259 | ||
1261 | /* register the generic driver, if we should */ | 1260 | /* register the generic driver, if we should */ |
1262 | result = usb_serial_generic_register(debug); | 1261 | result = usb_serial_generic_register(); |
1263 | if (result < 0) { | 1262 | if (result < 0) { |
1264 | pr_err("%s - registering generic driver failed\n", __func__); | 1263 | pr_err("%s - registering generic driver failed\n", __func__); |
1265 | goto exit_generic; | 1264 | goto exit_generic; |
@@ -1464,6 +1463,3 @@ EXPORT_SYMBOL_GPL(usb_serial_deregister_drivers); | |||
1464 | MODULE_AUTHOR(DRIVER_AUTHOR); | 1463 | MODULE_AUTHOR(DRIVER_AUTHOR); |
1465 | MODULE_DESCRIPTION(DRIVER_DESC); | 1464 | MODULE_DESCRIPTION(DRIVER_DESC); |
1466 | MODULE_LICENSE("GPL"); | 1465 | MODULE_LICENSE("GPL"); |
1467 | |||
1468 | module_param(debug, bool, S_IRUGO | S_IWUSR); | ||
1469 | MODULE_PARM_DESC(debug, "Debug enabled or not"); | ||
diff --git a/include/linux/usb/serial.h b/include/linux/usb/serial.h index b2b6d159b26b..5808713aa492 100644 --- a/include/linux/usb/serial.h +++ b/include/linux/usb/serial.h | |||
@@ -307,11 +307,11 @@ extern int ezusb_set_reset(struct usb_device *dev, unsigned char reset_bit); | |||
307 | 307 | ||
308 | /* USB Serial console functions */ | 308 | /* USB Serial console functions */ |
309 | #ifdef CONFIG_USB_SERIAL_CONSOLE | 309 | #ifdef CONFIG_USB_SERIAL_CONSOLE |
310 | extern void usb_serial_console_init(int debug, int minor); | 310 | extern void usb_serial_console_init(int minor); |
311 | extern void usb_serial_console_exit(void); | 311 | extern void usb_serial_console_exit(void); |
312 | extern void usb_serial_console_disconnect(struct usb_serial *serial); | 312 | extern void usb_serial_console_disconnect(struct usb_serial *serial); |
313 | #else | 313 | #else |
314 | static inline void usb_serial_console_init(int debug, int minor) { } | 314 | static inline void usb_serial_console_init(int minor) { } |
315 | static inline void usb_serial_console_exit(void) { } | 315 | static inline void usb_serial_console_exit(void) { } |
316 | static inline void usb_serial_console_disconnect(struct usb_serial *serial) {} | 316 | static inline void usb_serial_console_disconnect(struct usb_serial *serial) {} |
317 | #endif | 317 | #endif |
@@ -333,7 +333,7 @@ extern void usb_serial_generic_throttle(struct tty_struct *tty); | |||
333 | extern void usb_serial_generic_unthrottle(struct tty_struct *tty); | 333 | extern void usb_serial_generic_unthrottle(struct tty_struct *tty); |
334 | extern void usb_serial_generic_disconnect(struct usb_serial *serial); | 334 | extern void usb_serial_generic_disconnect(struct usb_serial *serial); |
335 | extern void usb_serial_generic_release(struct usb_serial *serial); | 335 | extern void usb_serial_generic_release(struct usb_serial *serial); |
336 | extern int usb_serial_generic_register(int debug); | 336 | extern int usb_serial_generic_register(void); |
337 | extern void usb_serial_generic_deregister(void); | 337 | extern void usb_serial_generic_deregister(void); |
338 | extern int usb_serial_generic_submit_read_urbs(struct usb_serial_port *port, | 338 | extern int usb_serial_generic_submit_read_urbs(struct usb_serial_port *port, |
339 | gfp_t mem_flags); | 339 | gfp_t mem_flags); |