Commit 109782ec authored by Rusty Russell's avatar Rusty Russell Committed by Florian Tobias Schandinat

i810: fix module_param bool abuse.

The driver says "module_param(ddc3, bool, 0);".  But it's not a used
as a bool, it's used as a count.

Make it a bool.

Cc: Antonino Daplas <adaplas@gmail.com>
Cc: linux-fbdev@vger.kernel.org
Signed-off-by: default avatarRusty Russell <rusty@rustcorp.com.au>
Signed-off-by: default avatarFlorian Tobias Schandinat <FlorianSchandinat@gmx.de>
parent 00f1f199
...@@ -147,7 +147,7 @@ static int vyres __devinitdata; ...@@ -147,7 +147,7 @@ static int vyres __devinitdata;
static int sync __devinitdata; static int sync __devinitdata;
static int extvga __devinitdata; static int extvga __devinitdata;
static int dcolor __devinitdata; static int dcolor __devinitdata;
static int ddc3 __devinitdata = 2; static bool ddc3 __devinitdata;
/*------------------------------------------------------------*/ /*------------------------------------------------------------*/
...@@ -1776,7 +1776,7 @@ static void __devinit i810_init_defaults(struct i810fb_par *par, ...@@ -1776,7 +1776,7 @@ static void __devinit i810_init_defaults(struct i810fb_par *par,
if (sync) if (sync)
par->dev_flags |= ALWAYS_SYNC; par->dev_flags |= ALWAYS_SYNC;
par->ddc_num = ddc3; par->ddc_num = (ddc3 ? 3 : 2);
if (bpp < 8) if (bpp < 8)
bpp = 8; bpp = 8;
...@@ -1999,7 +1999,7 @@ static int __devinit i810fb_setup(char *options) ...@@ -1999,7 +1999,7 @@ static int __devinit i810fb_setup(char *options)
else if (!strncmp(this_opt, "dcolor", 6)) else if (!strncmp(this_opt, "dcolor", 6))
dcolor = 1; dcolor = 1;
else if (!strncmp(this_opt, "ddc3", 4)) else if (!strncmp(this_opt, "ddc3", 4))
ddc3 = 3; ddc3 = true;
else else
mode_option = this_opt; mode_option = this_opt;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment