Skip to content

feat(ubx): add support for UBX_MSG_MON_SPAN and full support of UBX_MON_RF - #222

Merged
ThomasRigi merged 2 commits into
mainfrom
pr-mon-span
Sep 9, 2026
Merged

ThomasRigi merged 2 commits into
mainfrom
pr-mon-span

Conversation

@ThomasRigi

@ThomasRigi ThomasRigi commented Aug 6, 2026

Copy link
Copy Markdown
Member

Updated version of the submodule for supporting UBX_MSG_MON_SPAN

See PX4/PX4-Autopilot#20474 for the initial version by @dagar .

Main PR: PX4/PX4-Autopilot#28173

@ThomasRigi
ThomasRigi force-pushed the pr-mon-span branch 2 times, most recently from 36688e8 to 17e8997 Compare August 6, 2026 15:15
Comment thread src/ubx.cpp Outdated
Comment thread src/ubx.cpp
@ThomasRigi
ThomasRigi requested review from dagar and dakejahl August 6, 2026 15:55
Comment thread src/ubx.cpp Outdated
Comment thread src/ubx.cpp Outdated
Comment thread src/ubx.cpp Outdated
@ThomasRigi ThomasRigi changed the title feat(ubx): add support for UBX_MSG_MON_SPAN feat(ubx): add support for UBX_MSG_MON_SPAN and full support of UBX_MON_RF Aug 11, 2026
Comment thread src/ubx.cpp Outdated
Comment thread src/ubx.cpp Outdated
@dakejahl

Copy link
Copy Markdown
Contributor

needs a rebase

Comment thread src/ubx.h

@dakejahl dakejahl left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice looks good, yeah let's add the fix for UART1_ENABLED too

Comment thread src/ubx.cpp Outdated
Comment thread src/ubx.h Outdated
Comment thread src/ubx.h
Comment thread src/ubx.cpp
Comment thread src/ubx.cpp
Comment thread src/ubx.h
@ThomasRigi
ThomasRigi force-pushed the pr-mon-span branch 2 times, most recently from 1c9d75d to cfcda14 Compare September 8, 2026 17:32
@ThomasRigi

Copy link
Copy Markdown
Member Author

I rebased and squashed my commits for a cleaner history. From my tests on an F9P it's all good. @dakejahl I'm happy if you can give it a go on an L1/L5 and X20 receiver. Will report with more testing details in the main PR.

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown

ABI/API Compatibility Report

Status: ⚠️ Breaking changes detected

Details

�[0;31m[ERROR]�[0m Missing dependencies: 
Install with: sudo apt-get install 

This comment is automatically updated on each push.

@ThomasRigi
ThomasRigi merged commit 928062a into main Sep 9, 2026
5 checks passed
@ThomasRigi
ThomasRigi deleted the pr-mon-span branch September 9, 2026 09:42
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