Skip to content

Release/9.1 to master#11662

Merged
sensei-hacker merged 12 commits into
masterfrom
release/9.1
Jun 18, 2026
Merged

Release/9.1 to master#11662
sensei-hacker merged 12 commits into
masterfrom
release/9.1

Conversation

@sensei-hacker

Copy link
Copy Markdown
Member

No description provided.

sensei-hacker and others added 12 commits June 7, 2026 00:34
dmavar=0 maps to DMA1 Stream 0, which is already claimed by M1 (TIM8_CH2).
When DSHOT is active, the LED strip silently fails to initialize because
its DMA stream is already owned. Change to dmavar=8 (DMA2 Stream 0),
matching the pattern used by other H7 targets (MATEKH743, MAMBAH743, etc.).
AEDROXH7: fix LED strip DMA conflict with motor outputs
The fixed-wing waypoint cross-track controller (nav_fw_wp_tracking_accuracy) kept
its rate/error state in function-local statics that were never reset. After a turn
or loiter, where the controller is disengaged (needToCalculateCircularLoiter), the
first sample on the next leg was computed from stale data from the previous leg,
causing a small heading twitch when rejoining the course line.

Re-seed the controller state to the current cross-track error while it is not
actively steering, so it re-engages cleanly on the next leg.
When a WP mission JUMP activated its target waypoint, isWaypointReached() could read a wpTurnSmoothingActive flag left over from the previous waypoint. The JUMP defers target setup by one navigation loop, and in that gap the FW position controller re-arms the flag from the previous waypoint's geometry. With nav_fw_wp_turn_smoothing = CUT this marks the jump target as reached on its first loop (and cascades the following WP via the bearing-miss check), so the mission jumps to target+1 instead of the target.

Reset wpTurnSmoothingActive when a new active waypoint is set so a freshly activated waypoint cannot inherit the previous waypoint's smoothing-reached state. Turn smoothing is otherwise unchanged: the FW controller recomputes the flag every position update from the new waypoint geometry.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Fixed wing nav: reset WP cross-track tracking state on re-engagement
FW nav: fix WP JUMP target skipped by stale turn-smoothing flag
Removed a link to a specific OLED display from the documentation.
Resolves #11660
Remove outdated OLED display link from Display.md
@qodo-code-review

Copy link
Copy Markdown
Contributor

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@github-actions

Copy link
Copy Markdown

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.

@sensei-hacker sensei-hacker merged commit 4939a7f into master Jun 18, 2026
24 checks passed
@github-actions

Copy link
Copy Markdown

Test firmware build ready — commit 2ca81dd

Download firmware for PR #11662

240 targets built. Find your board's .hex file by name on that page (e.g. MATEKF405SE.hex). Files are individually downloadable — no GitHub login required.

Development build for testing only. Use Full Chip Erase when flashing.

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.

4 participants