Skip to content

Enable PWM for nrf52832#318

Merged
bors[bot] merged 4 commits intonrf-rs:masterfrom
Dirbaio:nrf52832-pwm
May 14, 2021
Merged

Enable PWM for nrf52832#318
bors[bot] merged 4 commits intonrf-rs:masterfrom
Dirbaio:nrf52832-pwm

Conversation

@Dirbaio
Copy link
Copy Markdown
Member

@Dirbaio Dirbaio commented May 14, 2021

It compiles cleanly, except for the PAC missing fields for the tasks registers. Replaced those with .bits(1), which works on all chips.

Untested on hardware as I don't have a nrf52832 at hand.

@jonathanpallant
Copy link
Copy Markdown
Member

Tested OK on a DWM1001C.

@jonathanpallant
Copy link
Copy Markdown
Member

@bors r+

@jonathanpallant jonathanpallant removed the request for review from jonas-schievink May 14, 2021 17:32
@jonas-schievink
Copy link
Copy Markdown
Contributor

@bors is for the rust-lang instance, the one we use doesn't use @:

bors r=jonathanpallant

@bors
Copy link
Copy Markdown
Contributor

bors Bot commented May 14, 2021

Build succeeded:

@bors bors Bot merged commit 39c0b8f into nrf-rs:master May 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants