jlaitine commented on PR #17011: URL: https://github.com/apache/nuttx/pull/17011#issuecomment-3433949157
The code which redirected the ndelay&udelay to oneshot driver ended up using systick to implement udelay, which caused e.g. udelay(1) to sleep 10000-20000 microseconds instead of busylooping for one microsecond. This just blew up many places where a short bysyloop was needed, in device drivers Just when you revert the fixes, please *make sure* that ndelay/udelay never end up to a systick based delay! Matteo Golin kirjoitti keskiviikko 22. lokakuuta 2025: > @linguini1 commented on this pull request. > > > > > @@ -12,6 +12,7 @@ CONFIG_ARCH_BOARD_SIM=y > CONFIG_ARCH_CHIP="sim" > CONFIG_ARCH_SIM=y > CONFIG_BOARD_LATE_INITIALIZE=y > +CONFIG_BOARD_LOOPSPERMSEC=0 > > Xiang can probably answer better, but my understanding is that once the `CLOCKDEVICE` logic is merged in upstream, that can replace the delay method used by `arch_alarm.c` and make it more accurate (not using busy-wait, not using system-ticks). Unfortunately I have been to busy to go back and re-watch the talk, but I plan to sometime soon. > > -- > Reply to this email directly or view it on GitHub: > https://github.com/apache/nuttx/pull/17011#discussion_r2452619791 > You are receiving this because you were mentioned. > > Message ID: ***@***.*** -- 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]
