-
Notifications
You must be signed in to change notification settings - Fork 3k
FPGA PWM: wait 1 period before measurement #11005
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
@jeromecoutant, thank you for your changes. |
Two issues: * Downcasting too early * Potential for a uint32_t overflow in an intermediate calculation Passing test requires ARMmbed#11005 to be merged.
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.
Confirm that the test still works on K64F
.
Two issues: * Downcasting too early * Potential for a uint32_t overflow in an intermediate calculation Passing test requires ARMmbed#11005 to be merged.
Curiosity, why do we need to wait (commit msg does not provide the answer, not everyone might know?) |
CI started |
Test run: SUCCESSSummary: 4 of 4 test jobs passed |
Two issues: * Downcasting too early * Potential for a uint32_t overflow in an intermediate calculation Passing test requires ARMmbed#11005 to be merged.
Two issues: * Downcasting too early * Potential for a uint32_t overflow in an intermediate calculation Passing test requires #11005 to be merged.
Two issues: * Downcasting too early * Potential for a uint32_t overflow in an intermediate calculation Passing test requires ARMmbed#11005 to be merged.
Description
Nightly session
@LMESTM @c1728p9 @MarceloSalazar
Pull request type