Skip to content

Conversation

@skorya
Copy link

@skorya skorya commented Jan 25, 2026

PR Type

Enhancement


Description

  • Updates 5.8GHz VTX frequency table with new values

  • Changes all five band frequency arrays (A-R bands)

  • Shifts frequencies from 5GHz to 6GHz range


Diagram Walkthrough

flowchart LR
  A["5.8GHz Frequency Table"] -- "Update all 40 frequencies" --> B["6GHz Frequency Table"]
  B -- "Bands A-R" --> C["Updated vtx58frequencyTable"]
Loading

File Walkthrough

Relevant files
Configuration changes
vtx_string.c
Update 5.8GHz VTX frequency table to 6GHz                               

src/main/io/vtx_string.c

  • Replaces all 40 frequency values in vtx58frequencyTable array
  • Updates band A frequencies from 5865-5725 MHz to 6110-6250 MHz
  • Updates band B frequencies from 5733-5866 MHz to 6270-6410 MHz
  • Updates bands E, F, R with corresponding 6GHz range frequencies
+5/-5     

@github-actions
Copy link

Branch Targeting Suggestion

You've targeted the master branch with this PR. Please consider if a version branch might be more appropriate:

  • maintenance-9.x - If your change is backward-compatible and won't create compatibility issues between INAV firmware and Configurator 9.x versions. This will allow your PR to be included in the next 9.x release.

  • maintenance-10.x - If your change introduces compatibility requirements between firmware and configurator that would break 9.x compatibility. This is for PRs which will be included in INAV 10.x

If master is the correct target for this change, no action is needed.


This is an automated suggestion to help route contributions to the appropriate branch.

@qodo-code-review
Copy link
Contributor

ⓘ Your approaching your monthly quota for Qodo. Upgrade your plan

PR Compliance Guide 🔍

All compliance sections have been disabled in the configurations.

{ 5705, 5685, 5665, 5645, 5885, 5905, 5925, 5945 }, // E
{ 5740, 5760, 5780, 5800, 5820, 5840, 5860, 5880 }, // F
{ 5658, 5695, 5732, 5769, 5806, 5843, 5880, 5917 }, // R
{ 6110, 6130, 6150, 6170, 6190, 6220, 6230, 6250 }, // A
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggestion: Change the 6th channel value in Band A from 6220 to 6210 to maintain a uniform 20 MHz frequency step. [possible issue, importance: 7]

Suggested change
{ 6110, 6130, 6150, 6170, 6190, 6220, 6230, 6250 }, // A
{ 6110, 6130, 6150, 6170, 6190, 6210, 6230, 6250 }, // A

@skorya skorya closed this by deleting the head repository Jan 25, 2026
@sensei-hacker
Copy link
Member

Thanks though. Looking forward to your next PR!

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants