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

Fix channel selection for DMA2 Stream6. #10029

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

manuelilg
Copy link

For TIM1_CH3 the wrong Channel gets selected during timer definition/configuration.
Correct is to use Channel6 which is only for TIM1_CH3 instead of the Channel0 which is for TIM1_CH1, TIM1_CH2 and TIM1_CH3.
See also https://github.com/iNavFlight/inav/blob/release_7.1.1/src/main/drivers/timer_def_stm32f4xx.h#L35
and table below
Screenshot from 2024-05-05 14-09-52

At start up, when only the DSHOT command 0 is send, everything is ok.
But as soon as other commands/values are send then this happens:
Screenshot from 2024-05-05 14-04-39

Additional Infos:
Tested on Target: FLYWOOF411
But the other controllers (F405 and F722) have the same DMA request mapping.
ESC protocol: DSHOT300

To use Channel6 which is only for TIM1_CH3 instead of the Channel0 which is for TIM1_CH1, TIM1_CH2 and TIM1_CH3.
@DzikuVx DzikuVx added this to the 8.0 milestone May 14, 2024
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.

None yet

2 participants