diff options
author | Rusty Russell <rusty@rustcorp.com.au> | 2012-01-12 18:02:20 -0500 |
---|---|---|
committer | Rusty Russell <rusty@rustcorp.com.au> | 2012-01-12 18:02:20 -0500 |
commit | 90ab5ee94171b3e28de6bb42ee30b527014e0be7 (patch) | |
tree | fcf89889f6e881f2b231d3d20287c08174ce4b54 /drivers/usb/gadget/serial.c | |
parent | 476bc0015bf09dad39d36a8b19f76f0c181d1ec9 (diff) |
module_param: make bool parameters really bool (drivers & misc)
module_param(bool) used to counter-intuitively take an int. In
fddd5201 (mid-2009) we allowed bool or int/unsigned int using a messy
trick.
It's time to remove the int/unsigned int option. For this version
it'll simply give a warning, but it'll break next kernel version.
Acked-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Diffstat (limited to 'drivers/usb/gadget/serial.c')
-rw-r--r-- | drivers/usb/gadget/serial.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/usb/gadget/serial.c b/drivers/usb/gadget/serial.c index ed1b816e58d8..ad9e5b2df642 100644 --- a/drivers/usb/gadget/serial.c +++ b/drivers/usb/gadget/serial.c | |||
@@ -123,11 +123,11 @@ MODULE_AUTHOR("Al Borchers"); | |||
123 | MODULE_AUTHOR("David Brownell"); | 123 | MODULE_AUTHOR("David Brownell"); |
124 | MODULE_LICENSE("GPL"); | 124 | MODULE_LICENSE("GPL"); |
125 | 125 | ||
126 | static int use_acm = true; | 126 | static bool use_acm = true; |
127 | module_param(use_acm, bool, 0); | 127 | module_param(use_acm, bool, 0); |
128 | MODULE_PARM_DESC(use_acm, "Use CDC ACM, default=yes"); | 128 | MODULE_PARM_DESC(use_acm, "Use CDC ACM, default=yes"); |
129 | 129 | ||
130 | static int use_obex = false; | 130 | static bool use_obex = false; |
131 | module_param(use_obex, bool, 0); | 131 | module_param(use_obex, bool, 0); |
132 | MODULE_PARM_DESC(use_obex, "Use CDC OBEX, default=no"); | 132 | MODULE_PARM_DESC(use_obex, "Use CDC OBEX, default=no"); |
133 | 133 | ||