Skip to content

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

Merged
merged 1 commit into from
Jul 15, 2019

Conversation

jeromecoutant
Copy link
Collaborator

Description

Nightly session

@LMESTM @c1728p9 @MarceloSalazar

Pull request type

[x] Fix
[ ] Refactor
[ ] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

@ciarmcom ciarmcom requested review from a team July 9, 2019 21:00
@ciarmcom
Copy link
Member

ciarmcom commented Jul 9, 2019

@jeromecoutant, thank you for your changes.
@ARMmbed/mbed-os-hal @ARMmbed/mbed-os-test @ARMmbed/mbed-os-maintainers please review.

stevew817 added a commit to SiliconLabs/mbed-os that referenced this pull request Jul 9, 2019
Two issues:
* Downcasting too early
* Potential for a uint32_t overflow in an intermediate calculation

Passing test requires ARMmbed#11005 to be merged.
@c1728p9
Copy link
Contributor

c1728p9 commented Jul 9, 2019

CC @fkjagodzinski @mprse

Copy link
Contributor

@mprse mprse left a 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.

c1728p9 pushed a commit to c1728p9/mbed-os that referenced this pull request Jul 10, 2019
Two issues:
* Downcasting too early
* Potential for a uint32_t overflow in an intermediate calculation

Passing test requires ARMmbed#11005 to be merged.
@0xc0170
Copy link
Contributor

0xc0170 commented Jul 10, 2019

Curiosity, why do we need to wait (commit msg does not provide the answer, not everyone might know?)

@evedon
Copy link
Contributor

evedon commented Jul 12, 2019

CI started

@mbed-ci
Copy link

mbed-ci commented Jul 12, 2019

Test run: SUCCESS

Summary: 4 of 4 test jobs passed
Build number : 1
Build artifacts

@artokin artokin merged commit 51b44d1 into ARMmbed:master Jul 15, 2019
@jeromecoutant jeromecoutant deleted the PR_PWM branch July 16, 2019 09:21
yogpan01 pushed a commit to yogpan01/mbed that referenced this pull request Jul 18, 2019
Two issues:
* Downcasting too early
* Potential for a uint32_t overflow in an intermediate calculation

Passing test requires ARMmbed#11005 to be merged.
evedon pushed a commit that referenced this pull request Jul 25, 2019
Two issues:
* Downcasting too early
* Potential for a uint32_t overflow in an intermediate calculation

Passing test requires #11005 to be merged.
kotkcy pushed a commit to kotkcy/mbed-os that referenced this pull request Aug 7, 2019
Two issues:
* Downcasting too early
* Potential for a uint32_t overflow in an intermediate calculation

Passing test requires ARMmbed#11005 to be merged.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants