![]() | ||||||||||
Messages in this thread |
On Sat, 23 Feb 2008 01:10:45 -0500 Andres Salomon <dilinger@queued.net> wrote: > @@ -425,7 +424,10 @@ static int __init gxfb_setup(char *options) > if (!*opt) > continue; > > - mode_option = opt; > + if (!strncmp(opt, "fbsize:", 7)) > + fbsize = simple_strtoul(opt+7, NULL, 0); > + else > + mode_option = opt; > } The above shouldn't be necessary. And it should have been documented in Documentation/kernel-parameters.txt. And "fbsize=N" would be a lot more conventional than "fbsize:N" I suspect that the formulation you have here will not permit "fbsize:128k", whereas "fbsize=128k" or "gxfb.fbsize=128k" should work. Needs checking. > return 0; > @@ -456,5 +458,8 @@ module_exit(gxfb_cleanup); > module_param(mode_option, charp, 0); > MODULE_PARM_DESC(mode_option, "video mode (<x>x<y>[-<bpp>][@<refr>])"); > > +module_param(fbsize, int, 0); > +MODULE_PARM_DESC(fbsize, "video memory size"); > + Because the module_param() already makes fbsize available on the kernel boot command line via gxfb.fbsize=42 (or something similar). -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/ | |||||||||
| Last update: 2008-02-28 01:35 [from the cache] ©2003-2008 | ||||||||||