-
Notifications
You must be signed in to change notification settings - Fork 2k
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
cpu/rpx0xx: add initial pwm support #20473
Conversation
With a logic analyzer attached and the following commands:
|
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.
reads good, test results provided
please squash your fixups
Signed-off-by: Dylan Laduranty <dylan.laduranty@mesotic.com>
Signed-off-by: Dylan Laduranty <dylan.laduranty@mesotic.com>
Signed-off-by: Dylan Laduranty <dylan.laduranty@mesotic.com>
9092d9a
to
8b84fc5
Compare
Rebased and squashed. |
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.
trusting @dylad tests
Thanks for the review @kfessel ! |
Contribution description
This PR adds an initial PWM peripheral driver for RP2040-based board.
PWM IP block is feed by the system clock (fixed) so it is not very flexible but it works as intended.
So far I've only add the LED pin as PWM as it is convenient to check with the
osci
command fromtests/periph/pwm
However, I've also tested locally other pins to check the overall logic.
Testing procedure
Play with
tests/periph/pwm
onrpi-pico
for instance.Issues/PRs references
None.