Clarify some NFS parsing and point out things that need to be fixed.

modules.d/95nfs/nfsroot

@@ -63,7 +63,7 @@ netif="$1"
 root="$2"
 NEWROOT="$3"
 
-# If it's not nfs we don't continue
+# Continue if nfs prefix
 case "${root%%:*}" in
     nfs|nfs4);;
     *) return;;
@@ -80,6 +80,7 @@ root_to_var $root
 [ -z "$path" ] && root_to_var $nfs:$new_root_path
 
 #Empty path defaults to "/tftpboot/%s"
+# FIXME: This should happen only in nfsroot.txt legacy mode
 [ -z "$path" ] && path="/tftpboot/%s"
 
 if [ -z "$server" ] ; then

modules.d/95nfs/parse-nfsroot.sh

@@ -73,6 +73,7 @@ netroot_to_var() {
 [ -z "$nfsroot" ] && nfsroot=$(getarg nfsroot=)
 
 # Handle old style <server-ip>:/<path
+# FIXME: root= is not handled by this yet.
 case "$netroot" in
     [0-9]*:/*|[0-9]*\.[0-9]*\.[0-9]*[!:]|/*)
        netroot=nfs:$netroot;;
@@ -89,7 +90,7 @@ case "${root%%:*}" in
     ;;
 esac
 
-# If it's not empty or nfs we don't continue
+# Continue if nfs or blank prefix
 case "${netroot%%:*}" in
     ''|nfs|nfs4|/dev/nfs);;
     *) return;;
@@ -107,7 +108,7 @@ if [ -n "$nfsroot" ] ; then
     esac
 fi
 
-# If it's not nfs we don't continue
+# Continue if nfs prefix
 case "${netroot%%:*}" in
     nfs|nfs4|/dev/nfs);;
     *) return;;