Laczen commented on PR #16793: URL: https://github.com/apache/nuttx/pull/16793#issuecomment-3136045910
> > > > Please wait until it is clear how multiple ftl (simple, dhara. ...) are going to be supported. > > > > See #16789. > > > > > > > > > The rename doesn't impact multiple FTL support. > > > > > > Maybe not, but if the result of multiple FTL support is that both `ftl_initialize` and `ftl_initialize_by_path` are needed/useful this PR would need to be undone. > > ftl_initialize_by_path is more functionality than ftl_initialize, so all ftl_initialize could switch to call ftl_initialize_by_path without problem. Why do you think multiple FTL need the old ftl_initialize? Consistency, if there are other device drivers that are using the `xxx_initialize(yyy, minor, ...)` (which I think is used e.g. for `/dev/eth`) it would be better to keep this pattern. This of course would only be valid if it is needed to keep the old ftl_initialize. -- 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
