Skip to content

docs: add MSP message routing architecture reference#11734

Merged
sensei-hacker merged 1 commit into
iNavFlight:maintenance-10.xfrom
sensei-hacker:docs-extract-msp
Jul 22, 2026
Merged

docs: add MSP message routing architecture reference#11734
sensei-hacker merged 1 commit into
iNavFlight:maintenance-10.xfrom
sensei-hacker:docs-extract-msp

Conversation

@sensei-hacker

Copy link
Copy Markdown
Member

Summary

  • New reference doc explaining how MSP commands are routed through mspFcProcessCommand()'s four handler functions (mspFcProcessOutCommand, mspFcProcessInCommand, mspFCProcessInOutCommand, and direct handling in the main router) in fc_msp.c
  • Includes the routing decision tree, INOUT-then-IN fallback order, and guidance on which handler to extend when adding a new MSP command
  • All code excerpts and "typical commands" categorizations verified against current fc_msp.c source

Test plan

  • Docs-only change, no code affected
  • Verified all function signatures, routing order, and code excerpts against current src/main/fc/fc_msp.c
  • Ran inav-code-review; one inaccurate code excerpt (MSP_SET_HEAD) was caught and corrected to match source verbatim

Documents how MSP commands are routed through mspFcProcessCommand's
four handler functions (OUT/IN/INOUT/direct) in fc_msp.c, including
the routing decision tree and guidance on which handler to extend
when adding a new MSP command.
@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 →

@sensei-hacker
sensei-hacker merged commit 5d735db into iNavFlight:maintenance-10.x Jul 22, 2026
1 check failed
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