-
Notifications
You must be signed in to change notification settings - Fork 7.4k
roll-up issue for timer/clock enhancements #19282
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
I like the concept of making timeouts independent of milliseconds, great plan. I read through the APIs, and the fact that everything gets converted to local units at compile time makes this quite low overhead. 64-bit numbers are a little problematic, I noticed one issue with 64-bit numbers while working with the previous implementation of k_uptime_get(). It is hard to printout the numbers, in printk() the %lld and %llu formatters don't seem to work for 64-bit numbers. (as a side note shouldn't printk be k_printf?) It would seem to me that you should include some support functions to convert the time numbers, independent of the storage format of k_timeout_t (32-bit, 64-bit, or structs with flags with 63, 63, 61 bit numbers, or whatever) into something human readable should be part of the package. -Lawrence- |
@lawrencek52 Agreed. We've got |
most items resolved, remaining bullets do not have enough details or context, so closing. If needed, please open seperate issues with more details. |
@pabigot I believe your concerns are now all addressed in #76335 as an umbrella RFC for everything that has been proposed around precision timing. But maybe you want to give the linked RFC a quick review? I'm now returning to implementing layer 1 and two of the proposed clock architecture, so this will be an active RFC with new PRs being linked to it in the future. |
This issue is a placeholder to track the progress of changes to the controlled delay and clock infrastructure of Zephyr to support a variety of needs identified in various issues.
See the TSC presentation for background.
The text was updated successfully, but these errors were encountered: