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/char/tpm/tpm_tis.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/char/tpm/tpm_tis.c')
-rw-r--r-- | drivers/char/tpm/tpm_tis.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/char/tpm/tpm_tis.c b/drivers/char/tpm/tpm_tis.c index 10cc44ceb5d1..a1748621111b 100644 --- a/drivers/char/tpm/tpm_tis.c +++ b/drivers/char/tpm/tpm_tis.c | |||
@@ -255,7 +255,7 @@ out: | |||
255 | return size; | 255 | return size; |
256 | } | 256 | } |
257 | 257 | ||
258 | static int itpm; | 258 | static bool itpm; |
259 | module_param(itpm, bool, 0444); | 259 | module_param(itpm, bool, 0444); |
260 | MODULE_PARM_DESC(itpm, "Force iTPM workarounds (found on some Lenovo laptops)"); | 260 | MODULE_PARM_DESC(itpm, "Force iTPM workarounds (found on some Lenovo laptops)"); |
261 | 261 | ||
@@ -500,7 +500,7 @@ static irqreturn_t tis_int_handler(int dummy, void *dev_id) | |||
500 | return IRQ_HANDLED; | 500 | return IRQ_HANDLED; |
501 | } | 501 | } |
502 | 502 | ||
503 | static int interrupts = 1; | 503 | static bool interrupts = 1; |
504 | module_param(interrupts, bool, 0444); | 504 | module_param(interrupts, bool, 0444); |
505 | MODULE_PARM_DESC(interrupts, "Enable interrupts"); | 505 | MODULE_PARM_DESC(interrupts, "Enable interrupts"); |
506 | 506 | ||
@@ -828,7 +828,7 @@ static struct platform_driver tis_drv = { | |||
828 | 828 | ||
829 | static struct platform_device *pdev; | 829 | static struct platform_device *pdev; |
830 | 830 | ||
831 | static int force; | 831 | static bool force; |
832 | module_param(force, bool, 0444); | 832 | module_param(force, bool, 0444); |
833 | MODULE_PARM_DESC(force, "Force device probe rather than using ACPI entry"); | 833 | MODULE_PARM_DESC(force, "Force device probe rather than using ACPI entry"); |
834 | static int __init init_tis(void) | 834 | static int __init init_tis(void) |