From 991c07eef9ddc25c062891d03a5ab13dd6de5ca8 Mon Sep 17 00:00:00 2001 From: Lorenz Meier Date: Mon, 12 Apr 2021 17:46:49 +0200 Subject: [PATCH] Global: Allow users to arm with USB This change reflects the medium-term experience with users using USB as data link or power source in R&D setups. While we still discourage this for volume products, not allowing it seems like a major hurdle and blocker in R&D setups. --- src/lib/circuit_breaker/circuit_breaker_params.c | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/lib/circuit_breaker/circuit_breaker_params.c b/src/lib/circuit_breaker/circuit_breaker_params.c index de34a6a01f..f3fde3c265 100644 --- a/src/lib/circuit_breaker/circuit_breaker_params.c +++ b/src/lib/circuit_breaker/circuit_breaker_params.c @@ -153,8 +153,12 @@ PARAM_DEFINE_INT32(CBRK_BUZZER, 0); * Circuit breaker for USB link check * * Setting this parameter to 197848 will disable the USB connected - * checks in the commander. - * WARNING: ENABLING THIS CIRCUIT BREAKER IS AT OWN RISK + * checks in the commander, setting it to 0 keeps them enabled (recommended). + * + * We are generally recommending to not fly with the USB link + * connected and production vehicles should set this parameter to + * zero to prevent users from flying USB powered. However, for R&D purposes + * it has proven over the years to work just fine. * * @reboot_required true * @min 0 @@ -162,7 +166,7 @@ PARAM_DEFINE_INT32(CBRK_BUZZER, 0); * @category Developer * @group Circuit Breaker */ -PARAM_DEFINE_INT32(CBRK_USB_CHK, 0); +PARAM_DEFINE_INT32(CBRK_USB_CHK, 197848); /** * Circuit breaker for position error check