lkml.org 
[lkml]   [2004]   [Oct]   [30]   [last100]   RSS Feed
Views: [more markup]  [less markup]  [headers]  [forward] 
 
Messages in this thread
/
DateFri, 29 Oct 2004 21:35:41 -0700
From"Randy.Dunlap" <>
SubjectRe: [PATCH] fbdev: Convert MODULE_PARM to module_param in neofb
Linux Kernel Mailing List wrote:
> ChangeSet 1.2337.1.21, 2004/10/29 13:24:51-07:00, adaplas@hotpop.com
> 
> 	[PATCH] fbdev: Convert MODULE_PARM to module_param in neofb
> 	Convert MODULE_PARM to module_param in neofb
> 
> diff -Nru a/drivers/video/neofb.c b/drivers/video/neofb.c
> @@ -100,18 +100,18 @@
>  MODULE_AUTHOR("(c) 2001-2002  Denis Oliver Kropp <dok@convergence.de>");
>  MODULE_LICENSE("GPL");
>  MODULE_DESCRIPTION("FBDev driver for NeoMagic PCI Chips");
> -MODULE_PARM(internal, "i");
> +module_param(internal, bool, 0);
>  MODULE_PARM_DESC(internal, "Enable output on internal LCD Display.");
> -MODULE_PARM(external, "i");
> +module_param(external, bool, 0);
>  MODULE_PARM_DESC(external, "Enable output on external CRT.");
> -MODULE_PARM(libretto, "i");
> +module_param(libretto, bool, 0);
>  MODULE_PARM_DESC(libretto, "Force Libretto 100/110 800x480 LCD.");
> -MODULE_PARM(nostretch, "i");
> +module_param(nostretch, bool, 0);
>  MODULE_PARM_DESC(nostretch,
>  		 "Disable stretching of modes smaller than LCD.");
> -MODULE_PARM(nopciburst, "i");
> +module_param(nopciburst, bool, 0);
>  MODULE_PARM_DESC(nopciburst, "Disable PCI burst mode.");
> -MODULE_PARM(mode_option, "s");
> +module_param(mode_option, charp, 0);
>  MODULE_PARM_DESC(mode_option, "Preferred video mode ('640x480-8@60', etc)");

Since neofb_setup() still parses options, what does this do?
Provide 2 ways of passing options/arguments?

Same questions for intelfb and i810fb.

Oh, is it this?
neofb_setup() is used for non-modular option parsing,
and module_param() is used for modular or non-modular option parsing.

IOW, using neofb_setup() for option parsing maintains some
backward compatibility for options?

-- 
~Randy
-
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: 2005-03-22 14:07    [from the cache]
©2003-2008