diff options
author | Jarod Wilson <jarod@redhat.com> | 2016-10-20 13:55:19 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2016-10-20 14:51:09 -0400 |
commit | 8b6b4135e4fb2b537f33b811c13f77bee25ca8d3 (patch) | |
tree | 7dc0a6266b83daab36cbf664bd441d92af5a599f /drivers/net/wan/hdlc.c | |
parent | 9c22b4a34eddbaa5b5243c8cd27e31aa36e676e1 (diff) |
net: use core MTU range checking in WAN drivers
- set min/max_mtu in all hdlc drivers, remove hdlc_change_mtu
- sent max_mtu in lec driver, remove lec_change_mtu
- set min/max_mtu in x25_asy driver
CC: netdev@vger.kernel.org
CC: Krzysztof Halasa <khc@pm.waw.pl>
CC: Krzysztof Halasa <khalasa@piap.pl>
CC: Jan "Yenya" Kasprzak <kas@fi.muni.cz>
CC: Francois Romieu <romieu@fr.zoreil.com>
CC: Kevin Curtis <kevin.curtis@farsite.co.uk>
CC: Zhao Qiang <qiang.zhao@nxp.com>
Signed-off-by: Jarod Wilson <jarod@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/wan/hdlc.c')
-rw-r--r-- | drivers/net/wan/hdlc.c | 11 |
1 files changed, 2 insertions, 9 deletions
diff --git a/drivers/net/wan/hdlc.c b/drivers/net/wan/hdlc.c index 9bd4aa8083ce..7221a53b8b14 100644 --- a/drivers/net/wan/hdlc.c +++ b/drivers/net/wan/hdlc.c | |||
@@ -46,14 +46,6 @@ static const char* version = "HDLC support module revision 1.22"; | |||
46 | 46 | ||
47 | static struct hdlc_proto *first_proto; | 47 | static struct hdlc_proto *first_proto; |
48 | 48 | ||
49 | int hdlc_change_mtu(struct net_device *dev, int new_mtu) | ||
50 | { | ||
51 | if ((new_mtu < 68) || (new_mtu > HDLC_MAX_MTU)) | ||
52 | return -EINVAL; | ||
53 | dev->mtu = new_mtu; | ||
54 | return 0; | ||
55 | } | ||
56 | |||
57 | static int hdlc_rcv(struct sk_buff *skb, struct net_device *dev, | 49 | static int hdlc_rcv(struct sk_buff *skb, struct net_device *dev, |
58 | struct packet_type *p, struct net_device *orig_dev) | 50 | struct packet_type *p, struct net_device *orig_dev) |
59 | { | 51 | { |
@@ -237,6 +229,8 @@ static void hdlc_setup_dev(struct net_device *dev) | |||
237 | dev->flags = IFF_POINTOPOINT | IFF_NOARP; | 229 | dev->flags = IFF_POINTOPOINT | IFF_NOARP; |
238 | dev->priv_flags = IFF_WAN_HDLC; | 230 | dev->priv_flags = IFF_WAN_HDLC; |
239 | dev->mtu = HDLC_MAX_MTU; | 231 | dev->mtu = HDLC_MAX_MTU; |
232 | dev->min_mtu = 68; | ||
233 | dev->max_mtu = HDLC_MAX_MTU; | ||
240 | dev->type = ARPHRD_RAWHDLC; | 234 | dev->type = ARPHRD_RAWHDLC; |
241 | dev->hard_header_len = 16; | 235 | dev->hard_header_len = 16; |
242 | dev->addr_len = 0; | 236 | dev->addr_len = 0; |
@@ -353,7 +347,6 @@ MODULE_AUTHOR("Krzysztof Halasa <khc@pm.waw.pl>"); | |||
353 | MODULE_DESCRIPTION("HDLC support module"); | 347 | MODULE_DESCRIPTION("HDLC support module"); |
354 | MODULE_LICENSE("GPL v2"); | 348 | MODULE_LICENSE("GPL v2"); |
355 | 349 | ||
356 | EXPORT_SYMBOL(hdlc_change_mtu); | ||
357 | EXPORT_SYMBOL(hdlc_start_xmit); | 350 | EXPORT_SYMBOL(hdlc_start_xmit); |
358 | EXPORT_SYMBOL(hdlc_open); | 351 | EXPORT_SYMBOL(hdlc_open); |
359 | EXPORT_SYMBOL(hdlc_close); | 352 | EXPORT_SYMBOL(hdlc_close); |