From 8a0243aef57e010e09fcbc179fa06097ae55b15d Mon Sep 17 00:00:00 2001 From: patacongo Date: Fri, 25 Mar 2011 15:15:59 +0000 Subject: Add netdev_unregister() git-svn-id: svn://svn.code.sf.net/p/nuttx/code/trunk@3415 42af7a65-404d-4744-a932-0658087f49c3 --- nuttx/include/nuttx/net.h | 5 ++ nuttx/net/Makefile | 2 +- nuttx/net/netdev_register.c | 10 +-- nuttx/net/netdev_unregister.c | 161 ++++++++++++++++++++++++++++++++++++++++++ nuttx/net/uip/uip_igmpinit.c | 1 + 5 files changed, 173 insertions(+), 6 deletions(-) create mode 100644 nuttx/net/netdev_unregister.c (limited to 'nuttx') diff --git a/nuttx/include/nuttx/net.h b/nuttx/include/nuttx/net.h index 2294a38dc..3d96f4e88 100644 --- a/nuttx/include/nuttx/net.h +++ b/nuttx/include/nuttx/net.h @@ -202,6 +202,11 @@ EXTERN int net_vfcntl(int sockfd, int cmd, va_list ap); EXTERN int netdev_register(FAR struct uip_driver_s *dev); +/* netdev-unregister.c *********************************************************/ +/* Unregister a network device driver. */ + +EXTERN int netdev_unregister(FAR struct uip_driver_s *dev); + /* net_foreach.c ************************************************************/ /* Enumerates all registered network devices */ diff --git a/nuttx/net/Makefile b/nuttx/net/Makefile index 40eb2af50..fb6b327dc 100644 --- a/nuttx/net/Makefile +++ b/nuttx/net/Makefile @@ -58,7 +58,7 @@ endif NETDEV_ASRCS = NETDEV_CSRCS = netdev_register.c netdev_ioctl.c net_poll.c netdev_txnotify.c \ netdev_findbyname.c netdev_findbyaddr.c netdev_count.c \ - netdev_foreach.c + netdev_foreach.c netdev_unregister.c include uip/Make.defs endif diff --git a/nuttx/net/netdev_register.c b/nuttx/net/netdev_register.c index fd5656cfc..d9a5edc53 100644 --- a/nuttx/net/netdev_register.c +++ b/nuttx/net/netdev_register.c @@ -104,7 +104,7 @@ void netdev_semtake(void) while (uip_lockedwait(&g_netdev_sem) != 0) { - /* The only case that an error should occr here is if + /* The only case that an error should occur here is if * the wait was awakened by a signal. */ @@ -116,17 +116,17 @@ void netdev_semtake(void) * Function: netdev_register * * Description: - * Register a netword device driver and assign a name to it so tht it can + * Register a network device driver and assign a name to it so tht it can * be found in subsequent network ioctl operations on the device. * * Parameters: * dev - The device driver structure to register * * Returned Value: - * 0:Success; -1 on failure + * 0:Success; negated errno on failure * * Assumptions: - * Called during system initialization from normal user mode + * Called during system initialization from normal user mode * ****************************************************************************/ @@ -140,7 +140,7 @@ int netdev_register(FAR struct uip_driver_s *dev) /* Assign a device name to the interface */ devnum = g_next_devnum++; - snprintf( dev->d_ifname, IFNAMSIZ, NETDEV_FORMAT, devnum ); + snprintf(dev->d_ifname, IFNAMSIZ, NETDEV_FORMAT, devnum ); /* Add the device to the list of known network devices */ diff --git a/nuttx/net/netdev_unregister.c b/nuttx/net/netdev_unregister.c new file mode 100644 index 000000000..76be54b9a --- /dev/null +++ b/nuttx/net/netdev_unregister.c @@ -0,0 +1,161 @@ +/**************************************************************************** + * net/netdev_unregister.c + * + * Copyright (C) 2011 Gregory Nutt. All rights reserved. + * Author: Gregory Nutt + * + * Redistribution and use in source and binary forms, with or without + * modification, are permitted provided that the following conditions + * are met: + * + * 1. Redistributions of source code must retain the above copyright + * notice, this list of conditions and the following disclaimer. + * 2. Redistributions in binary form must reproduce the above copyright + * notice, this list of conditions and the following disclaimer in + * the documentation and/or other materials provided with the + * distribution. + * 3. Neither the name NuttX nor the names of its contributors may be + * used to endorse or promote products derived from this software + * without specific prior written permission. + * + * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS + * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT + * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS + * FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE + * COPYRIGHT OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, + * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, + * BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS + * OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED + * AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT + * LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN + * ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE + * POSSIBILITY OF SUCH DAMAGE. + * + ****************************************************************************/ + +/**************************************************************************** + * Included Files + ****************************************************************************/ + +#include +#if defined(CONFIG_NET) && CONFIG_NSOCKET_DESCRIPTORS > 0 + +#include +#include +#include +#include +#include +#include +#include + +#include +#include +#include + +#include "net_internal.h" + +/**************************************************************************** + * Definitions + ****************************************************************************/ + +#ifdef CONFIG_NET_SLIP +# define NETDEV_FORMAT "sl%d" +#else +# define NETDEV_FORMAT "eth%d" +#endif + +/**************************************************************************** + * Priviate Types + ****************************************************************************/ + +/**************************************************************************** + * Private Data + ****************************************************************************/ + +static int g_next_devnum = 0; + +/**************************************************************************** + * Public Data + ****************************************************************************/ + +/**************************************************************************** + * Private Functions + ****************************************************************************/ + +/**************************************************************************** + * Global Functions + ****************************************************************************/ + +/**************************************************************************** + * Function: netdev_unregister + * + * Description: + * Unregister a network device driver. + * + * Parameters: + * dev - The device driver structure to un-register + * + * Returned Value: + * 0:Success; negated errno on failure + * + * Assumptions: + * Currently only called for USB networking devices when the device is + * physically removed from the slot + * + ****************************************************************************/ + +int netdev_unregister(FAR struct uip_driver_s *dev) +{ + struct uip_driver_s *prev; + struct uip_driver_s *curr; + + if (dev) + { + netdev_semtake(); + + /* Find the device in the list of known network devices */ + + for (prev = NULL, curr = g_netdevices; + curr && curr != dev; + prev = curr, curr = curr->flink); + + /* Remove the device to the list of known network devices */ + + if (curr) + { + /* Where was the entry */ + + if (prev) + { + /* The entry was in the middle or at the end of the list */ + + prev->flink = curr->flink; + } + else + { + /* The entry was at the beginning of the list */ + + g_netdevices = curr; + } + + curr->flink = NULL; + } + + netdev_semgive(); + +#ifdef CONFIG_NET_ETHERNET + nlldbg("Unregistered MAC: %02x:%02x:%02x:%02x:%02x:%02x as dev: %s\n", + dev->d_mac.ether_addr_octet[0], dev->d_mac.ether_addr_octet[1], + dev->d_mac.ether_addr_octet[2], dev->d_mac.ether_addr_octet[3], + dev->d_mac.ether_addr_octet[4], dev->d_mac.ether_addr_octet[5], + dev->d_ifname); +#else + nlldbg("Registered dev: %s\n", dev->d_ifname); +#endif + return OK; + } + + return -EINVAL; +} + +#endif /* CONFIG_NET && CONFIG_NSOCKET_DESCRIPTORS */ diff --git a/nuttx/net/uip/uip_igmpinit.c b/nuttx/net/uip/uip_igmpinit.c index 1bdf34457..b6483b0a7 100755 --- a/nuttx/net/uip/uip_igmpinit.c +++ b/nuttx/net/uip/uip_igmpinit.c @@ -80,6 +80,7 @@ uip_ipaddr_t g_allrouters; * Perform one-time IGMP initialization. * ****************************************************************************/ + void uip_igmpinit(void) { nvdbg("IGMP initializing\n"); -- cgit v1.2.3