michallenc commented on code in PR #16642:
URL: https://github.com/apache/nuttx/pull/16642#discussion_r2190841020


##########
drivers/mtd/ftl.c:
##########
@@ -914,5 +1010,5 @@ int ftl_initialize(int minor, FAR struct mtd_dev_s *mtd)
   /* Do the real work by ftl_initialize_by_path */
 
   snprintf(path, DEV_NAME_MAX, "/dev/mtdblock%d", minor);
-  return ftl_initialize_by_path(path, mtd);
+  return ftl_initialize_by_path(path, mtd, O_RDWR);

Review Comment:
   It's true it is not that hard to replace `ftl_initialize` with 
`ftl_initialize_by_path` because you have to know `/dev/mtdblockX` path anyway 
if you want to register BCH layer upon FTL.



-- 
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