Skip to contentSkip to conversation

drivers/net/hamradio/: cleanups

PATCH In review Adrian Bunk 3 participants 6 messages May 2, 2005 – Jul 23, 2005

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

PATCH 1/1 drivers/net/hamradio/: cleanups

This patch contains the following cleanups:
- mkiss.c: make a needlessly global variable static
- dmascc.c: remove the unused global function dmascc_setup

---

This patch was already sent on:
- 19 Apr 2005

 drivers/net/hamradio/dmascc.c |   10 ----------
 drivers/net/hamradio/mkiss.c  |    2 +-
 2 files changed, 1 insertion(+), 11 deletions(-)
drivers/net/hamradio/dmascc.c renamed from drivers/net/hamradio/dmascc.c.old +0 -10
@@ -311,16 +311,6 @@
 	}
 }
 
-#ifndef MODULE
-void __init dmascc_setup(char *str, int *ints)
-{
-	int i;
-
-	for (i = 0; i < MAX_NUM_DEVS && i < ints[0]; i++)
-		io[i] = ints[i + 1];
-}
-#endif
-
 static int __init dmascc_init(void)
 {
 	int h, i, j, n;
drivers/net/hamradio/mkiss.c renamed from drivers/net/hamradio/mkiss.c.old +1 -1
@@ -65,7 +65,7 @@
 
 static ax25_ctrl_t **ax25_ctrls;
 
-int ax25_maxdev = AX25_MAXDEV;		/* Can be overridden with insmod! */
+static int ax25_maxdev = AX25_MAXDEV;	/* Can be overridden with insmod! */
 
 static struct tty_ldisc	ax_ldisc;
 
#1 lore raw
Quoted 2 lines · Adrian Bunk

This patch contains the following cleanups:
- dmascc.c: remove the unused global function dmascc_setup

Better to use it, then remove it.

Jeff

#2 lore raw
Quoted 5 lines · Jeff Garzik
Quoted 2 lines · Adrian Bunk

This patch contains the following cleanups:
- dmascc.c: remove the unused global function dmascc_setup

Better to use it, then remove it.

Can you give me a hint how it should be used?

Why doesn't dmascc_init together with the MODULE_PARM(io,...) work in the non-modular case?

Quoted 1 line

Jeff

cu
Adrian

Signature · 4 lines
       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed
#3 lore raw
In reply to Jeff Garzik

On Sun, 26 Jun 2005 18:26:49 -0400 Jeff Garzik wrote:

| Adrian Bunk wrote:
| > This patch contains the following cleanups:
| > - dmascc.c: remove the unused global function dmascc_setup
| 
| Better to use it, then remove it.
                    than ??

---
~Randy

#4 lore raw
Quoted 8 lines · randy_dunlap

On Sun, 26 Jun 2005 18:26:49 -0400 Jeff Garzik wrote:

| Adrian Bunk wrote:
| > This patch contains the following cleanups:
| > - dmascc.c: remove the unused global function dmascc_setup
| | Better to use it, then remove it.
                    than ??

Yes. Use it via __setup() or similar.

Jeff

#5 lore raw
Quoted 9 lines · Jeff Garzik
Quoted 8 lines · randy_dunlap

On Sun, 26 Jun 2005 18:26:49 -0400 Jeff Garzik wrote:

| Adrian Bunk wrote:
| > This patch contains the following cleanups:
| > - dmascc.c: remove the unused global function dmascc_setup
| 
| Better to use it, then remove it.
Quoted 3 lines
Quoted 1 line
                   than ??

Yes. Use it via __setup() or similar.

Hi Jeff,

I still haven't gotten any answer from you regarding the following question:

Can you give me a hint how it should be used?

Why doesn't dmascc_init together with the MODULE_PARM(io,...) work in the non-modular case?

Quoted 1 line

Jeff

cu
Adrian

Signature · 4 lines
       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed