Skip to content

Commit

Permalink
ArduSub: move serial_manager parameters up to base class
Browse files Browse the repository at this point in the history
  • Loading branch information
peterbarker committed Feb 28, 2024
1 parent 075e645 commit 7ad74a5
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 4 deletions.
13 changes: 10 additions & 3 deletions ArduSub/Parameters.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -63,9 +63,7 @@ const AP_Param::Info Sub::var_info[] = {
// @Increment: .5
GSCALAR(throttle_filt, "PILOT_THR_FILT", 0),

// @Group: SERIAL
// @Path: ../libraries/AP_SerialManager/AP_SerialManager.cpp
GOBJECT(serial_manager, "SERIAL", AP_SerialManager),
// AP_SerialManager was here

// @Param: GCS_PID_MASK
// @DisplayName: GCS PID tuning mask
Expand Down Expand Up @@ -775,6 +773,15 @@ void Sub::load_parameters()
#if HAL_LOGGING_ENABLED
AP_Param::convert_class(g.k_param_logger, &logger, logger.var_info, 0, true);
#endif

static const AP_Param::TopLevelObjectConversion toplevel_conversions[] {
#if AP_SERIALMANAGER_ENABLED
// PARAMETER_CONVERSION - Added: Feb-2024
{ &serial_manager, serial_manager.var_info, g.k_param_serial_manager_old },
#endif
};

AP_Param::convert_toplevel_objects(toplevel_conversions, ARRAY_SIZE(toplevel_conversions));
}

void Sub::convert_old_parameters()
Expand Down
2 changes: 1 addition & 1 deletion ArduSub/Parameters.h
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ class Parameters {
k_param_BoardConfig = 20, // Board configuration (Pixhawk/Linux/etc)
k_param_scheduler, // Scheduler (for debugging/perf_info)
k_param_logger, // AP_Logger Logging
k_param_serial_manager, // Serial ports, AP_SerialManager
k_param_serial_manager_old, // Serial ports, AP_SerialManager
k_param_notify, // Notify Library, AP_Notify
k_param_arming = 26, // Arming checks
k_param_can_mgr,
Expand Down

0 comments on commit 7ad74a5

Please sign in to comment.