diff options
author | Krzysztof Helt <krzysztof.h1@wp.pl> | 2008-04-28 05:15:08 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-04-28 11:58:37 -0400 |
commit | 1abf91729faf2fd9b16b5987a68fb99fe5dcc75a (patch) | |
tree | 3158b7a2fcafde4a55eea288e8f8faf90487bd2b | |
parent | cc6c549c7a9808cc7a8a5afbfa54dbbd2262509d (diff) |
arkfb: add option mode_option
Add the option "mode_option". This is one step toward changing all fb
drivers to have common "mode_option" parameter.
Signed-off-by: Krzysztof Helt <krzysztof.h1@wp.pl>
Cc: "Antonino A. Daplas" <adaplas@pol.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | drivers/video/arkfb.c | 14 |
1 files changed, 8 insertions, 6 deletions
diff --git a/drivers/video/arkfb.c b/drivers/video/arkfb.c index 8a1b07c74394..0f6e0043069d 100644 --- a/drivers/video/arkfb.c +++ b/drivers/video/arkfb.c | |||
@@ -101,7 +101,7 @@ static const struct svga_timing_regs ark_timing_regs = { | |||
101 | 101 | ||
102 | /* Module parameters */ | 102 | /* Module parameters */ |
103 | 103 | ||
104 | static char *mode = "640x480-8@60"; | 104 | static char *mode_option __devinitdata = "640x480-8@60"; |
105 | 105 | ||
106 | #ifdef CONFIG_MTRR | 106 | #ifdef CONFIG_MTRR |
107 | static int mtrr = 1; | 107 | static int mtrr = 1; |
@@ -111,8 +111,10 @@ MODULE_AUTHOR("(c) 2007 Ondrej Zajicek <santiago@crfreenet.org>"); | |||
111 | MODULE_LICENSE("GPL"); | 111 | MODULE_LICENSE("GPL"); |
112 | MODULE_DESCRIPTION("fbdev driver for ARK 2000PV"); | 112 | MODULE_DESCRIPTION("fbdev driver for ARK 2000PV"); |
113 | 113 | ||
114 | module_param(mode, charp, 0444); | 114 | module_param(mode_option, charp, 0444); |
115 | MODULE_PARM_DESC(mode, "Default video mode ('640x480-8@60', etc)"); | 115 | MODULE_PARM_DESC(mode_option, "Default video mode ('640x480-8@60', etc)"); |
116 | module_param_named(mode, mode_option, charp, 0444); | ||
117 | MODULE_PARM_DESC(mode, "Default video mode ('640x480-8@60', etc) (deprecated)"); | ||
116 | 118 | ||
117 | #ifdef CONFIG_MTRR | 119 | #ifdef CONFIG_MTRR |
118 | module_param(mtrr, int, 0444); | 120 | module_param(mtrr, int, 0444); |
@@ -999,10 +1001,10 @@ static int __devinit ark_pci_probe(struct pci_dev *dev, const struct pci_device_ | |||
999 | info->pseudo_palette = (void*) (par->pseudo_palette); | 1001 | info->pseudo_palette = (void*) (par->pseudo_palette); |
1000 | 1002 | ||
1001 | /* Prepare startup mode */ | 1003 | /* Prepare startup mode */ |
1002 | rc = fb_find_mode(&(info->var), info, mode, NULL, 0, NULL, 8); | 1004 | rc = fb_find_mode(&(info->var), info, mode_option, NULL, 0, NULL, 8); |
1003 | if (! ((rc == 1) || (rc == 2))) { | 1005 | if (! ((rc == 1) || (rc == 2))) { |
1004 | rc = -EINVAL; | 1006 | rc = -EINVAL; |
1005 | dev_err(&(dev->dev), "mode %s not found\n", mode); | 1007 | dev_err(&(dev->dev), "mode %s not found\n", mode_option); |
1006 | goto err_find_mode; | 1008 | goto err_find_mode; |
1007 | } | 1009 | } |
1008 | 1010 | ||
@@ -1190,7 +1192,7 @@ static int __init arkfb_init(void) | |||
1190 | return -ENODEV; | 1192 | return -ENODEV; |
1191 | 1193 | ||
1192 | if (option && *option) | 1194 | if (option && *option) |
1193 | mode = option; | 1195 | mode_option = option; |
1194 | #endif | 1196 | #endif |
1195 | 1197 | ||
1196 | pr_debug("arkfb: initializing\n"); | 1198 | pr_debug("arkfb: initializing\n"); |