AP_NavEKF2: EKF2 prefix for prearm failures and camelCase

This commit is contained in:
Mark Whitehorn 2019-10-30 09:25:23 -06:00 committed by Andrew Tridgell
parent 95ea6ad807
commit 7c6630e216
2 changed files with 18 additions and 20 deletions

View File

@ -621,10 +621,10 @@ void NavEKF2::check_log_write(void)
// Initialise the filter
bool NavEKF2::InitialiseFilter(void)
{
init_failure = INIT_FAILURES::UNKNOWN;
initFailure = InitFailures::UNKNOWN;
if (_enable == 0) {
if (_ahrs->get_ekf_type() == 2) {
init_failure = INIT_FAILURES::NO_ENABLE;
initFailure = InitFailures::NO_ENABLE;
}
return false;
}
@ -648,11 +648,11 @@ bool NavEKF2::InitialiseFilter(void)
// this prevents dereferencing a null _ahrs pointer in NavEKF2_core::getEulerAngles
if (ins.get_accel_count() == 0) {
init_failure = INIT_FAILURES::NO_IMUS;
initFailure = InitFailures::NO_IMUS;
return false;
}
if (_imuMask == 0) {
init_failure = INIT_FAILURES::NO_MASK;
initFailure = InitFailures::NO_MASK;
return false;
}
@ -665,7 +665,7 @@ bool NavEKF2::InitialiseFilter(void)
// check if there is enough memory to create the EKF cores
if (hal.util->available_memory() < sizeof(NavEKF2_core)*num_cores + 4096) {
init_failure = INIT_FAILURES::NO_MEM;
initFailure = InitFailures::NO_MEM;
gcs().send_text(MAV_SEVERITY_CRITICAL, "NavEKF2: not enough memory");
_enable.set(0);
return false;
@ -675,7 +675,7 @@ bool NavEKF2::InitialiseFilter(void)
core = (NavEKF2_core*)hal.util->malloc_type(sizeof(NavEKF2_core)*num_cores, AP_HAL::Util::MEM_FAST);
if (core == nullptr) {
_enable.set(0);
init_failure = INIT_FAILURES::NO_ALLOC;
initFailure = InitFailures::NO_ALLOC;
gcs().send_text(MAV_SEVERITY_CRITICAL, "NavEKF2: allocation failed");
return false;
}
@ -1466,7 +1466,7 @@ uint32_t NavEKF2::getLastVelNorthEastReset(Vector2f &vel) const
const char *NavEKF2::prearm_failure_reason(void) const
{
if (!core) {
return init_failure_reason[int(init_failure)];
return initFailureReason[int(initFailure)];
}
for (uint8_t i = 0; i < num_cores; i++) {
const char * failure = core[i].prearm_failure_reason();

View File

@ -493,27 +493,25 @@ private:
// time of last lane switch
uint32_t lastLaneSwitch_ms;
enum class INIT_FAILURES {
OK,
NO_MEM,
enum class InitFailures {
UNKNOWN,
NO_ENABLE,
NO_IMUS,
NO_MASK,
NO_MEM,
NO_ALLOC,
UNKNOWN,
NUM_INIT_FAILURES
};
// initialization failure reasons
const char* init_failure_reason[int(INIT_FAILURES::NUM_INIT_FAILURES)] {
"OK",
"insufficient memory available",
"EK2_enable is false",
"no IMUs available",
"EK2_IMU_MASK is zero",
"memory allocation failed",
"unknown initialization failure"
const char* initFailureReason[int(InitFailures::NUM_INIT_FAILURES)] {
"EKF2: unknown initialization failure",
"EKF2: EK2_enable is false",
"EKF2: no IMUs available",
"EKF2: EK2_IMU_MASK is zero",
"EKF2: insufficient memory available",
"EKF2: memory allocation failed"
};
INIT_FAILURES init_failure;
InitFailures initFailure;
// update the yaw reset data to capture changes due to a lane switch
// new_primary - index of the ekf instance that we are about to switch to as the primary