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

review RC parsing and NuttX parity bit #10939

Open
dagar opened this issue Nov 28, 2018 · 6 comments
Open

review RC parsing and NuttX parity bit #10939

dagar opened this issue Nov 28, 2018 · 6 comments

Comments

@dagar
Copy link
Member

dagar commented Nov 28, 2018

SBUS uses an even parity bit. Is it actually being used with current NuttX?

@dagar
Copy link
Member Author

dagar commented Feb 27, 2019

@davids5 what was the conclusion here?

@davids5
Copy link
Member

davids5 commented Feb 27, 2019

It is configured in the correct mode IIRC the error is not propagated nor that chatrater dropped. The issues that shed light on this was a mis configuration. There is some "Framing" in the decoder that may detect a problem or not. Are there any open issues that indicate SBUS related problems?

@PX4 PX4 deleted a comment from stale bot Jun 24, 2019
@stale stale bot removed the Admin: Wont fix label Jun 24, 2019
@PX4 PX4 deleted a comment from stale bot Sep 25, 2019
@stale stale bot removed the Admin: Wont fix label Sep 25, 2019
@stale stale bot added the stale label Dec 24, 2019
@PX4 PX4 deleted a comment from stale bot Dec 24, 2019
@stale stale bot removed the stale label Dec 24, 2019
@stale
Copy link

stale bot commented Mar 23, 2020

This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions.

@stale stale bot added the stale label Mar 23, 2020
@stale
Copy link

stale bot commented Jun 21, 2020

This issue has been automatically marked as stale because it has not had recent activity. Thank you for your contributions.

@junwoo091400
Copy link
Contributor

Is this still an issue?

@stale stale bot removed the stale label Jul 27, 2022
@davids5
Copy link
Member

davids5 commented Jul 27, 2022

There is a PR in to test a fix #19446 but I am not sure if the testing that has been done is testing the right things or reporting the right problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants