pkarashchenko commented on a change in pull request #5345:
URL: https://github.com/apache/incubator-nuttx/pull/5345#discussion_r795935435



##########
File path: fs/mount/fs_automount.c
##########
@@ -551,6 +862,28 @@ FAR void *automount_initialize(FAR const struct 
automount_lower_s *lower)
       ferr("ERROR: Failed to schedule work: %d\n", ret);
     }
 
+#ifdef CONFIG_FS_AUTOMOUNTER_DRIVER
+
+  /* Initialize the new automount driver instance */
+
+  nxsem_init(&priv->exclsem, 0, 1);
+
+  /* Register driver */
+
+  sprintf(devpath, CONFIG_FS_AUTOMOUNTER_VFS_PATH "/automount%u",

Review comment:
       Done. Please review. I'm just not sure should 
`CONFIG_FS_AUTOMOUNTER_VFS_PATH` be just `/var`, `/var/mnt` or `/var/mount`. 
Because now it leads to `/var/mount/mnt/sdcard0` and seems to be some king of 
duplication of `mount` and `mnt`.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to