Skip to content

Pwm: allow get_duty to return the old value for some time#238

Merged
bors[bot] merged 1 commit into
rust-embedded:masterfrom
eldruin:pwm-trait-get-old-ok
Jul 22, 2020
Merged

Pwm: allow get_duty to return the old value for some time#238
bors[bot] merged 1 commit into
rust-embedded:masterfrom
eldruin:pwm-trait-get-old-ok

Conversation

@eldruin

@eldruin eldruin commented Jul 22, 2020

Copy link
Copy Markdown
Member

This is #140 applied to the Pwm trait as well, since the same concerns apply. Follows up on #236.

PWM is often implemented in a buffered way to allow glitch-free
operation; as a result, implementing a strict "you get what you last
set" is not feasible for some implementations.
@rust-highfive

Copy link
Copy Markdown

r? @therealprof

(rust_highfive has picked a reviewer for you, use r? to override)

@therealprof therealprof left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

bors r+

@bors

bors Bot commented Jul 22, 2020

Copy link
Copy Markdown
Contributor

Build succeeded:

@bors bors Bot merged commit d81cf7c into rust-embedded:master Jul 22, 2020
@eldruin eldruin deleted the pwm-trait-get-old-ok branch July 22, 2020 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Review is incomplete T-hal

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants