diff options
| author | Adrian Bunk <bunk@stusta.de> | 2006-01-13 21:10:22 -0500 |
|---|---|---|
| committer | Jeff Garzik <jgarzik@pobox.com> | 2006-01-17 08:38:12 -0500 |
| commit | f03aa2d89ad600a1ed21a223f196776f217cfe00 (patch) | |
| tree | c99eec8e63fbc952d5b7180b32fe1cbf5accb9ed /drivers/net/arcnet | |
| parent | ff5688ae1cedfb175b5ed0f319d03ad2e5ee005d (diff) | |
[PATCH] drivers/net/arcnet/: possible cleanups
This patch contains the following possible cleanups:
- make needlessly global code static
- arcnet.c: remove the unneeded EXPORT_SYMBOL(arc_proto_null)
- arcnet.c: remove the unneeded EXPORT_SYMBOL(arcnet_dump_packet)
To make Jeff happy, arcnet.c still prints
arcnet: v3.93 BETA 2000/04/29 - by Avery Pennarun et al.
Signed-off-by: Adrian Bunk <bunk@stusta.de>
Signed-off-by: Jeff Garzik <jgarzik@pobox.com>
Diffstat (limited to 'drivers/net/arcnet')
| -rw-r--r-- | drivers/net/arcnet/arc-rawmode.c | 2 | ||||
| -rw-r--r-- | drivers/net/arcnet/arcnet.c | 17 | ||||
| -rw-r--r-- | drivers/net/arcnet/rfc1051.c | 2 | ||||
| -rw-r--r-- | drivers/net/arcnet/rfc1201.c | 2 |
4 files changed, 13 insertions, 10 deletions
diff --git a/drivers/net/arcnet/arc-rawmode.c b/drivers/net/arcnet/arc-rawmode.c index e1ea29b0cd14..e7555d4e6ff1 100644 --- a/drivers/net/arcnet/arc-rawmode.c +++ b/drivers/net/arcnet/arc-rawmode.c | |||
| @@ -42,7 +42,7 @@ static int build_header(struct sk_buff *skb, struct net_device *dev, | |||
| 42 | static int prepare_tx(struct net_device *dev, struct archdr *pkt, int length, | 42 | static int prepare_tx(struct net_device *dev, struct archdr *pkt, int length, |
| 43 | int bufnum); | 43 | int bufnum); |
| 44 | 44 | ||
| 45 | struct ArcProto rawmode_proto = | 45 | static struct ArcProto rawmode_proto = |
| 46 | { | 46 | { |
| 47 | .suffix = 'r', | 47 | .suffix = 'r', |
| 48 | .mtu = XMTU, | 48 | .mtu = XMTU, |
diff --git a/drivers/net/arcnet/arcnet.c b/drivers/net/arcnet/arcnet.c index 409d61d8d083..64e2caf3083d 100644 --- a/drivers/net/arcnet/arcnet.c +++ b/drivers/net/arcnet/arcnet.c | |||
| @@ -62,6 +62,7 @@ static int null_build_header(struct sk_buff *skb, struct net_device *dev, | |||
| 62 | static int null_prepare_tx(struct net_device *dev, struct archdr *pkt, | 62 | static int null_prepare_tx(struct net_device *dev, struct archdr *pkt, |
| 63 | int length, int bufnum); | 63 | int length, int bufnum); |
| 64 | 64 | ||
| 65 | static void arcnet_rx(struct net_device *dev, int bufnum); | ||
| 65 | 66 | ||
| 66 | /* | 67 | /* |
| 67 | * one ArcProto per possible proto ID. None of the elements of | 68 | * one ArcProto per possible proto ID. None of the elements of |
| @@ -72,7 +73,7 @@ static int null_prepare_tx(struct net_device *dev, struct archdr *pkt, | |||
| 72 | struct ArcProto *arc_proto_map[256], *arc_proto_default, | 73 | struct ArcProto *arc_proto_map[256], *arc_proto_default, |
| 73 | *arc_bcast_proto, *arc_raw_proto; | 74 | *arc_bcast_proto, *arc_raw_proto; |
| 74 | 75 | ||
| 75 | struct ArcProto arc_proto_null = | 76 | static struct ArcProto arc_proto_null = |
| 76 | { | 77 | { |
| 77 | .suffix = '?', | 78 | .suffix = '?', |
| 78 | .mtu = XMTU, | 79 | .mtu = XMTU, |
| @@ -91,7 +92,6 @@ EXPORT_SYMBOL(arc_proto_map); | |||
| 91 | EXPORT_SYMBOL(arc_proto_default); | 92 | EXPORT_SYMBOL(arc_proto_default); |
| 92 | EXPORT_SYMBOL(arc_bcast_proto); | 93 | EXPORT_SYMBOL(arc_bcast_proto); |
| 93 | EXPORT_SYMBOL(arc_raw_proto); | 94 | EXPORT_SYMBOL(arc_raw_proto); |
| 94 | EXPORT_SYMBOL(arc_proto_null); | ||
| 95 | EXPORT_SYMBOL(arcnet_unregister_proto); | 95 | EXPORT_SYMBOL(arcnet_unregister_proto); |
| 96 | EXPORT_SYMBOL(arcnet_debug); | 96 | EXPORT_SYMBOL(arcnet_debug); |
| 97 | EXPORT_SYMBOL(alloc_arcdev); | 97 | EXPORT_SYMBOL(alloc_arcdev); |
| @@ -119,7 +119,7 @@ static int __init arcnet_init(void) | |||
| 119 | 119 | ||
| 120 | arcnet_debug = debug; | 120 | arcnet_debug = debug; |
| 121 | 121 | ||
| 122 | printk(VERSION); | 122 | printk("arcnet loaded.\n"); |
| 123 | 123 | ||
| 124 | #ifdef ALPHA_WARNING | 124 | #ifdef ALPHA_WARNING |
| 125 | BUGLVL(D_EXTRA) { | 125 | BUGLVL(D_EXTRA) { |
| @@ -179,8 +179,8 @@ EXPORT_SYMBOL(arcnet_dump_skb); | |||
| 179 | * Dump the contents of an ARCnet buffer | 179 | * Dump the contents of an ARCnet buffer |
| 180 | */ | 180 | */ |
| 181 | #if (ARCNET_DEBUG_MAX & (D_RX | D_TX)) | 181 | #if (ARCNET_DEBUG_MAX & (D_RX | D_TX)) |
| 182 | void arcnet_dump_packet(struct net_device *dev, int bufnum, char *desc, | 182 | static void arcnet_dump_packet(struct net_device *dev, int bufnum, |
| 183 | int take_arcnet_lock) | 183 | char *desc, int take_arcnet_lock) |
| 184 | { | 184 | { |
| 185 | struct arcnet_local *lp = dev->priv; | 185 | struct arcnet_local *lp = dev->priv; |
| 186 | int i, length; | 186 | int i, length; |
| @@ -209,7 +209,10 @@ void arcnet_dump_packet(struct net_device *dev, int bufnum, char *desc, | |||
| 209 | 209 | ||
| 210 | } | 210 | } |
| 211 | 211 | ||
| 212 | EXPORT_SYMBOL(arcnet_dump_packet); | 212 | #else |
| 213 | |||
| 214 | #define arcnet_dump_packet(dev, bufnum, desc,take_arcnet_lock) do { } while (0) | ||
| 215 | |||
| 213 | #endif | 216 | #endif |
| 214 | 217 | ||
| 215 | 218 | ||
| @@ -997,7 +1000,7 @@ irqreturn_t arcnet_interrupt(int irq, void *dev_id, struct pt_regs *regs) | |||
| 997 | * This is a generic packet receiver that calls arcnet??_rx depending on the | 1000 | * This is a generic packet receiver that calls arcnet??_rx depending on the |
| 998 | * protocol ID found. | 1001 | * protocol ID found. |
| 999 | */ | 1002 | */ |
| 1000 | void arcnet_rx(struct net_device *dev, int bufnum) | 1003 | static void arcnet_rx(struct net_device *dev, int bufnum) |
| 1001 | { | 1004 | { |
| 1002 | struct arcnet_local *lp = dev->priv; | 1005 | struct arcnet_local *lp = dev->priv; |
| 1003 | struct archdr pkt; | 1006 | struct archdr pkt; |
diff --git a/drivers/net/arcnet/rfc1051.c b/drivers/net/arcnet/rfc1051.c index 6d7913704fb5..6d6c69f036ef 100644 --- a/drivers/net/arcnet/rfc1051.c +++ b/drivers/net/arcnet/rfc1051.c | |||
| @@ -43,7 +43,7 @@ static int prepare_tx(struct net_device *dev, struct archdr *pkt, int length, | |||
| 43 | int bufnum); | 43 | int bufnum); |
| 44 | 44 | ||
| 45 | 45 | ||
| 46 | struct ArcProto rfc1051_proto = | 46 | static struct ArcProto rfc1051_proto = |
| 47 | { | 47 | { |
| 48 | .suffix = 's', | 48 | .suffix = 's', |
| 49 | .mtu = XMTU - RFC1051_HDR_SIZE, | 49 | .mtu = XMTU - RFC1051_HDR_SIZE, |
diff --git a/drivers/net/arcnet/rfc1201.c b/drivers/net/arcnet/rfc1201.c index 6b6ae4bf3d39..bee34226abfa 100644 --- a/drivers/net/arcnet/rfc1201.c +++ b/drivers/net/arcnet/rfc1201.c | |||
| @@ -43,7 +43,7 @@ static int prepare_tx(struct net_device *dev, struct archdr *pkt, int length, | |||
| 43 | int bufnum); | 43 | int bufnum); |
| 44 | static int continue_tx(struct net_device *dev, int bufnum); | 44 | static int continue_tx(struct net_device *dev, int bufnum); |
| 45 | 45 | ||
| 46 | struct ArcProto rfc1201_proto = | 46 | static struct ArcProto rfc1201_proto = |
| 47 | { | 47 | { |
| 48 | .suffix = 'a', | 48 | .suffix = 'a', |
| 49 | .mtu = 1500, /* could be more, but some receivers can't handle it... */ | 49 | .mtu = 1500, /* could be more, but some receivers can't handle it... */ |
