linguini1 opened a new issue, #17004: URL: https://github.com/apache/nuttx/issues/17004
### Is your feature request related to a problem? Please describe. The `CONFIG_BOARD_LOOPSPERMSEC` option has a default value of 5000, which is not applicable to any boards and is not a "sane default". This results in configurations missing the config option having completely incorrect results from the `up_xdelay` series of functions. ### Describe the solution you'd like I believe that the default value for this option should be set to `-1`, such that when compiling the code, if the value is `-1` (checked at compile time), an error is thrown. This will force the user to set the configuration value themselves using something like `calib_udelay` and they will be aware of the issue. The problem is that if it has a silent, incorrect default, users will have timing issues with some drivers and will try to debug driver failures instead of realizing this issue (the config option isn't super easy to know about without scouring documentation). This is especially frustrating on new board ports where many different things could be going wrong. The developer shouldn't waste time on a silent problem that is easily solvable. ### Describe alternatives you've considered _No response_ ### Verification - [x] I have verified before submitting the report. -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
