micropython-esp32 icon indicating copy to clipboard operation
micropython-esp32 copied to clipboard

Implement MCPWM module support

Open MrSurly opened this issue 8 years ago • 8 comments

I'm going to begin development on a thin wrapper around MCPWM, starting with implementation of capture.

MrSurly avatar May 19 '17 19:05 MrSurly

The MCPWM link is dead... new link is MCPWM

eugenio avatar Jun 07 '17 11:06 eugenio

Updated URL.

MrSurly avatar Jun 07 '17 17:06 MrSurly

Sounds like a great idea. Someone earlier (LinuxConfAU, I think) brought up the idea of using the MCPWM "RMT" function to clock bits out to NeoPixel type LEDs, which seems like it might work too.

nickzoic avatar Aug 03 '17 12:08 nickzoic

Since I needed 6 PWM channels with independent frequencies, I extended the esp32-module with MCPWM support: https://github.com/bskp/micropython_esp32_mcpwm. Currently, only basic functionality is covered and I do not plan to dive much deeper. However, I'm happy if it helps someone else!

bskp avatar Sep 24 '19 09:09 bskp

@bskp FYI this repository is no longer active. All activity is now at https://github.com/micropython/micropython, and also at https://forum.micropython.org

dpgeorge avatar Sep 24 '19 12:09 dpgeorge

@dpgeorge thank you for the notice. Is there a chance for this feature being merged upstream, even though the API deviates from the one of machine.PWM? If so, I'll open a pull request.

bskp avatar Sep 24 '19 12:09 bskp

G'day @bksp for what it's worth I still think this is a great hardware feature to have exposed in MicroPython :-)

The work in this repo has now been merged back into the main micropython/micropython repo, and it'd be well worth opening this issue over there and working towards an agreement on an API change to allow support for this. I'm happy to help if I can.

@damien did we work out how to lock this repo to prevent confusion?

nickzoic avatar Sep 24 '19 23:09 nickzoic

did we work out how to lock this repo to prevent confusion?

It's possible to archive a repository to make it read-only, but I think it's good to keep this un-archived so we can close/update issues/PRs here as they are resolved upstream. (And we rarely get comments on this repo anymore anyway.)

dpgeorge avatar Sep 25 '19 03:09 dpgeorge