From: Rusty Russell Date: Thu, 12 Jan 2012 22:57:34 +0000 (+1030) Subject: module_param: make bool parameters really bool (drivers/video/i810) X-Git-Url: http://drtracing.org/?a=commitdiff_plain;h=f787f32e67e00b072f46b2ae3c454d2c0a1fcdb7;p=deliverable%2Flinux.git module_param: make bool parameters really bool (drivers/video/i810) 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. Signed-off-by: Rusty Russell Signed-off-by: Florian Tobias Schandinat --- diff --git a/drivers/video/i810/i810_main.c b/drivers/video/i810/i810_main.c index 5a4ac3c85331..b83f36190cae 100644 --- a/drivers/video/i810/i810_main.c +++ b/drivers/video/i810/i810_main.c @@ -135,8 +135,8 @@ static struct pci_driver i810fb_driver = { static char *mode_option __devinitdata = NULL; static int vram __devinitdata = 4; static int bpp __devinitdata = 8; -static int mtrr __devinitdata; -static int accel __devinitdata; +static bool mtrr __devinitdata; +static bool accel __devinitdata; static int hsync1 __devinitdata; static int hsync2 __devinitdata; static int vsync1 __devinitdata; @@ -144,9 +144,9 @@ static int vsync2 __devinitdata; static int xres __devinitdata; static int yres; static int vyres __devinitdata; -static int sync __devinitdata; -static int extvga __devinitdata; -static int dcolor __devinitdata; +static bool sync __devinitdata; +static bool extvga __devinitdata; +static bool dcolor __devinitdata; static bool ddc3 __devinitdata; /*------------------------------------------------------------*/