Mirror/ardupilot
5
0
Fork 0
mirror of https://github.com/ArduPilot/ardupilot synced 2025-01-23 09:08:30 -04:00
Code Issues Packages Projects Releases Wiki Activity

Compass: use constructor to set initial values for _learn and _use_for_yaw

Browse Source 
this ensures they are set if you have MAG_ENABLE==0
This commit is contained in:
Andrew Tridgell 2012-02-25 17:34:33 +11:00
parent 900388a85a
commit d0a6359b37
2 changed files with 2 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
libraries/AP_Compass/AP_Compass_HMC5843.cpp
View File

@ -138,9 +138,6 @@ AP_Compass_HMC5843::init()
uint16_t expected_yz = 715;
float gain_multiple = 1.0;
// call the parent class init for common code
Compass::init();
delay(10);
// determine if we are using 5843 or 5883L

12
libraries/AP_Compass/Compass.cpp
View File

@ -18,6 +18,8 @@ Compass::Compass(void) :
_declination (0.0),
_null_init_done(false),
_null_enable(false),
_learn(1),
_use_for_yaw(1),
product_id(AP_COMPASS_TYPE_UNKNOWN)
{
// Default the orientation matrix to none - will be overridden at group load time
@ -25,21 +27,11 @@ Compass::Compass(void) :
_orientation_matrix.set(ROTATION_NONE);
}
//_group
// Default init method, just returns success.
//
bool
Compass::init()
{
// enable learning by default
if (!_learn.load()) {
_learn.set(1);
}
// enable use for yaw calculation by default
if (!_use_for_yaw.load()) {
_use_for_yaw.set(1);
}
return true;
}