Skip to content

Conversation

@bkleiner
Copy link
Collaborator

@bkleiner bkleiner commented Oct 1, 2024

via the MSP_VTX_CONFIG command

@bkleiner bkleiner force-pushed the feature-msp-vtx-read branch from c937f78 to 4f03a0a Compare December 10, 2024 15:48
@bkleiner bkleiner merged commit 2dad829 into iNavFlight:master Dec 10, 2024
20 checks passed
sensei-hacker added a commit to sensei-hacker/inav-configurator that referenced this pull request Dec 19, 2025
Resolves merge conflict in PR iNavFlight#2202 by adding support for dynamic
power_min from firmware while maintaining backward compatibility.

Changes:
- Added FC.VTX_CONFIG.power_min field (default 1)
- Updated MSPHelper.js to read power_min from firmware (INAV 9.1+)
- Added defensive buffer checking for backward compatibility with 9.0
- Falls back to device-type logic if firmware doesn't send power_min
  (MSP VTX: min=0, others: min=1)
- Resolved merge conflict in js/vtx.js by removing both POWER_MIN
  constant and getMinPower()/getMaxPower() functions
- Updated tabs/configuration.js to use FC.VTX_CONFIG.power_min instead
  of VTX.POWER_MIN

This implementation:
- Works with current firmware 9.0 (graceful fallback)
- Ready for future firmware 9.1 with power_min support
- Eliminates hardcoded device-type logic in configurator
- Maintains single source of truth in firmware

Related: iNavFlight#2202
Related: iNavFlight#2206
Related: iNavFlight/inav#10395

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
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