diff options
author | Krzysztof Hałasa <khc@pm.waw.pl> | 2008-06-30 13:06:40 -0400 |
---|---|---|
committer | Krzysztof Hałasa <khc@pm.waw.pl> | 2008-07-23 17:00:25 -0400 |
commit | 4dfce4075aa4e2eee35e52a78dbabfe37d94c908 (patch) | |
tree | 962f5cde27a9b7eedc45b4c018d07d10b455d590 | |
parent | d6f8aa8586dabe605454f3c98a5c1a577c3e0cfb (diff) |
WAN: cosmetic changes to generic HDLC
Signed-off-by: Krzysztof Hałasa <khc@pm.waw.pl>
-rw-r--r-- | drivers/net/wan/hdlc.c | 25 | ||||
-rw-r--r-- | drivers/net/wan/hdlc_cisco.c | 29 | ||||
-rw-r--r-- | drivers/net/wan/hdlc_fr.c | 19 | ||||
-rw-r--r-- | drivers/net/wan/hdlc_ppp.c | 15 | ||||
-rw-r--r-- | drivers/net/wan/hdlc_raw.c | 15 | ||||
-rw-r--r-- | drivers/net/wan/hdlc_raw_eth.c | 17 | ||||
-rw-r--r-- | drivers/net/wan/hdlc_x25.c | 17 |
7 files changed, 65 insertions, 72 deletions
diff --git a/drivers/net/wan/hdlc.c b/drivers/net/wan/hdlc.c index e3a536477c7e..1f2a140c9f7c 100644 --- a/drivers/net/wan/hdlc.c +++ b/drivers/net/wan/hdlc.c | |||
@@ -22,20 +22,19 @@ | |||
22 | * - proto->start() and stop() are called with spin_lock_irq held. | 22 | * - proto->start() and stop() are called with spin_lock_irq held. |
23 | */ | 23 | */ |
24 | 24 | ||
25 | #include <linux/module.h> | ||
26 | #include <linux/kernel.h> | ||
27 | #include <linux/slab.h> | ||
28 | #include <linux/poll.h> | ||
29 | #include <linux/errno.h> | 25 | #include <linux/errno.h> |
26 | #include <linux/hdlc.h> | ||
30 | #include <linux/if_arp.h> | 27 | #include <linux/if_arp.h> |
28 | #include <linux/inetdevice.h> | ||
31 | #include <linux/init.h> | 29 | #include <linux/init.h> |
32 | #include <linux/skbuff.h> | 30 | #include <linux/kernel.h> |
31 | #include <linux/module.h> | ||
32 | #include <linux/notifier.h> | ||
33 | #include <linux/pkt_sched.h> | 33 | #include <linux/pkt_sched.h> |
34 | #include <linux/inetdevice.h> | 34 | #include <linux/poll.h> |
35 | #include <linux/lapb.h> | ||
36 | #include <linux/rtnetlink.h> | 35 | #include <linux/rtnetlink.h> |
37 | #include <linux/notifier.h> | 36 | #include <linux/skbuff.h> |
38 | #include <linux/hdlc.h> | 37 | #include <linux/slab.h> |
39 | #include <net/net_namespace.h> | 38 | #include <net/net_namespace.h> |
40 | 39 | ||
41 | 40 | ||
@@ -109,7 +108,7 @@ static int hdlc_device_event(struct notifier_block *this, unsigned long event, | |||
109 | 108 | ||
110 | if (dev->get_stats != hdlc_get_stats) | 109 | if (dev->get_stats != hdlc_get_stats) |
111 | return NOTIFY_DONE; /* not an HDLC device */ | 110 | return NOTIFY_DONE; /* not an HDLC device */ |
112 | 111 | ||
113 | if (event != NETDEV_CHANGE) | 112 | if (event != NETDEV_CHANGE) |
114 | return NOTIFY_DONE; /* Only interrested in carrier changes */ | 113 | return NOTIFY_DONE; /* Only interrested in carrier changes */ |
115 | 114 | ||
@@ -357,7 +356,7 @@ static struct packet_type hdlc_packet_type = { | |||
357 | 356 | ||
358 | 357 | ||
359 | static struct notifier_block hdlc_notifier = { | 358 | static struct notifier_block hdlc_notifier = { |
360 | .notifier_call = hdlc_device_event, | 359 | .notifier_call = hdlc_device_event, |
361 | }; | 360 | }; |
362 | 361 | ||
363 | 362 | ||
@@ -367,8 +366,8 @@ static int __init hdlc_module_init(void) | |||
367 | 366 | ||
368 | printk(KERN_INFO "%s\n", version); | 367 | printk(KERN_INFO "%s\n", version); |
369 | if ((result = register_netdevice_notifier(&hdlc_notifier)) != 0) | 368 | if ((result = register_netdevice_notifier(&hdlc_notifier)) != 0) |
370 | return result; | 369 | return result; |
371 | dev_add_pack(&hdlc_packet_type); | 370 | dev_add_pack(&hdlc_packet_type); |
372 | return 0; | 371 | return 0; |
373 | } | 372 | } |
374 | 373 | ||
diff --git a/drivers/net/wan/hdlc_cisco.c b/drivers/net/wan/hdlc_cisco.c index 849819c2552d..44e64b15dbd1 100644 --- a/drivers/net/wan/hdlc_cisco.c +++ b/drivers/net/wan/hdlc_cisco.c | |||
@@ -9,19 +9,18 @@ | |||
9 | * as published by the Free Software Foundation. | 9 | * as published by the Free Software Foundation. |
10 | */ | 10 | */ |
11 | 11 | ||
12 | #include <linux/module.h> | ||
13 | #include <linux/kernel.h> | ||
14 | #include <linux/slab.h> | ||
15 | #include <linux/poll.h> | ||
16 | #include <linux/errno.h> | 12 | #include <linux/errno.h> |
13 | #include <linux/hdlc.h> | ||
17 | #include <linux/if_arp.h> | 14 | #include <linux/if_arp.h> |
15 | #include <linux/inetdevice.h> | ||
18 | #include <linux/init.h> | 16 | #include <linux/init.h> |
19 | #include <linux/skbuff.h> | 17 | #include <linux/kernel.h> |
18 | #include <linux/module.h> | ||
20 | #include <linux/pkt_sched.h> | 19 | #include <linux/pkt_sched.h> |
21 | #include <linux/inetdevice.h> | 20 | #include <linux/poll.h> |
22 | #include <linux/lapb.h> | ||
23 | #include <linux/rtnetlink.h> | 21 | #include <linux/rtnetlink.h> |
24 | #include <linux/hdlc.h> | 22 | #include <linux/skbuff.h> |
23 | #include <linux/slab.h> | ||
25 | 24 | ||
26 | #undef DEBUG_HARD_HEADER | 25 | #undef DEBUG_HARD_HEADER |
27 | 26 | ||
@@ -68,9 +67,9 @@ struct cisco_state { | |||
68 | static int cisco_ioctl(struct net_device *dev, struct ifreq *ifr); | 67 | static int cisco_ioctl(struct net_device *dev, struct ifreq *ifr); |
69 | 68 | ||
70 | 69 | ||
71 | static inline struct cisco_state * state(hdlc_device *hdlc) | 70 | static inline struct cisco_state* state(hdlc_device *hdlc) |
72 | { | 71 | { |
73 | return(struct cisco_state *)(hdlc->state); | 72 | return (struct cisco_state *)hdlc->state; |
74 | } | 73 | } |
75 | 74 | ||
76 | 75 | ||
@@ -172,7 +171,7 @@ static int cisco_rx(struct sk_buff *skb) | |||
172 | data->address != CISCO_UNICAST) | 171 | data->address != CISCO_UNICAST) |
173 | goto rx_error; | 172 | goto rx_error; |
174 | 173 | ||
175 | switch(ntohs(data->protocol)) { | 174 | switch (ntohs(data->protocol)) { |
176 | case CISCO_SYS_INFO: | 175 | case CISCO_SYS_INFO: |
177 | /* Packet is not needed, drop it. */ | 176 | /* Packet is not needed, drop it. */ |
178 | dev_kfree_skb_any(skb); | 177 | dev_kfree_skb_any(skb); |
@@ -336,7 +335,7 @@ static struct hdlc_proto proto = { | |||
336 | static const struct header_ops cisco_header_ops = { | 335 | static const struct header_ops cisco_header_ops = { |
337 | .create = cisco_hard_header, | 336 | .create = cisco_hard_header, |
338 | }; | 337 | }; |
339 | 338 | ||
340 | static int cisco_ioctl(struct net_device *dev, struct ifreq *ifr) | 339 | static int cisco_ioctl(struct net_device *dev, struct ifreq *ifr) |
341 | { | 340 | { |
342 | cisco_proto __user *cisco_s = ifr->ifr_settings.ifs_ifsu.cisco; | 341 | cisco_proto __user *cisco_s = ifr->ifr_settings.ifs_ifsu.cisco; |
@@ -359,10 +358,10 @@ static int cisco_ioctl(struct net_device *dev, struct ifreq *ifr) | |||
359 | return 0; | 358 | return 0; |
360 | 359 | ||
361 | case IF_PROTO_CISCO: | 360 | case IF_PROTO_CISCO: |
362 | if(!capable(CAP_NET_ADMIN)) | 361 | if (!capable(CAP_NET_ADMIN)) |
363 | return -EPERM; | 362 | return -EPERM; |
364 | 363 | ||
365 | if(dev->flags & IFF_UP) | 364 | if (dev->flags & IFF_UP) |
366 | return -EBUSY; | 365 | return -EBUSY; |
367 | 366 | ||
368 | if (copy_from_user(&new_settings, cisco_s, size)) | 367 | if (copy_from_user(&new_settings, cisco_s, size)) |
@@ -372,7 +371,7 @@ static int cisco_ioctl(struct net_device *dev, struct ifreq *ifr) | |||
372 | new_settings.timeout < 2) | 371 | new_settings.timeout < 2) |
373 | return -EINVAL; | 372 | return -EINVAL; |
374 | 373 | ||
375 | result=hdlc->attach(dev, ENCODING_NRZ,PARITY_CRC16_PR1_CCITT); | 374 | result = hdlc->attach(dev, ENCODING_NRZ,PARITY_CRC16_PR1_CCITT); |
376 | if (result) | 375 | if (result) |
377 | return result; | 376 | return result; |
378 | 377 | ||
diff --git a/drivers/net/wan/hdlc_fr.c b/drivers/net/wan/hdlc_fr.c index 62e93dac6b13..d3d5055741ad 100644 --- a/drivers/net/wan/hdlc_fr.c +++ b/drivers/net/wan/hdlc_fr.c | |||
@@ -33,20 +33,19 @@ | |||
33 | 33 | ||
34 | */ | 34 | */ |
35 | 35 | ||
36 | #include <linux/module.h> | ||
37 | #include <linux/kernel.h> | ||
38 | #include <linux/slab.h> | ||
39 | #include <linux/poll.h> | ||
40 | #include <linux/errno.h> | 36 | #include <linux/errno.h> |
37 | #include <linux/etherdevice.h> | ||
38 | #include <linux/hdlc.h> | ||
41 | #include <linux/if_arp.h> | 39 | #include <linux/if_arp.h> |
40 | #include <linux/inetdevice.h> | ||
42 | #include <linux/init.h> | 41 | #include <linux/init.h> |
43 | #include <linux/skbuff.h> | 42 | #include <linux/kernel.h> |
43 | #include <linux/module.h> | ||
44 | #include <linux/pkt_sched.h> | 44 | #include <linux/pkt_sched.h> |
45 | #include <linux/inetdevice.h> | 45 | #include <linux/poll.h> |
46 | #include <linux/lapb.h> | ||
47 | #include <linux/rtnetlink.h> | 46 | #include <linux/rtnetlink.h> |
48 | #include <linux/etherdevice.h> | 47 | #include <linux/skbuff.h> |
49 | #include <linux/hdlc.h> | 48 | #include <linux/slab.h> |
50 | 49 | ||
51 | #undef DEBUG_PKT | 50 | #undef DEBUG_PKT |
52 | #undef DEBUG_ECN | 51 | #undef DEBUG_ECN |
@@ -96,7 +95,7 @@ typedef struct { | |||
96 | unsigned ea1: 1; | 95 | unsigned ea1: 1; |
97 | unsigned cr: 1; | 96 | unsigned cr: 1; |
98 | unsigned dlcih: 6; | 97 | unsigned dlcih: 6; |
99 | 98 | ||
100 | unsigned ea2: 1; | 99 | unsigned ea2: 1; |
101 | unsigned de: 1; | 100 | unsigned de: 1; |
102 | unsigned becn: 1; | 101 | unsigned becn: 1; |
diff --git a/drivers/net/wan/hdlc_ppp.c b/drivers/net/wan/hdlc_ppp.c index 00308337928e..4efe9e6d32d5 100644 --- a/drivers/net/wan/hdlc_ppp.c +++ b/drivers/net/wan/hdlc_ppp.c | |||
@@ -9,19 +9,18 @@ | |||
9 | * as published by the Free Software Foundation. | 9 | * as published by the Free Software Foundation. |
10 | */ | 10 | */ |
11 | 11 | ||
12 | #include <linux/module.h> | ||
13 | #include <linux/kernel.h> | ||
14 | #include <linux/slab.h> | ||
15 | #include <linux/poll.h> | ||
16 | #include <linux/errno.h> | 12 | #include <linux/errno.h> |
13 | #include <linux/hdlc.h> | ||
17 | #include <linux/if_arp.h> | 14 | #include <linux/if_arp.h> |
15 | #include <linux/inetdevice.h> | ||
18 | #include <linux/init.h> | 16 | #include <linux/init.h> |
19 | #include <linux/skbuff.h> | 17 | #include <linux/kernel.h> |
18 | #include <linux/module.h> | ||
20 | #include <linux/pkt_sched.h> | 19 | #include <linux/pkt_sched.h> |
21 | #include <linux/inetdevice.h> | 20 | #include <linux/poll.h> |
22 | #include <linux/lapb.h> | ||
23 | #include <linux/rtnetlink.h> | 21 | #include <linux/rtnetlink.h> |
24 | #include <linux/hdlc.h> | 22 | #include <linux/skbuff.h> |
23 | #include <linux/slab.h> | ||
25 | #include <net/syncppp.h> | 24 | #include <net/syncppp.h> |
26 | 25 | ||
27 | struct ppp_state { | 26 | struct ppp_state { |
diff --git a/drivers/net/wan/hdlc_raw.c b/drivers/net/wan/hdlc_raw.c index bbbb819d764c..8612311748f4 100644 --- a/drivers/net/wan/hdlc_raw.c +++ b/drivers/net/wan/hdlc_raw.c | |||
@@ -9,19 +9,18 @@ | |||
9 | * as published by the Free Software Foundation. | 9 | * as published by the Free Software Foundation. |
10 | */ | 10 | */ |
11 | 11 | ||
12 | #include <linux/module.h> | ||
13 | #include <linux/kernel.h> | ||
14 | #include <linux/slab.h> | ||
15 | #include <linux/poll.h> | ||
16 | #include <linux/errno.h> | 12 | #include <linux/errno.h> |
13 | #include <linux/hdlc.h> | ||
17 | #include <linux/if_arp.h> | 14 | #include <linux/if_arp.h> |
15 | #include <linux/inetdevice.h> | ||
18 | #include <linux/init.h> | 16 | #include <linux/init.h> |
19 | #include <linux/skbuff.h> | 17 | #include <linux/kernel.h> |
18 | #include <linux/module.h> | ||
20 | #include <linux/pkt_sched.h> | 19 | #include <linux/pkt_sched.h> |
21 | #include <linux/inetdevice.h> | 20 | #include <linux/poll.h> |
22 | #include <linux/lapb.h> | ||
23 | #include <linux/rtnetlink.h> | 21 | #include <linux/rtnetlink.h> |
24 | #include <linux/hdlc.h> | 22 | #include <linux/skbuff.h> |
23 | #include <linux/slab.h> | ||
25 | 24 | ||
26 | 25 | ||
27 | static int raw_ioctl(struct net_device *dev, struct ifreq *ifr); | 26 | static int raw_ioctl(struct net_device *dev, struct ifreq *ifr); |
diff --git a/drivers/net/wan/hdlc_raw_eth.c b/drivers/net/wan/hdlc_raw_eth.c index 26dee600506f..a13fc3207520 100644 --- a/drivers/net/wan/hdlc_raw_eth.c +++ b/drivers/net/wan/hdlc_raw_eth.c | |||
@@ -9,20 +9,19 @@ | |||
9 | * as published by the Free Software Foundation. | 9 | * as published by the Free Software Foundation. |
10 | */ | 10 | */ |
11 | 11 | ||
12 | #include <linux/module.h> | ||
13 | #include <linux/kernel.h> | ||
14 | #include <linux/slab.h> | ||
15 | #include <linux/poll.h> | ||
16 | #include <linux/errno.h> | 12 | #include <linux/errno.h> |
13 | #include <linux/etherdevice.h> | ||
14 | #include <linux/hdlc.h> | ||
17 | #include <linux/if_arp.h> | 15 | #include <linux/if_arp.h> |
16 | #include <linux/inetdevice.h> | ||
18 | #include <linux/init.h> | 17 | #include <linux/init.h> |
19 | #include <linux/skbuff.h> | 18 | #include <linux/kernel.h> |
19 | #include <linux/module.h> | ||
20 | #include <linux/pkt_sched.h> | 20 | #include <linux/pkt_sched.h> |
21 | #include <linux/inetdevice.h> | 21 | #include <linux/poll.h> |
22 | #include <linux/lapb.h> | ||
23 | #include <linux/rtnetlink.h> | 22 | #include <linux/rtnetlink.h> |
24 | #include <linux/etherdevice.h> | 23 | #include <linux/skbuff.h> |
25 | #include <linux/hdlc.h> | 24 | #include <linux/slab.h> |
26 | 25 | ||
27 | static int raw_eth_ioctl(struct net_device *dev, struct ifreq *ifr); | 26 | static int raw_eth_ioctl(struct net_device *dev, struct ifreq *ifr); |
28 | 27 | ||
diff --git a/drivers/net/wan/hdlc_x25.c b/drivers/net/wan/hdlc_x25.c index e808720030ef..8b7e5d2e2ac9 100644 --- a/drivers/net/wan/hdlc_x25.c +++ b/drivers/net/wan/hdlc_x25.c | |||
@@ -9,20 +9,19 @@ | |||
9 | * as published by the Free Software Foundation. | 9 | * as published by the Free Software Foundation. |
10 | */ | 10 | */ |
11 | 11 | ||
12 | #include <linux/module.h> | ||
13 | #include <linux/kernel.h> | ||
14 | #include <linux/slab.h> | ||
15 | #include <linux/poll.h> | ||
16 | #include <linux/errno.h> | 12 | #include <linux/errno.h> |
13 | #include <linux/hdlc.h> | ||
17 | #include <linux/if_arp.h> | 14 | #include <linux/if_arp.h> |
18 | #include <linux/init.h> | ||
19 | #include <linux/skbuff.h> | ||
20 | #include <linux/pkt_sched.h> | ||
21 | #include <linux/inetdevice.h> | 15 | #include <linux/inetdevice.h> |
16 | #include <linux/init.h> | ||
17 | #include <linux/kernel.h> | ||
22 | #include <linux/lapb.h> | 18 | #include <linux/lapb.h> |
19 | #include <linux/module.h> | ||
20 | #include <linux/pkt_sched.h> | ||
21 | #include <linux/poll.h> | ||
23 | #include <linux/rtnetlink.h> | 22 | #include <linux/rtnetlink.h> |
24 | #include <linux/hdlc.h> | 23 | #include <linux/skbuff.h> |
25 | 24 | #include <linux/slab.h> | |
26 | #include <net/x25device.h> | 25 | #include <net/x25device.h> |
27 | 26 | ||
28 | static int x25_ioctl(struct net_device *dev, struct ifreq *ifr); | 27 | static int x25_ioctl(struct net_device *dev, struct ifreq *ifr); |