acassis commented on a change in pull request #5635: URL: https://github.com/apache/incubator-nuttx/pull/5635#discussion_r815865145
########## File path: fs/romfs/Kconfig ########## @@ -11,4 +11,13 @@ config FS_ROMFS Enable ROMFS filesystem support if FS_ROMFS + +config FS_ROMFS_CACHE_NODE + bool "Enable cache node of ROMFS file system" + default n Review comment: Maybe it could be default y if !CONFIG_DEFAULT_SMALL. I think if a feature brings benefit to users (like speed/performance) it could be enabled by default. Otherwise we are creating many useful features that many people don't know that exist (he/she will need to search high and low the menuconfig to discover it). -- 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