diff options
author | Rusty Russell <rusty@rustcorp.com.au> | 2011-12-19 09:08:01 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2011-12-19 22:27:29 -0500 |
commit | eb93992207dadb946a3b5cf4544957dc924a6f58 (patch) | |
tree | 8d840ea0e0e5eb3cf6207a2608d434475a9076b4 /drivers/net/usb | |
parent | 3db1cd5c05f35fb43eb134df6f321de4e63141f2 (diff) |
module_param: make bool parameters really bool (net & drivers/net)
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.
(Thanks to Joe Perches for suggesting coccinelle for 0/1 -> true/false).
Cc: "David S. Miller" <davem@davemloft.net>
Cc: netdev@vger.kernel.org
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/usb')
-rw-r--r-- | drivers/net/usb/pegasus.c | 4 | ||||
-rw-r--r-- | drivers/net/usb/smsc75xx.c | 2 | ||||
-rw-r--r-- | drivers/net/usb/smsc95xx.c | 2 |
3 files changed, 4 insertions, 4 deletions
diff --git a/drivers/net/usb/pegasus.c b/drivers/net/usb/pegasus.c index 769f5090bda1..908b42710399 100644 --- a/drivers/net/usb/pegasus.c +++ b/drivers/net/usb/pegasus.c | |||
@@ -55,8 +55,8 @@ static const char driver_name[] = "pegasus"; | |||
55 | #define BMSR_MEDIA (BMSR_10HALF | BMSR_10FULL | BMSR_100HALF | \ | 55 | #define BMSR_MEDIA (BMSR_10HALF | BMSR_10FULL | BMSR_100HALF | \ |
56 | BMSR_100FULL | BMSR_ANEGCAPABLE) | 56 | BMSR_100FULL | BMSR_ANEGCAPABLE) |
57 | 57 | ||
58 | static int loopback; | 58 | static bool loopback; |
59 | static int mii_mode; | 59 | static bool mii_mode; |
60 | static char *devid; | 60 | static char *devid; |
61 | 61 | ||
62 | static struct usb_eth_dev usb_dev_id[] = { | 62 | static struct usb_eth_dev usb_dev_id[] = { |
diff --git a/drivers/net/usb/smsc75xx.c b/drivers/net/usb/smsc75xx.c index 7d62c39f65cf..0d5da82f0ff7 100644 --- a/drivers/net/usb/smsc75xx.c +++ b/drivers/net/usb/smsc75xx.c | |||
@@ -76,7 +76,7 @@ struct usb_context { | |||
76 | struct usbnet *dev; | 76 | struct usbnet *dev; |
77 | }; | 77 | }; |
78 | 78 | ||
79 | static int turbo_mode = true; | 79 | static bool turbo_mode = true; |
80 | module_param(turbo_mode, bool, 0644); | 80 | module_param(turbo_mode, bool, 0644); |
81 | MODULE_PARM_DESC(turbo_mode, "Enable multiple frames per Rx transaction"); | 81 | MODULE_PARM_DESC(turbo_mode, "Enable multiple frames per Rx transaction"); |
82 | 82 | ||
diff --git a/drivers/net/usb/smsc95xx.c b/drivers/net/usb/smsc95xx.c index 56f3894d701a..db217ad66f26 100644 --- a/drivers/net/usb/smsc95xx.c +++ b/drivers/net/usb/smsc95xx.c | |||
@@ -59,7 +59,7 @@ struct usb_context { | |||
59 | struct usbnet *dev; | 59 | struct usbnet *dev; |
60 | }; | 60 | }; |
61 | 61 | ||
62 | static int turbo_mode = true; | 62 | static bool turbo_mode = true; |
63 | module_param(turbo_mode, bool, 0644); | 63 | module_param(turbo_mode, bool, 0644); |
64 | MODULE_PARM_DESC(turbo_mode, "Enable multiple frames per Rx transaction"); | 64 | MODULE_PARM_DESC(turbo_mode, "Enable multiple frames per Rx transaction"); |
65 | 65 | ||