acassis commented on code in PR #14266: URL: https://github.com/apache/nuttx/pull/14266#discussion_r1799587744
########## include/nuttx/mmcsd.h: ########## @@ -95,6 +95,16 @@ struct mmc_ioc_cmd unsigned int blksz; unsigned int blocks; + /* Sleep at least postsleep_min_us useconds, and at most + * postsleep_max_us useconds *after* issuing command. Needed for + * some read commands for which cards have no other way of indicating + * they're ready for the next command (i.e. there is no equivalent of + * a "busy" indicator for read operations). + */ + + unsigned int postsleep_min_us; + unsigned int postsleep_max_us; Review Comment: Since it is only necessary for some "limited" SDcards, isn't it better to wrap it around a config flag? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
