drivers/net/hamradio/: cleanups
Subsystems (2)
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;
Quoted 2 lines · Adrian Bunk
This patch contains the following cleanups:
-dmascc.c: remove the unused global functiondmascc_setup
Better to use it, then remove it.
Jeff
Quoted 5 lines · Jeff Garzik
Quoted 2 lines · Adrian Bunk
This patch contains the following cleanups:
-dmascc.c: remove the unused global functiondmascc_setupBetter 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 SeedOn 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
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
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