Skip to content

Commit

Permalink
[PATCH] WAN: register_hdlc_device() doesn't need dev_alloc_name()
Browse files Browse the repository at this point in the history
David Boggs noticed that register_hdlc_device() no longer needs
to call dev_alloc_name() as it's called by register_netdev().
register_hdlc_device() is currently equivalent to register_netdev().

hdlc_setup() is now EXPORTed as per David's request.

Signed-off-by: Krzysztof Halasa <khc@pm.waw.pl>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
  • Loading branch information
Krzysztof Halasa authored and Jeff Garzik committed Jun 23, 2006
1 parent aa95abe commit 4a31e34
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 23 deletions.
24 changes: 2 additions & 22 deletions drivers/net/wan/hdlc_generic.c
Original file line number Diff line number Diff line change
Expand Up @@ -259,7 +259,7 @@ int hdlc_ioctl(struct net_device *dev, struct ifreq *ifr, int cmd)
}
}

static void hdlc_setup(struct net_device *dev)
void hdlc_setup(struct net_device *dev)
{
hdlc_device *hdlc = dev_to_hdlc(dev);

Expand Down Expand Up @@ -288,26 +288,6 @@ struct net_device *alloc_hdlcdev(void *priv)
return dev;
}

int register_hdlc_device(struct net_device *dev)
{
int result = dev_alloc_name(dev, "hdlc%d");
if (result < 0)
return result;

result = register_netdev(dev);
if (result != 0)
return -EIO;

#if 0
if (netif_carrier_ok(dev))
netif_carrier_off(dev); /* no carrier until DCD goes up */
#endif

return 0;
}



void unregister_hdlc_device(struct net_device *dev)
{
rtnl_lock();
Expand All @@ -326,8 +306,8 @@ EXPORT_SYMBOL(hdlc_open);
EXPORT_SYMBOL(hdlc_close);
EXPORT_SYMBOL(hdlc_set_carrier);
EXPORT_SYMBOL(hdlc_ioctl);
EXPORT_SYMBOL(hdlc_setup);
EXPORT_SYMBOL(alloc_hdlcdev);
EXPORT_SYMBOL(register_hdlc_device);
EXPORT_SYMBOL(unregister_hdlc_device);

static struct packet_type hdlc_packet_type = {
Expand Down
2 changes: 1 addition & 1 deletion include/linux/hdlc.h
Original file line number Diff line number Diff line change
Expand Up @@ -188,7 +188,7 @@ int hdlc_x25_ioctl(struct net_device *dev, struct ifreq *ifr);
int hdlc_ioctl(struct net_device *dev, struct ifreq *ifr, int cmd);

/* Must be used by hardware driver on module startup/exit */
int register_hdlc_device(struct net_device *dev);
#define register_hdlc_device(dev) register_netdev(dev)
void unregister_hdlc_device(struct net_device *dev);

struct net_device *alloc_hdlcdev(void *priv);
Expand Down

0 comments on commit 4a31e34

Please sign in to comment.