Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AP_RangeFinder: compile mavlink rangefinder in only if HAL_GCS_ENABLED #24497

Conversation

peterbarker
Copy link
Contributor

Board,AP_Periph
f303-Universal,-344

Before:

134295144 00000002 T AP_GPS_Backend::send_mavlink_gps_rtk(mavlink_channel_t)
134252500 00000032 T mavlink_frame_char
134251656 00000036 T mavlink_update_checksum
134251692 00000064 T mavlink_get_msg_entry
134252532 00000064 T mavlink_parse_char
134250652 00000080 T mavlink_sha256_init(mavlink_sha256_ctx*)
134313220 00000090 T AP_RangeFinder_MAVLink::handle_msg(__mavlink_message const&)
134251180 00000128 T mavlink_sha256_final_48(mavlink_sha256_ctx*, unsigned char*)
134408816 00000256 t mavlink_sha256_constant_256
134251308 00000348 T mavlink_signature_check(__mavlink_signing*, __mavlink_signing_streams*, __mavlink_message const*)
134252652 00000380 T AP_Periph_FW::can_send_ADSB(__mavlink_adsb_vehicle_t&)
134250732 00000448 T mavlink_sha256_update(mavlink_sha256_ctx*, void const*, unsigned long)
134251756 00000744 T mavlink_frame_char_buffer
134409072 00003996 t mavlink_get_msg_entry::mavlink_message_crcs

After:

pbarker@fx:~/rc/ardupilot((HEAD detached from 0c48652bb2d))$ arm-nm build/f303-Universal/bin/AP_Periph | grep mavlink
134295144 00000002 T AP_GPS_Backend::send_mavlink_gps_rtk(mavlink_channel_t)
134252500 00000032 T mavlink_frame_char
134251656 00000036 T mavlink_update_checksum
134251692 00000064 T mavlink_get_msg_entry
134252532 00000064 T mavlink_parse_char
134250652 00000080 T mavlink_sha256_init(mavlink_sha256_ctx*)
134251180 00000128 T mavlink_sha256_final_48(mavlink_sha256_ctx*, unsigned char*)
134408520 00000256 t mavlink_sha256_constant_256
134251308 00000348 T mavlink_signature_check(__mavlink_signing*, __mavlink_signing_streams*, __mavlink_message const*)
134252652 00000380 T AP_Periph_FW::can_send_ADSB(__mavlink_adsb_vehicle_t&)
134250732 00000448 T mavlink_sha256_update(mavlink_sha256_ctx*, void const*, unsigned long)
134251756 00000744 T mavlink_frame_char_buffer
134408776 00003996 t mavlink_get_msg_entry::mavlink_message_crcs
pbarker@fx:~/rc/ardupilot((HEAD detached from 0c48652bb2d))$ 

@peterbarker peterbarker force-pushed the pr/rangefinder-mavlink-requires-hal-gcs-enabled branch from 0a0cdec to f12c614 Compare August 3, 2023 22:00
@peterbarker
Copy link
Contributor Author

Board               AP_Periph  blimp  bootloader  copter  heli  iofirmware  plane  rover  sub
Durandal                       *      *           *       *                 *      *      *
HerePro             *                                                                     
Hitec-Airspeed      *                                                                     
KakuteH7-bdshot                *      *           *       *                 *      *      *
MatekF405                      *      *           *       *                 *      *      *
Pixhawk1-1M-bdshot             *                  *       *                 *      *      *
f103-QiotekPeriph   *                                                                     
f303-Universal      -344                                                                  
iomcu                                                           *                         
revo-mini                      *      *           *       *                 *      *      *
skyviper-v2450                                    *                                       

@tridge tridge merged commit 5670b8a into ArduPilot:master Aug 8, 2023
81 checks passed
@peterbarker peterbarker deleted the pr/rangefinder-mavlink-requires-hal-gcs-enabled branch August 25, 2023 23:15
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.

None yet

3 participants