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

HDZero VTX Table update #486

Merged

Conversation

nerdCopter
Copy link
Member

@nerdCopter nerdCopter marked this pull request as ready for review August 30, 2024 19:11
@limonspb
Copy link
Member

@sugaarK could you plz check

@sugaarK
Copy link
Member

sugaarK commented Aug 30, 2024

The original preset is the legal race band channels for most of the world and what HDzero published. hDzero added some channels that are not legal to the VTX ( even more of you unlock ) and MSP VTX automatically overwrites the VTX table with its current firmware VTX table on power up and handshake with the FC. So it’s non issue but I’m not sure we change the preset as it’s as legal every where as the one..

@sugaarK
Copy link
Member

sugaarK commented Aug 30, 2024

@ctzsnooze

@ctzsnooze
Copy link
Member

ctzsnooze commented Aug 31, 2024

@sugaarK - check the proposed change in the commit.

The channels in the modified HDZero list, which get flashed to the FC, are:

  • RaceBand 1-8 (5658-59217)
  • Fatshark 1, 2 and 4 (5740-5800)
  • Boscam E 1 (5705)

I'm sure some of these are not legal, in some countries, but we can't control that.

The outliers are the RaceBand extremes, and perhaps Boscam E1, which is lower than Fatshark 1 at 5740. The Fatshark ones are in the middle of the standard FPV frequency ranges.

The values in our Preset won't work as they are. They only work if there is telemetry, to allow the HDZero goggles to over-ride our Preset.

Having a Present that won't work makes it a bit pointless. This PR fixes it.

@sugaarK sugaarK merged commit 56fa424 into betaflight:master Aug 31, 2024
1 check passed
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.

5 participants