LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
* [PATCH 3/11] /drivers/char/ip2/ip2main.c replaced init_module&cleanup_module with module_init&module_exit
@ 2008-03-05  5:49 Jon Schindler
  2008-03-05  6:28 ` Harvey Harrison
  0 siblings, 1 reply; 2+ messages in thread
From: Jon Schindler @ 2008-03-05  5:49 UTC (permalink / raw)
  To: linux-kernel, kernel-janitors, trivial, Andrew Morton

Replaced init_module and cleanup_module with static functions and module_init/module_exit.

Signed-off-by: Jon Schindler <jkschind@gmail.com>
---
diff --git a/drivers/char/ip2/ip2main.c b/drivers/char/ip2/ip2main.c
index b1d6cad..048e7c8 100644
--- a/drivers/char/ip2/ip2main.c
+++ b/drivers/char/ip2/ip2main.c
@@ -354,14 +354,15 @@ have_requested_irq( char irq )
 /* the driver initialisation function and returns what it returns.            */
 /******************************************************************************/
 #ifdef MODULE
-int
-init_module(void)
+static int __init
+ip2_init_module(void)
 {
 #ifdef IP2DEBUG_INIT
 	printk (KERN_DEBUG "Loading module ...\n" );
 #endif
     return 0;
 }
+module_init(ip2_init_module);
 #endif /* MODULE */
 
 /******************************************************************************/
@@ -380,8 +381,8 @@ init_module(void)
 /* driver should be returned since it may be unloaded from memory.            */
 /******************************************************************************/
 #ifdef MODULE
-void
-cleanup_module(void)
+void __exit
+ip2_cleanup_module(void)
 {
 	int err;
 	int i;
@@ -451,6 +452,7 @@ cleanup_module(void)
 	printk (KERN_DEBUG "IP2 Unloaded\n" );
 #endif
 }
+module_exit(ip2_cleanup_module);
 #endif /* MODULE */
 
 static const struct tty_operations ip2_ops = {

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH 3/11] /drivers/char/ip2/ip2main.c replaced init_module&cleanup_module with module_init&module_exit
  2008-03-05  5:49 [PATCH 3/11] /drivers/char/ip2/ip2main.c replaced init_module&cleanup_module with module_init&module_exit Jon Schindler
@ 2008-03-05  6:28 ` Harvey Harrison
  0 siblings, 0 replies; 2+ messages in thread
From: Harvey Harrison @ 2008-03-05  6:28 UTC (permalink / raw)
  To: Jon Schindler; +Cc: linux-kernel, kernel-janitors, trivial, Andrew Morton

On Tue, 2008-03-04 at 23:49 -0600, Jon Schindler wrote:
> Replaced init_module and cleanup_module with static functions and module_init/module_exit.
> 

trivial comments

>  #ifdef MODULE
> -int
> -init_module(void)
> +static int __init
> +ip2_init_module(void)
>  {

+static int __init ip2_init_module(void)

> ****************************************************************/
>  #ifdef MODULE
> -void
> -cleanup_module(void)
> +void __exit
> +ip2_cleanup_module(void)
>  {

+void __exit ip2_cleanup_module(void)

Cheers,

Harvey


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2008-03-05  6:29 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2008-03-05  5:49 [PATCH 3/11] /drivers/char/ip2/ip2main.c replaced init_module&cleanup_module with module_init&module_exit Jon Schindler
2008-03-05  6:28 ` Harvey Harrison

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).