Skip to content

Revert accidental merge of maintenance-10.x into maintenance-9.x#11606

Merged
sensei-hacker merged 4 commits into
iNavFlight:maintenance-9.xfrom
sensei-hacker:fix/revert-bad-merge
May 31, 2026
Merged

Revert accidental merge of maintenance-10.x into maintenance-9.x#11606
sensei-hacker merged 4 commits into
iNavFlight:maintenance-9.xfrom
sensei-hacker:fix/revert-bad-merge

Conversation

@sensei-hacker
Copy link
Copy Markdown
Member

@sensei-hacker sensei-hacker commented May 31, 2026

Summary

On 2026-05-24, commit 10343b4de1 accidentally merged the entire maintenance-10.x branch into maintenance-9.x, contaminating it with DroneCAN, CRSF sensor input, Output Assignment API v2, and 28,393 build/library artifacts that should not be present in 9.x.

This PR reverts that contamination via 4 commits structured for clean review:

Commit Files What it does
900a88b — cherry-pick 27,143 Removes build_hw/ CMake artifacts + adds to .gitignore
7a9b7bb — lib/ revert 919 Removes DroneCAN DSDLC lib, reverts STM32F4/F7/H7 HAL to 9.x versions
2db9595 — stm32cubef7 revert 319 Removes STM32CubeF7 SDK extraction files
d5ac9bb — code revert ~156 Reverts src/, cmake/, docs/ — the actual firmware code

The first three commits are mechanical reversions that require no line-by-line review. The fourth commit is the one that needs scrutiny.

What is preserved

All legitimate 9.x work added after the bad merge is fully preserved:

Verification

  • git diff bb61218284 HEAD -- src/main/flight/mixer.h → no delta (clean base)
  • git diff bb61218284 HEAD -- src/main/fc/fc_msp.c → +3 lines (programmingPidInit() from PR Reset free PIDs if coefficients are changed via CLI or MSP #11431 only)
  • No DroneCAN, CRSF sensor, Output Assignment API, or build_hw/ artifacts
  • Builds cleanly: MATEKH743 (H7) and MATEKF405SE (F4) ✓

Context

A clean release/9.1 branch also exists (branched from bb61218284 with all legitimate work cherry-picked).
This PR cleans up maintenance-9.x itself so contributors who branch from it are not working against contaminated code.

daijoubu and others added 4 commits May 30, 2026 19:03
…nance-10.x

Removes the DroneCAN DSDLC library (lib/main/Dronecan -- entirely new in 10.x)
and reverts STM32F4/F7/H7 HAL libraries to their maintenance-9.x versions.
932 files. No post-merge commits touched lib/ -- this is a clean mechanical revert.
Reverts 319 STM32CubeF7 SDK extraction files introduced by the accidental
maintenance-10.x merge. No post-merge commits touched this directory --
this is a clean mechanical revert.
This reverts commit 10343b4, reversing
changes made to bb61218.
@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

Test firmware build ready — commit d5ac9bb

Download firmware for PR #11606

238 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.

@sensei-hacker sensei-hacker merged commit b0c5ecf into iNavFlight:maintenance-9.x May 31, 2026
24 checks 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.

2 participants