Skip to content

Comments

Move RC protocol used for Aero from Linux HAL to AP_RCProtocol#29461

Open
peterbarker wants to merge 4 commits intoArduPilot:masterfrom
peterbarker:pr/sololink-to-rcprotocol
Open

Move RC protocol used for Aero from Linux HAL to AP_RCProtocol#29461
peterbarker wants to merge 4 commits intoArduPilot:masterfrom
peterbarker:pr/sololink-to-rcprotocol

Conversation

@peterbarker
Copy link
Contributor

MAVProxy PR for testing is here: ArduPilot/MAVProxy#1520

... near as I can tell both old and new code fails to lock against race conditions updating the RC data.

@peterbarker
Copy link
Contributor Author

Only compiler output change is aero:

Every 2.0s: ~/rc/ardupilot/Tools/scripts/filter_size_compare_branches_csv.py --hide /tmp/some.csv  ; wc /tmp/some.csv                                        threads: Sun Mar  9 18:39:51 2025

------------------------  -----  ------  -----  -----  -----  -----
Board                     blimp  copter  heli   plane  rover  sub
SITL_arm_linux_gnueabihf  0      0       0      0      0      0
SITL_x86_64_linux_gnu     0      0       0      0      0      0
aero                      66516  62280   66456  62248  62276  66564
------------------------  -----  ------  -----  -----  -----  -----
  361   361 10005 /tmp/some.csv

@peterbarker peterbarker force-pushed the pr/sololink-to-rcprotocol branch from 89b3281 to f91a5d5 Compare October 24, 2025 04:22
@rmackay9
Copy link
Contributor

I always say this kind of thing but I think the Intel Aero is long gone and it was never very popular. If it's causing any kind of support overhead, I vote for removing it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants