[KJ] [PATCH] drivers/video/vgastate.c

[KJ] [PATCH] drivers/video/vgastate.c

[Jerome Steunenberg]

This is my first patch to kernel-janitors. Please tell me if I got it all ok
In drivers/video/vgastate.c, the calls to init_module and cleanup_module have been replaced by calls to module_init and module_exit.

Signed-off-by: Jerome Steunenberg <jerome@steunenberg.org>



--- linux-2.6.13-rc3/drivers/video/vgastate.c.orig	2005-07-25 15:09:51.000000000 +0200
+++ linux-2.6.13-rc3/drivers/video/vgastate.c	2005-07-26 10:15:32.000000000 +0200
@@ -491,8 +491,17 @@ int restore_vga (struct vgastate *state)
  }

  #ifdef MODULE
-int init_module(void) { return 0; };
-void cleanup_module(void) {};
+static int __init vgastate_init(void)
+{
+	return 0;
+}
+
+static void __exit vgastate_exit(void)
+{
+}
+
+module_init(vgastate_init);
+module_exit(vgastate_exit);
  #endif

  EXPORT_SYMBOL(save_vga);
_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
https://lists.osdl.org/mailman/listinfo/kernel-janitors

Re: [KJ] [PATCH] drivers/video/vgastate.c

[Domen Puncer]

[-- Attachment #1: Type: text/plain, Size: 1064 bytes --]

On 28/07/05 16:22 +0200, Jerome Steunenberg wrote:
> This is my first patch to kernel-janitors. Please tell me if I got it all ok
> In drivers/video/vgastate.c, the calls to init_module and cleanup_module 
> have been replaced by calls to module_init and module_exit.

"Subject" could be improved

> 
> Signed-off-by: Jerome Steunenberg <jerome@steunenberg.org>
> 
> 
> 
> --- linux-2.6.13-rc3/drivers/video/vgastate.c.orig	2005-07-25 
> 15:09:51.000000000 +0200

Wrapped line. Try playing with your mailer settings.

> +++ linux-2.6.13-rc3/drivers/video/vgastate.c	2005-07-26 
> 10:15:32.000000000 +0200
> @@ -491,8 +491,17 @@ int restore_vga (struct vgastate *state)
>  }
> 
>  #ifdef MODULE
> -int init_module(void) { return 0; };
> -void cleanup_module(void) {};
> +static int __init vgastate_init(void)
> +{
> +	return 0;
> +}
> +
> +static void __exit vgastate_exit(void)
> +{
> +}
> +
> +module_init(vgastate_init);
> +module_exit(vgastate_exit);
>  #endif

How about just removing it? A quick hello world module confirmed it
works without these functions.

[-- Attachment #2: Type: text/plain, Size: 168 bytes --]

_______________________________________________
Kernel-janitors mailing list
Kernel-janitors@lists.osdl.org
https://lists.osdl.org/mailman/listinfo/kernel-janitors