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/can | |
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/can')
-rw-r--r-- | drivers/net/can/vcan.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/can/vcan.c b/drivers/net/can/vcan.c index f93e2d6fc88c..ea2d94285936 100644 --- a/drivers/net/can/vcan.c +++ b/drivers/net/can/vcan.c | |||
@@ -63,7 +63,7 @@ MODULE_AUTHOR("Urs Thuermann <urs.thuermann@volkswagen.de>"); | |||
63 | * See Documentation/networking/can.txt for details. | 63 | * See Documentation/networking/can.txt for details. |
64 | */ | 64 | */ |
65 | 65 | ||
66 | static int echo; /* echo testing. Default: 0 (Off) */ | 66 | static bool echo; /* echo testing. Default: 0 (Off) */ |
67 | module_param(echo, bool, S_IRUGO); | 67 | module_param(echo, bool, S_IRUGO); |
68 | MODULE_PARM_DESC(echo, "Echo sent frames (for testing). Default: 0 (Off)"); | 68 | MODULE_PARM_DESC(echo, "Echo sent frames (for testing). Default: 0 (Off)"); |
69 | 69 | ||