-
Notifications
You must be signed in to change notification settings - Fork 36
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
[Spinla_4_1][DShot] workaound to use DShot in Spinal ver 4.1 #624
Closed
sugihara-16
wants to merge
36
commits into
jsk-ros-pkg:master
from
sugihara-16:PR/feature/Spinal_4_1/DShot
Closed
[Spinla_4_1][DShot] workaound to use DShot in Spinal ver 4.1 #624
sugihara-16
wants to merge
36
commits into
jsk-ros-pkg:master
from
sugihara-16:PR/feature/Spinal_4_1/DShot
Conversation
This file contains 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
… code can be flashed into mcu. However, the telemetry has not been tested yet, since the DShot function should be implemented first.
…nd generate code, not tested.
…pe error in dshot.cpp; fix a 'static' keyword error and add dma address for STM32H7 in dshot.h. When building, some include errors occur.
…size is too small. However, error still occur when accessing the memory.
…l for DMA in stm32H7. change a permission setting to allow NPU access. pwm_test works fine on real machine!
…ug. the received data is checked in a clever way.
…ccessful. Still problem
…ne. Unsuccessful. Still problem" This reverts commit 699c1d1.
…e. tested successfully!" This reverts commit 5d1518e.
…ne motor during one round. [tested using Oscilloscope]
…ted! Now the freq. is limited to 125hz. 250 hz will cause strange behavior.
This PR itself cannot be compatible with the current master system, so it is integrated into #630 . |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
What is this
Enable using dshot protocol in Spinal 4.1.
Details
Followings are main modifications: