adafruit / adafruit/Adafruit_CircuitPython_Motor
Add support for phase/enable driver interface.
- Dominant language
- Python
- Stars
- 66
- Forks
- 41
- PR merge metrics
- No merged PRs in 30d
Description
DC motor drivers like the [DRV8256E](https://www.ti.com/lit/ds/symlink/drv8256.pdf) use a phase/enable interface. This interface requires a single PWM "enable" signal and a direction (phase) signal, as opposed to the dual-PWM IN/IN interface used by drivers like the Adafruit DRV8833. From what I can tell, this library doesn't currently support the phase/enable interface, only the IN/IN interface.
The first question is: Am I overlooking something?
If not, I have a [fork](https://github.com/adafruit/Adafruit_CircuitPython_Motor/compare/main...lcmcninch:Adafruit_CircuitPython_Motor:main) that enhances the `DCMotor` class to optionally support this driver interface. I'm not sure how common this type of driver is. Would you be interested in a pull request with these changes?
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by reading the DCMotor class and compare its current interface handling with the linked fork. Confirm whether phase/enable drivers are already supported, then evaluate the fork's optional support for a PWM enable signal and direction signal. Done means the library supports this interface without regressing dual-PWM IN/IN drivers; no test file is named in the issue.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- embedded-iot
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100