Skip to content
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

Pwm #94

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Pwm #94

wants to merge 2 commits into from

Conversation

francolanzi
Copy link

I added support for PWM signals.
It's a prerequisite to write the line: "dtoverlay=pwm-2chan" in the file /boot/config.txt to enable the PWM.
I tested it briefly in my Raspberry Pi 3 B, but if someone could do a deeper test, feel free to do that.
Any suggestion is welcome!

@JamesBarwell
Copy link
Owner

Thanks very much for the contribution. It's not something I've really explored much as I haven't needed it myself, and it can be difficult keeping up with the limited functionality this module offers currently.

There was a request for it back in #62 though - maybe there is more demand?

I hope you won't mind if I leave this for a bit until I've chance to read up on how it all works a bit more.

@sholzmayer
Copy link

sholzmayer commented Dec 20, 2020

For me this implementation looks similar to the python implementation. There is some further explanation on how the pwm management on the raspberry pi kernel works.

I would be glad to see this pull request merged 🎉.

@dzek69
Copy link

dzek69 commented Aug 28, 2021

+1 for merging :)

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.

4 participants