diff --git a/ROMFS/px4fmu_common/init.d/rcS b/ROMFS/px4fmu_common/init.d/rcS index 7d30d390ad37..ebdec2309226 100644 --- a/ROMFS/px4fmu_common/init.d/rcS +++ b/ROMFS/px4fmu_common/init.d/rcS @@ -36,6 +36,12 @@ set SDCARD_FORMAT no set STARTUP_TUNE 1 set VEHICLE_TYPE none +# Airframe parameter versioning +# Value set to 1 by default but can optionally be overridden in the airframe configuration startup script. +# Airframe maintainers can ensure a reset to the airframe defaults during an update by increasing by one. +# e.g. add line "set PARAM_DEFAULTS_VER 2" in your airframe file to build the first update that enfoces a reset. +set PARAM_DEFAULTS_VER 1 + # # Print full system version. # @@ -182,13 +188,11 @@ else netman update -i eth0 fi - # - # If the airframe has been previously reset SYS_AUTCONFIG will have been set to 1 and other params will be reset on the next boot. - # + # To trigger a parameter reset during boot SYS_AUTCONFIG was set to 1 before if param greater SYS_AUTOCONFIG 0 then - # Reset params except Airframe, RC calibration, sensor calibration, flight modes, total flight time, and next flight UUID. - param reset_all SYS_AUTOSTART RC* CAL_* COM_FLTMODE* LND_FLIGHT* TC_* COM_FLIGHT* + # Reset parameters except airframe, parameter version, RC calibration, sensor calibration, flight modes, total flight time, flight UUID + param reset_all SYS_AUTOSTART SYS_PARAM_VER RC* CAL_* COM_FLTMODE* LND_FLIGHT* TC_* COM_FLIGHT* fi # @@ -233,6 +237,17 @@ else fi unset AUTOSTART_PATH + # Check parameter version and reset upon airframe configuration version mismatch. + # Reboot required because "param reset_all" would reset all "param set" lines from airframe. + if ! param compare SYS_PARAM_VER ${PARAM_DEFAULTS_VER} + then + echo "Switched to different parameter version. Resetting parameters." + param set SYS_PARAM_VER ${PARAM_DEFAULTS_VER} + param set SYS_AUTOCONFIG 1 + param save + reboot + fi + # # Start the tone_alarm driver. # Needs to be started after the parameters are loaded (for CBRK_BUZZER). @@ -605,6 +620,7 @@ unset LOGGER_ARGS unset LOGGER_BUF unset PARAM_FILE unset PARAM_BACKUP_FILE +unset PARAM_DEFAULTS_VER unset RC_INPUT_ARGS unset SDCARD_AVAILABLE unset SDCARD_EXT_PATH diff --git a/src/lib/systemlib/system_params.c b/src/lib/systemlib/system_params.c index 23c0234380c9..1ca273be8473 100644 --- a/src/lib/systemlib/system_params.c +++ b/src/lib/systemlib/system_params.c @@ -83,6 +83,19 @@ PARAM_DEFINE_INT32(SYS_AUTOCONFIG, 0); */ PARAM_DEFINE_INT32(SYS_HITL, 0); +/** + * Parameter version + * + * This is used internally only: an airframe configuration might set an expected + * parameter version value via PARAM_DEFAULTS_VER. This is checked on bootup + * against SYS_PARAM_VER, and if they do not match, parameters are reset and + * reloaded from the airframe configuration. + * + * @min 0 + * @group System + */ +PARAM_DEFINE_INT32(SYS_PARAM_VER, 1); + /** * Enable auto start of rate gyro thermal calibration at the next power up. * diff --git a/src/modules/mavlink/mavlink_parameters.cpp b/src/modules/mavlink/mavlink_parameters.cpp index d5d774696d68..2d8c47da823d 100644 --- a/src/modules/mavlink/mavlink_parameters.cpp +++ b/src/modules/mavlink/mavlink_parameters.cpp @@ -310,6 +310,9 @@ MavlinkParametersManager::send() param_find("TRIG_MODE"); param_find("UAVCAN_ENABLE"); + // parameter only used in startup script but should show on ground station + param_find("SYS_PARAM_VER"); + _first_send = true; }