Update TODO list and add REVISIT comment

1 files changed, 6 insertions, 1 deletions

diff --git a/nuttx/drivers/serial/serial.c b/nuttx/drivers/serial/serial.c
index 3ae4bd2e4..4d40527fe 100644
--- a/nuttx/drivers/serial/serial.c
+++ b/nuttx/drivers/serial/serial.c
@@ -1121,7 +1121,12 @@ static int uart_close(FAR struct file *filep)
 
   /* We need to re-initialize the semaphores if this is the last close
    * of the device, as the close might be caused by pthread_cancel() of
-   * a thread currently blocking on any of them
+   * a thread currently blocking on any of them.
+   *
+   * REVISIT:  This logic *only* works in the case where the cancelled
+   * thread had the only reference to the serial driver.  If there other
+   * references, then the this logic will not be executed and the
+   * semaphore count will still be incorrect.
    */
 
   sem_reinit(&dev->xmitsem,  0, 0);