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

Copter: SUMD not working 3.6.5 #10346

Closed
lvale opened this issue Jan 25, 2019 · 7 comments
Closed

Copter: SUMD not working 3.6.5 #10346

lvale opened this issue Jan 25, 2019 · 7 comments

Comments

@lvale
Copy link
Member

lvale commented Jan 25, 2019

img_20190125_215453
img_20190125_215438

Same receiver plugged on RC IN on the following boards, all running Release version 3.6.5 Chibios:

PixHawk 1
MindRacer v2
CUAV v5
PixRacer

RC Receiver GR-12L has the capability of outputting either SBUS or SUMD

When set to SBUS all channels work as expected

When set to SUMD RC receiver is not detected by ArduPilot Copter 3.6.5 (tested both Copter and TradHeli) and no channel gets detected.

Checked with Logic analyser the output of the RC receiver and it is outputting data correctly on both modes.

According to this https://github.com/ArduPilot/ardupilot/blob/Copter-3.6/ArduCopter/ReleaseNotes.txt

this issue wasn't supposed to happen.

Same receiver with other flight stacks works both on SBUS and SUMD

@rmackay9
Copy link
Contributor

@lvale, is this a regression from 3.6.4 or something new that was expected to work in 3.6.5 but doesn't?

@lvale
Copy link
Member Author

lvale commented Jan 26, 2019

didn't work before.

according to fixes on 3.6.5rc1 this is new
RC protocol decoding for SRXL, SUMD and ST24 extended to all boards including pixracer and ChibiOS-only boards

@rmackay9
Copy link
Contributor

Great, thanks. That's quite a relief.

This is probably something that we should discuss with @tridge

@lvale
Copy link
Member Author

lvale commented Mar 2, 2019

Adding a Logic trace

with SUMD out
GR12L_v193_SUMD_OUT_20msPeriod.logicdata.zip

with SBUS out
GR12L_v193_SBUS_OUT_20msPeriod.logicdata.zip

tridge added a commit to tridge/ardupilot that referenced this issue Mar 4, 2019
based on data from Luis from issue ArduPilot#10346
@tridge
Copy link
Contributor

tridge commented Mar 4, 2019

@lvale I added the data from your test to the RC protocol test code and it passes fine. It get these channel values:
1516, 1500, 1100, 1500, 1900, 1900, 1900, 1900, 1900, 1900, 1900, 1900
so it really should be working!

tridge added a commit that referenced this issue Mar 4, 2019
based on data from Luis from issue #10346
drnic pushed a commit to drnic/ardupilot that referenced this issue Mar 4, 2019
based on data from Luis from issue ArduPilot#10346
LiYuanxing pushed a commit to LiYuanxing/ardupilot that referenced this issue Apr 25, 2019
@Ralfde
Copy link

Ralfde commented May 26, 2019

Would it be easier, if you let SBUS at the LEDSTRIP pin and set SUMD to the RX1 pin?
Then it would be maybe more easy for the software to use the inverted SBUS signal and to SUMD signal at the same time. Also it would be really interesting to switch from SUMD to SBUS an back, in case one of the signals would have failsafe. So RF redundancy could be used.

@peterbarker
Copy link
Contributor

Please reopen this issue if it is still a problem on our stable or master releases. tridge was unable to reproduce with the supplied information.

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

No branches or pull requests

5 participants