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