-
Notifications
You must be signed in to change notification settings - Fork 8.4k
Bluetooth: Host: Use macro for nanoseconds per second #89913
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
Conversation
|
Hello @alexapostolu, and thank you very much for your first pull request to the Zephyr project! |
|
@alexapostolu thanks for the PR! You still need to make the Compliance check pass before this can be merged, however. Seems like you have a too long line in the commit message. |
Thalley
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good, but please fix compliance :)
Use the macro NSEC_PER_SEC defined in sys_clock.h for nanoseconds per second when updating write stats. Signed-off-by: Alex Apostolu <apostolu240@gmail.com>
6fbbb9c to
a0354c1
Compare
|
Fixed the commit message |
|
|
Hi @alexapostolu! To celebrate this milestone and showcase your contribution, we'd love to award you the Zephyr Technical Contributor badge. If you're interested, please claim your badge by filling out this form: Claim Your Zephyr Badge. Thank you for your valuable input, and we look forward to seeing more of your contributions in the future! 🪁 |



Use the macro
NSEC_PER_SECdefined insys_clock.hfor nanoseconds per second when updating write stats.Fixes: #62641
Issue #62641 was mostly implemented, but as a comment pointed out there were two files that still used the
1000000000Uconstant instead of the macroNSEC_PER_SEC, which is defined as((NSEC_PER_USEC) * (USEC_PER_MSEC) * (MSEC_PER_SEC)).