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/hwmon/max1668.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/hwmon/max1668.c')
-rw-r--r-- | drivers/hwmon/max1668.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/hwmon/max1668.c b/drivers/hwmon/max1668.c index 6914195cfd35..88953f99e914 100644 --- a/drivers/hwmon/max1668.c +++ b/drivers/hwmon/max1668.c | |||
@@ -59,7 +59,7 @@ static unsigned short max1668_addr_list[] = { | |||
59 | #define DEV_ID_MAX1989 0xb | 59 | #define DEV_ID_MAX1989 0xb |
60 | 60 | ||
61 | /* read only mode module parameter */ | 61 | /* read only mode module parameter */ |
62 | static int read_only; | 62 | static bool read_only; |
63 | module_param(read_only, bool, 0); | 63 | module_param(read_only, bool, 0); |
64 | MODULE_PARM_DESC(read_only, "Don't set any values, read only mode"); | 64 | MODULE_PARM_DESC(read_only, "Don't set any values, read only mode"); |
65 | 65 | ||