search

ArduPilot / ardupilot

ArduPlane, ArduCopter, ArduRover, ArduSub source
http://ardupilot.org/
GNU General Public License v3.0
10.8k stars 17.26k forks source link

AP_RSSI: Add support for Spektrum Satellite "Fade Counter" to fake RSSI #7141

Open DavidIngraham opened 6 years ago

DavidIngraham commented 6 years ago

Issue details

Spektrum Satellite Receivers do not support RSSI, but do report a "Fade Counter" that signifies the number of dropped packets. This is documented here: https://www.spektrumrc.com/ProdInfo/Files/Remote%20Receiver%20Interfacing%20Rev%20A.pdf

While it is not a true RSSI value, I would argue that it would still be helpful to support this though the AP_RSSI library. AP_RSSI already supports reciever-protocol specific RSSI, so I think the changes would only need to be implemented in the PX4_firmware layer.

Cleanflight supports this. They have some good discussion here: https://github.com/cleanflight/cleanflight/issues/1658

Version

N/A (Enhancement)

Platform

[ ] All [ ] AntennaTracker [X] Copter [X] Plane [X] Rover [ ] Submarine

Airframe type

N/A

Hardware type

Pixhawk

Logs

N/A

geofrancis commented 6 years ago

I believe this has been implimented with sbus so rssi = % of packets received.

OXINARF commented 6 years ago

I'm not closing this as we don't usually do that for feature requests, but doesn't look very useful. Besides the technical difficulty of knowing the format of the received packet, the fades field will have jumps (and so will the RSSI) and when the limit is reached it doesn't even wrap around, it just stays fixed.

geofrancis commented 6 years ago

This #3274 is the commit that does what is requested here with SBUS and sumd instead of DSM.