Skip to content

Periodically Transmit Callsigns on Licensed Frequencies#372

Open
AarC10 wants to merge 7 commits intomainfrom
feature/Aaron/CallsignTransmission
Open

Periodically Transmit Callsigns on Licensed Frequencies#372
AarC10 wants to merge 7 commits intomainfrom
feature/Aaron/CallsignTransmission

Conversation

@AarC10
Copy link
Member

@AarC10 AarC10 commented Feb 12, 2026

Description

To be law abiding citizens we need to transmit callsigns after every transmission, or every 10 minutes. Just do every 5 minutes, to be safe while not murdering our transmit times for downlinking flight data

Type of change

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Brain

Checklist:

  • New functionality is documented in the necessary spots (i.e new functions documented in the header)
  • Unit tests cover any new functionality or edge cases that the PR was meant to resolve (if applicable)
  • The CI checks are passing
  • I reviewed my own code in the GitHub diff and am sure that each change is intentional
  • I feel comfortable about this code flying in a rocket

@AarC10 AarC10 requested a review from a team as a code owner February 12, 2026 23:49
@AarC10 AarC10 requested a review from Copilot February 12, 2026 23:49
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR introduces a periodic callsign broadcast mechanism intended to ensure the radio transmits an identifier while operating on licensed frequencies, using a timer-driven tenant and a helper check on the LoRa configuration.

Changes:

  • Added CCallsignBroadcastTenant to periodically enqueue a callsign payload for downlink.
  • Added CLora::IsLicensedFrequency() helper (410–450 MHz) to gate callsign behavior.
  • Hooked callsign tenant registration into CRadioModule::SetupCallbacks() when the current frequency is licensed.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
lib/f_core/radio/application/c_callsign_broadcast_tenant.cpp Implements timer callback to send a callsign LaunchLoraFrame.
include/f_core/radio/application/c_callsign_broadcast_tenant.h Declares the new callsign broadcast tenant.
include/f_core/radio/c_lora.h Adds IsLicensedFrequency() helper.
app/backplane/radio_module/src/c_radio_module.cpp Registers callsign broadcast during callback setup when licensed.
app/backplane/radio_module/include/c_radio_module.h Instantiates the new callsign broadcast tenant (currently with a hardcoded callsign).
include/f_core/radio/frame_handlers/c_lora_freq_request_tenant.h Removes extra blank lines.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

2 participants

Comments