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

Various soft serial fixes #3382

Merged
merged 4 commits into from
Mar 26, 2023
Merged

Various soft serial fixes #3382

merged 4 commits into from
Mar 26, 2023

Conversation

raphaelcoeffic
Copy link
Member

Summary of changes:

  • fixes for sending pulses/soft-serial using 32bit timers (X10E; mostly impacting MPM, DSM and similar)
  • fixes for soft-serial RX (mostly X10E; caused by coupling btw. PPM & S.PORT lines)
  • fix for USART driver RX side

This helps coping with spurious impulses on S.PORT created by payload sent over PPM pin (X10E).
... regardless of errors. That allows for reducing the effect of detected errors.
@ParkerEde
Copy link
Contributor

tested with Horus X10S Express and Taranis QX7S ACCESS and a MPM irx4+ MPM. Both works fine with this PR.
Thank you Raphael.

@pfeerick pfeerick added this to the 2.9 milestone Mar 23, 2023
@pfeerick pfeerick self-assigned this Mar 23, 2023
@pfeerick pfeerick added the bug/regression ↩️ A new version of EdgeTX broke something label Mar 26, 2023
@pfeerick
Copy link
Member

Flashing internal MPM on Zorro and TX16S still good, also external MPM on TX16 and X9D+2019.

@pfeerick pfeerick merged commit e016598 into main Mar 26, 2023
@pfeerick pfeerick deleted the fix-soft-serial branch March 26, 2023 01:23
richardclli added a commit that referenced this pull request Apr 17, 2023
richardclli added a commit to richardclli/edgetx that referenced this pull request Apr 17, 2023
richardclli added a commit to richardclli/edgetx that referenced this pull request Apr 17, 2023
richardclli added a commit to richardclli/edgetx that referenced this pull request Apr 17, 2023
richardclli added a commit to richardclli/edgetx that referenced this pull request Apr 17, 2023
richardclli added a commit to richardclli/edgetx that referenced this pull request Apr 19, 2023
richardclli added a commit to richardclli/edgetx that referenced this pull request Apr 22, 2023
richardclli added a commit to richardclli/edgetx that referenced this pull request Apr 24, 2023
richardclli added a commit to richardclli/edgetx that referenced this pull request Apr 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug/regression ↩️ A new version of EdgeTX broke something
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants