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

Add support for RADIO_LINK_STATS #26470

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

olliw42
Copy link
Contributor

@olliw42 olliw42 commented Mar 9, 2024

This is the second step in adding MAVLink receiver support, which is about radio link statistics. The first step was adding the MAVLinkRadio receiver class, which was about handling the RADIO_RC_CHANNELS mavlink message, and which was recently merged (#26356).

The PR is dependent on ArduPilot/mavlink#354, which brings in the definition of the RADIO_LINK_STATS message.

The PR will thus not work at this time, but is intended to show where things may go, and to spur the discussion on the code.

The PR does these things:

  • adds handling of RADIO_LINK_STATS to the AP_RCProtocol library. In this library the LQ and rssi fields are digested and made available via the usual ways.
  • adds logging. Since these are more than ca 7 data items to log, two new log items are created for the logging, RDRX and RDTX.
  • adds handling of RADIO_LINK_STATS to the GCS library. It reads the message and forwards it to the AP_RCProtocol library and does the logging.

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.

1 participant