drivers: stepper: a4988 #1
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Pancakem
force-pushed
the
a4988_driver
branch
from
015d828 to
14bb550
Compare
Pancakem
force-pushed
the
a4988_driver
branch
from
14bb550 to
0c55abb
Compare
Add a4998 2-phase motor controller driver Signed-off-by: Marvin Ouma <[email protected]>
Pancakem
force-pushed
the
a4988_driver
branch
from
0c55abb to
224ca5a
Compare
zacck
suggested changes
Feb 28, 2025
| add_subdirectory_ifdef(CONFIG_STEPPER_ADI_TMC adi_tmc) | ||
| add_subdirectory_ifdef(CONFIG_STEPPER_TI ti) | ||
| add_subdirectory_ifdef(CONFIG_STEP_DIR_STEPPER step_dir) | ||
| add_subdirectory_ifdef(CONFIG_STEPPER_ALLEGRO allegro) |
There was a problem hiding this comment.
@Pancakem perhaps we can move this to an alphabetical order to avoid the warnings?
zacck
reviewed
Feb 28, 2025
|
|
||
| static int a4988_stepper_enable(const struct device *dev, const bool enable) | ||
| { | ||
| /* enable and sleep pins need to be set logic low to be on */ |
There was a problem hiding this comment.
Hey @Pancakem the data sheet seems not to agree on this see page 10 on here https://www.pololu.com/file/download/A4988.pdf?file_id=0J450
Pancakem
pushed a commit
that referenced
this pull request
Jun 3, 2025
Current code does not build on Cortex-M0, seems like it does not like subs: Error: instruction not supported in Thumb16 mode -- `subs r3,#1' Adding a unified assembler language declaration in the snippet seems to fix the problem, also add an M0+ board so this is tested in CI. Signed-off-by: Fabio Baltieri <[email protected]>
Pancakem
pushed a commit
that referenced
this pull request
Sep 18, 2025
Add support for NV12/NV21, NV16/NV61 and YUV420/YVU420 (semi)planar formats which can be output by the main #1 pipe. Signed-off-by: Alain Volmat <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add a4998 2-phase motor controller driver