VTOL.md: Rename smix max fixed value#11582
Open
sensei-hacker wants to merge 32 commits into
Open
Conversation
Add documentation about using CMAKE_BUILD_TYPE=Release to save disk space
when building firmware for production/CI/releases.
Key points:
- Default RelWithDebInfo mode uses ~109 GB for all targets
- Release mode uses ~4-6 GB (96% reduction)
- Debug symbols are stripped from final .hex files anyway
- Final binaries are identical in both modes
Updated files:
- docs/development/Building in Linux.md
- Add brief note about debug symbols in single target section
- Add new "Building All Targets or Release Builds" section with
disk usage comparison and Release mode instructions
- docs/development/release-create.md
- Add "Building Firmware Locally" section with Release mode
instructions and disk usage information
Also fixes typo: "Bulding" -> "Building"
ANYFC, CLRACINGF4AIR (V1/V2/V3), FF_F35_LIGHTNING, WINGFC, FLYINGRCF4WINGMINI_NOT_RECOMMENDED, AIRBOTF7, and OMNIBUSF7NANOV7 all lack USE_FLASHFS/USE_SDCARD in their target config. Building _bl/_for_bl/_with_bl variants for these targets produces binaries that define MSP_FIRMWARE_UPDATE but have no storage backend, making OTA firmware update silently non-functional. Add NO_BOOTLOADER to cmake/stm32.cmake (following the DISABLE_MSC pattern) and set it on each affected target. This suppresses the broken bootloader build variants while leaving open the possibility of adding proper storage support to these targets in the future. Note: bootloader variants are not built by CI (they are not added to VALID_TARGETS), so this bug was never caught by automated builds. Fixes: iNavFlight#11521
- MSP_SIMULATOR v2/v3 protocol: refactor into readMspSimulatorValues() and mspProcessSimulatorCommand(); v3 uses 2-byte flags field - New HITL flags: HITL_RANGEFINDER, HITL_CURRENT_SENSOR, HITL_SIM_RC_INPUT, HITL_FAILSAFE_TRIGGERED, HITL_SITL_MODE - Fix GPS velocity: exclude HITL mode from validVelNE path (same as SITL) - battery.c: inject simulated current when HITL_CURRENT_SENSOR active - displayport_msp_osd.c: enable OSD vtxActive in SITL mode - runtime_config: add SIMULATOR_MSP_VERSION_3 and new HITL flag bits - common.h: promote RX_SIM and RANGEFINDER_FAKE out of SITL-only target.h - xplane.h: add stdbool.h include and simXPlaneClose() declaration - xplane.c: INAV-XITL plugin with DataRef state machine; add atomic thread safety and simXPlaneClose() from maintenance-9.x
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
STM32H743 based FC with dual ICM42688P gyros on SPI1/SPI4, MAX7456 OSD on SPI2, M25P16 flash on SPI3, BMP280/DPS310 baro on I2C1, 8 motors (TIM1x4, TIM8x2, TIM3x2), 8 UARTs, and PINIO for VTX and camera switching.
Corrected the spelling of 'Bulding' to 'Building' in the documentation.
…ch-8 Sensei hacker patch 8
Under INTERPROCEDURAL_OPTIMIZATION, the compiler inlines settingGet and settingGetIndex with divergent static settingsTable base addresses, causing MSPV2_SETTING to return wrong data for some settings while MSP2_COMMON_SETTING_INFO (name lookup) is correct.
…_fix OSD flight efficiency formatting fix
- Improve comment accuracy: describe observed behavior rather than the inferred mechanism; note why raw attribute is used instead of NOINLINE macro (NOINLINE is empty on non-F7/H7 targets but bug affects all LTO release builds) - Add noinline to settingGetPgn which has identical pointer arithmetic against settingsTable and is called from settingGetValuePointer - Add brief comments above settingGetIndex and settingGetPgn pointing back to the main explanation on settingGet
…to-mspv2setting-index fix: prevent LTO from breaking settingGet/settingGetIndex round-trip
PC-port pins on STM32H743 require ADC3. PC1 (VBat, ADC123_INP11) and PC2 (Current, ADC3_INP0) both work on ADC3; ADC1 init silently fails for these pins. Same fix applied to MAMBAH743, FOXEERH743.
…ets-no-storage fix: disable bootloader variants for targets without storage backend
- Add USE_I2C_DEVICE_2 (PB10/PB11) for external magnetometer - Route MAG_I2C_BUS to BUS_I2C2 to match board layout - Fix VBAT_SCALE_DEFAULT (was Betaflight value, not INAV scale) Both ADC and magnetometer verified working on hardware.
…dixh743 New target: SPEDIXH743
HITL/SITL X-Plane improvements (merge of iNavFlight#11253)
…ixes Needs testing: Issue iNavFlight#10830 Surface Mode Fixes
…FIX-Voltage-BUG change VBAT_SCALE_DEFAULT = 2100
…cmake-release-mode docs: Document CMAKE_BUILD_TYPE=Release for production builds
Contributor
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
Branch Targeting SuggestionYou've targeted the
If This is an automated suggestion to help route contributions to the appropriate branch. |
|
Test firmware build ready — commit Download firmware for PR #11582 235 targets built. Find your board's
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
In the servo mixer, the inout type "MAX" has been changed to "Fixed Value", because it's not always the max.