From b042f2101f7b77fb46757fedfa60189a27a58be7 Mon Sep 17 00:00:00 2001 From: Matthias Grob Date: Wed, 8 Jan 2025 14:29:47 +0100 Subject: [PATCH] system_params: clarify SYS_HAS_MAG description after answering the questions: Ah the value can be 2? Should I set the number of magnetometers the board has or include the external ones? --- src/lib/systemlib/system_params.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/src/lib/systemlib/system_params.c b/src/lib/systemlib/system_params.c index 1ca273be8473..d396039f3f16 100644 --- a/src/lib/systemlib/system_params.c +++ b/src/lib/systemlib/system_params.c @@ -189,11 +189,10 @@ PARAM_DEFINE_INT32(SYS_CAL_TMAX, 10); PARAM_DEFINE_INT32(SYS_HAS_GPS, 1); /** - * Control if the vehicle has a magnetometer + * Control if and how many magnetometers are expected * - * Set this to 0 if the board has no magnetometer. - * If set to 0, the preflight checks will not check for the presence of a - * magnetometer, otherwise N sensors are required. + * 0: System has no magnetometer, preflight checks should pass without one. + * 1-N: Require the presence of N magnetometer sensors for check to pass. * * @reboot_required true * @group System