Update TODO list and comments

3 files changed, 34 insertions, 44 deletions

diff --git a/nuttx/TODO b/nuttx/TODO
index 29cea8ef6..12af21d8e 100644
--- a/nuttx/TODO
+++ b/nuttx/TODO
@@ -1,4 +1,4 @@
-NuttX TODO List (Last updated January 10, 2014)
+NuttX TODO List (Last updated April 23, 2014)
 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
 
 This file summarizes known NuttX bugs, limitations, inconsistencies with
@@ -15,7 +15,7 @@ nuttx/
  (11)  Kernel Build
   (4)  C++ Support
   (6)  Binary loaders (binfmt/)
- (17)  Network (net/, drivers/net)
+ (16)  Network (net/, drivers/net)
   (4)  USB (drivers/usbdev, drivers/usbhost)
  (10)  Libraries (libc/, )
  (12)  File system/Generic drivers (fs/, drivers/)
@@ -48,8 +48,8 @@ nuttx/
 
 apps/
 
-  (5)  Network Utilities (apps/netutils/)
-  (4)  NuttShell (NSH) (apps/nshlib)
+  (4)  Network Utilities (apps/netutils/)
+  (3)  NuttShell (NSH) (apps/nshlib)
   (1)  System libraries apps/system (apps/system)
   (5)  Other Applications & Tests (apps/examples/)
 
@@ -900,17 +900,6 @@ o Network (net/, drivers/net)
   Status:      Open
   Priority:    Low unless you need it.
 
-  Title:       UNTESTED IGMPv2
-  Description: Support for client-side IGMPv2 multicast has been added but not yet
-               tested (because I don't have a proper environment for multicast testing).
-               There are most likely errors that need to be fixed at least in the
-               receipt of multicast packets.
-
-               In addition, an Ethernet driver that needs to work with the IGMP logic
-               will have to include additional support for multicast MAC address tables.
-  Status:      Open
-  Priority:    Low unless you need it.
-
   Title:       INTERFACES TO LEAVE/JOIN IGMP MULTICAST GROUP
   Description: The interfaces used to leave/join IGMP multicast groups is non-standard.
                RFC3678 (IGMPv3) suggests ioctl() commands to do this (SIOCSIPMSFILTER) but
@@ -927,18 +916,14 @@ o Network (net/, drivers/net)
                the mechanism for leaving and joining groups is hidden behind a wrapper
                function so that little of this incompatibilities need be exposed.
 
-  Title:       CONFIGURATIONS WITH TINY MTUS
-  Description: Many configurations have the MTU (CONFIG_NET_BUFSIZE) set to very small
-               numbers, less then the minimum MTU size that must be supported -- 576.
-               This can cause problems in some networks:  CONFIG_NET_BUFSIZE should
-               be set to at least 576 in all defconfig files.
-
-               The symptoms of using very small MTU sizes can be very strange.  With
-               Ubuntu 9.x and vsFtpd was that the total packet size did *not match* the
-               packet size in the IP header.  This then caused a TCP checksum failure
-               and the packet was rejected.
+  Title:       CLOSED CONNECTIONS IN THE BACKLOG
+               If a connection is backlogged but accept() is not called quickly, then
+               that connection may time out.  How should this be handled?  Should the
+               connection be removed from the backlog if it is times out or is closed?
+               Or should it remain in the backlog with a status indication so that accept()
+               can fail when it encounteres the invalid connection?
   Status:      Open
-  Priority:    Low... fix defconfig files as necessary.
+  Priority:    Medium.  Important on slow applications that will not accept connections promptly.
 
 o USB (drivers/usbdev, drivers/usbhost)
   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
@@ -2309,14 +2294,6 @@ o mc68hc1x (arch/hc)
 
 o Network Utilities (apps/netutils/)
 
-  Title:       UIP RESOLVER
-  Description: One critical part of netutils/ apps is untested: The uIP
-               resolver in netutils/resolv.  The webclient code has been
-               tested on host using gethosbyname(), but still depends on the
-               untested resolve logic.
-  Status:      Open
-  Priority:    Medium, Important but not core NuttX functionality
-
   Title:       PPP PORT
   Description: Port PPP support from http://contiki.cvs.sourceforge.net/contiki/contiki-2.x/backyard/core/net/ppp/
   Status:      Open
@@ -2364,11 +2341,6 @@ o NuttShell (NSH) (apps/nshlib)
   Status:      Open
   Priority:    Low (multiple network interfaces not fully supported yet anyway).
 
-  Title:       RUN NXFLAT PROGRAMS
-  Description: Add support to NSH to run NXFLAT programs from a ROMFS file system
-  Status:      Open
-  Priority:    Low (enhancement)
-
   Title:       ARP COMMAND
   Description: Add an ARP command so that we can see the contents of the ARP table.
   Status:      Open
diff --git a/nuttx/drivers/mtd/mtd_partition.c b/nuttx/drivers/mtd/mtd_partition.c
index 22041361c..1196ea355 100644
--- a/nuttx/drivers/mtd/mtd_partition.c
+++ b/nuttx/drivers/mtd/mtd_partition.c
@@ -1,7 +1,7 @@
 /****************************************************************************
  * drivers/mtd/mtd_partition.c
  *
- *   Copyright (C) 2013 Gregory Nutt. All rights reserved.
+ *   Copyright (C) 2013-2014 Gregory Nutt. All rights reserved.
  *   Author: Gregory Nutt <gnutt@nuttx.org>
  *
  * Redistribution and use in source and binary forms, with or without
@@ -757,7 +757,7 @@ static int part_procfs_stat(const char *relpath, struct stat *buf)
  * Description:
  *   Give an instance of an MTD driver, create a flash partition, ie.,
  *   another MTD driver instance that only operates with a sub-region of
- *   FLASH media.  That sub-region is defined by a sector offsetset and a
+ *   FLASH media.  That sub-region is defined by a sector offset and a
  *   sector count (where the size of a sector is provided the by parent MTD
  *   driver).
  *
@@ -767,6 +767,15 @@ static int part_procfs_stat(const char *relpath, struct stat *buf)
  *   partitions, that mutual exclusion would be provided by the file system
  *   above the FLASH driver.
  *
+ * Input parameters:
+ *   mtd        - The MTD device to be partitioned
+ *   firstblock - The offset in bytes to the first block
+ *   nblocks    - The number of blocks in the partition
+ *
+ * Returned Value:
+ *   On success, another MTD device representing the partition is returned.
+ *   A NULL value is returned on a failure.
+ *
  ****************************************************************************/
 
 FAR struct mtd_dev_s *mtd_partition(FAR struct mtd_dev_s *mtd, off_t firstblock,
@@ -800,7 +809,7 @@ FAR struct mtd_dev_s *mtd_partition(FAR struct mtd_dev_s *mtd, off_t firstblock,
 
   /* Adjust the offset and size if necessary so that they are multiples of
    * the erase block size (making sure that we do not go outside of the
-   * requested sub-region).  NOTE that eraseend is the first erase block
+   * requested sub-region).  NOTE that 'eraseend' is the first erase block
    * beyond the sub-region.
    */
 
@@ -855,7 +864,7 @@ FAR struct mtd_dev_s *mtd_partition(FAR struct mtd_dev_s *mtd, off_t firstblock,
 #endif
 
 #if defined(CONFIG_FS_PROCFS) && !defined(CONFIG_PROCFS_EXCLUDE_PARTITIONS)
-  /* Add this parition to the list of known partitions */
+  /* Add this partition to the list of known partitions */
 
   if (g_pfirstpartition == NULL)
     {
diff --git a/nuttx/include/nuttx/mtd/mtd.h b/nuttx/include/nuttx/mtd/mtd.h
index ad1842074..a3ac52ff0 100644
--- a/nuttx/include/nuttx/mtd/mtd.h
+++ b/nuttx/include/nuttx/mtd/mtd.h
@@ -188,7 +188,7 @@ extern "C"
  * Description:
  *   Given an instance of an MTD driver, create a flash partition, ie.,
  *   another MTD driver instance that only operates with a sub-region of
- *   FLASH media.  That sub-region is defined by a sector offsetset and a
+ *   FLASH media.  That sub-region is defined by a sector offset and a
  *   sector count (where the size of a sector is provided the by parent MTD
  *   driver).
  *
@@ -198,6 +198,15 @@ extern "C"
  *   partitions, that mutual exclusion would be provided by the file system
  *   above the FLASH driver.
  *
+ * Input parameters:
+ *   mtd        - The MTD device to be partitioned
+ *   firstblock - The offset in bytes to the first block
+ *   nblocks    - The number of blocks in the partition
+ *
+ * Returned Value:
+ *   On success, another MTD device representing the partition is returned.
+ *   A NULL value is returned on a failure.
+ *
  ****************************************************************************/
 
 FAR struct mtd_dev_s *mtd_partition(FAR struct mtd_dev_s *mtd,