Skip to contentSkip to conversation

add static declaration to non-global functions and init_module / cleanup_module fixes

PATCH In review Denis Cheng 2 participants 4 messages Jul 17, 2007

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 add static declaration to non-global functions and init_module / cleanup_module fixes

switch to use module_init / module_exit macros.

To: Bruce Janson <[email protected]>

---
 drivers/net/wireless/wavelan.c |    9 +++++++--
 1 files changed, 7 insertions(+), 2 deletions(-)
drivers/net/wireless/wavelan.c +7 -2
index 1cf090d..a489ef4 100644
@@ -4307,7 +4307,7 @@ out:
  * Insertion of the module
  * I'm now quite proud of the multi-device support.
  */
-int __init init_module(void)
+static int __init wavelan_init(void)
 {
 	int ret = -EIO;		/* Return error if no cards found */
 	int i;
@@ -4371,7 +4371,7 @@ int __init init_module(void)
 /*
  * Removal of the module
  */
-void cleanup_module(void)
+static void __exit wavelan_exit(void)
 {
 #ifdef DEBUG_MODULE_TRACE
 	printk(KERN_DEBUG "-> cleanup_module()\n");
@@ -4398,7 +4398,12 @@ void cleanup_module(void)
 	printk(KERN_DEBUG "<- cleanup_module()\n");
 #endif
 }
+
+module_init(wavelan_init);
+module_exit(wavelan_exit);
+
 #endif				/* MODULE */
+
 MODULE_LICENSE("GPL");
 
 /*
Signature · 1 line
1.5.2.2
#1 lore raw
    From [email protected] Tue Jul 17 14:03:38 2007
    ...
    From: Denis Cheng <[email protected]>
    To: Bruce Janson <[email protected]>
    Cc: Denis Cheng <[email protected]>, [email protected]
    ...
    
    switch to use module_init / module_exit macros.
    
    To: Bruce Janson <[email protected]>
    Cc: [email protected]
    Signed-off-by: Denis Cheng <[email protected]>
    ---
     drivers/net/wireless/wavelan.c |    9 +++++++--
     1 files changed, 7 insertions(+), 2 deletions(-)
    
    diff --git a/drivers/net/wireless/wavelan.c b/drivers/net/wireless/wavelan.c
    index 1cf090d..a489ef4 100644
    --- a/drivers/net/wireless/wavelan.c
    +++ b/drivers/net/wireless/wavelan.c
    @@ -4307,7 +4307,7 @@ out:
      * Insertion of the module
      * I'm now quite proud of the multi-device support.
      */
    -int __init init_module(void)
    +static int __init wavelan_init(void)
     {
     	int ret = -EIO;		/* Return error if no cards found */
     	int i;
    @@ -4371,7 +4371,7 @@ int __init init_module(void)
     /*
      * Removal of the module
      */
    -void cleanup_module(void)
    +static void __exit wavelan_exit(void)
     {
     #ifdef DEBUG_MODULE_TRACE
     	printk(KERN_DEBUG "-> cleanup_module()\n");
    @@ -4398,7 +4398,12 @@ void cleanup_module(void)
     	printk(KERN_DEBUG "<- cleanup_module()\n");
     #endif
     }
    +
    +module_init(wavelan_init);
    +module_exit(wavelan_exit);
    +
     #endif				/* MODULE */
    +
     MODULE_LICENSE("GPL");
     
     /*
    -- 
    1.5.2.2
    ...

Hi Denis, Janitors,
I have not been the maintainer of that device for a number
of years now (10?). I handed it over to Jean Tourrilhes:

http://www.hpl.hp.com/personal/Jean_Tourrilhes/index.html

but I have not kept track of it since then.

Regards,

Bruce Janson, Network Group, ICT Infrastructure     Email:  [email protected]
Rm 125c, Bldg H08, University Computing Centre      Phone:  +61-2-9351-4911

University of Sydney, NSW, Australia, 2006

#3 lore raw
In reply to Denis Cheng
    From [email protected] Tue Jul 17 14:48:34 2007
    ...
    On 7/17/07, Bruce Janson <[email protected]> wrote:
    ...
    >     I have not been the maintainer of that device for a number
    > of years now (10?).  I handed it over to Jean Tourrilhes:
    >
    >   http://www.hpl.hp.com/personal/Jean_Tourrilhes/index.html
    >
    > but I have not kept track of it since then.
    ...

    But now who can take care of it? Who have the hardware to test?
    ...

A more recent maintainer?