Commit a8140543 authored by Krzysztof Helt's avatar Krzysztof Helt Committed by Linus Torvalds

s3fb: add option mode_option

Add the option "mode_option".  It also moves mtrr variable to devinitdata
section.
Signed-off-by: default avatarKrzysztof Helt <krzysztof.h1@wp.pl>
Cc: "Antonino A. Daplas" <adaplas@pol.net>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent 1abf9172
...@@ -132,10 +132,10 @@ static const struct svga_timing_regs s3_timing_regs = { ...@@ -132,10 +132,10 @@ static const struct svga_timing_regs s3_timing_regs = {
/* Module parameters */ /* Module parameters */
static char *mode = "640x480-8@60"; static char *mode_option __devinitdata = "640x480-8@60";
#ifdef CONFIG_MTRR #ifdef CONFIG_MTRR
static int mtrr = 1; static int mtrr __devinitdata = 1;
#endif #endif
static int fasttext = 1; static int fasttext = 1;
...@@ -145,8 +145,10 @@ MODULE_AUTHOR("(c) 2006-2007 Ondrej Zajicek <santiago@crfreenet.org>"); ...@@ -145,8 +145,10 @@ MODULE_AUTHOR("(c) 2006-2007 Ondrej Zajicek <santiago@crfreenet.org>");
MODULE_LICENSE("GPL"); MODULE_LICENSE("GPL");
MODULE_DESCRIPTION("fbdev driver for S3 Trio/Virge"); MODULE_DESCRIPTION("fbdev driver for S3 Trio/Virge");
module_param(mode, charp, 0444); module_param(mode_option, charp, 0444);
MODULE_PARM_DESC(mode, "Default video mode ('640x480-8@60', etc)"); MODULE_PARM_DESC(mode_option, "Default video mode ('640x480-8@60', etc)");
module_param_named(mode, mode_option, charp, 0444);
MODULE_PARM_DESC(mode, "Default video mode ('640x480-8@60', etc) (deprecated)");
#ifdef CONFIG_MTRR #ifdef CONFIG_MTRR
module_param(mtrr, int, 0444); module_param(mtrr, int, 0444);
...@@ -960,10 +962,10 @@ static int __devinit s3_pci_probe(struct pci_dev *dev, const struct pci_device_i ...@@ -960,10 +962,10 @@ static int __devinit s3_pci_probe(struct pci_dev *dev, const struct pci_device_i
info->pseudo_palette = (void*) (par->pseudo_palette); info->pseudo_palette = (void*) (par->pseudo_palette);
/* Prepare startup mode */ /* Prepare startup mode */
rc = fb_find_mode(&(info->var), info, mode, NULL, 0, NULL, 8); rc = fb_find_mode(&(info->var), info, mode_option, NULL, 0, NULL, 8);
if (! ((rc == 1) || (rc == 2))) { if (! ((rc == 1) || (rc == 2))) {
rc = -EINVAL; rc = -EINVAL;
dev_err(&(dev->dev), "mode %s not found\n", mode); dev_err(&(dev->dev), "mode %s not found\n", mode_option);
goto err_find_mode; goto err_find_mode;
} }
...@@ -1168,7 +1170,7 @@ static int __init s3fb_setup(char *options) ...@@ -1168,7 +1170,7 @@ static int __init s3fb_setup(char *options)
else if (!strncmp(opt, "fasttext:", 9)) else if (!strncmp(opt, "fasttext:", 9))
fasttext = simple_strtoul(opt + 9, NULL, 0); fasttext = simple_strtoul(opt + 9, NULL, 0);
else else
mode = opt; mode_option = opt;
} }
return 0; return 0;
......
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