altos: Use stdbool true/false instead of TRUE/FALSE
[fw/altos] / src / drivers / ao_mpu6000.c
index 5e75b78a13787e8546235bcaeeb44de4c45e0a41..8c85ab01fd82c00041128a9147e3f6a26f2caa39 100644 (file)
@@ -3,7 +3,8 @@
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
  *
  * This program is distributed in the hope that it will be useful, but
  * WITHOUT ANY WARRANTY; without even the implied warranty of
@@ -21,7 +22,6 @@
 
 #if HAS_MPU6000
 
-static uint8_t ao_mpu6000_wake;
 static uint8_t ao_mpu6000_configured;
 
 #ifndef AO_MPU6000_I2C_INDEX
@@ -55,7 +55,7 @@ _ao_mpu6000_reg_write(uint8_t addr, uint8_t value)
 #else
        ao_i2c_get(AO_MPU6000_I2C_INDEX);
        ao_i2c_start(AO_MPU6000_I2C_INDEX, MPU6000_ADDR_WRITE);
-       ao_i2c_send(d, 2, AO_MPU6000_I2C_INDEX, TRUE);
+       ao_i2c_send(d, 2, AO_MPU6000_I2C_INDEX, true);
        ao_i2c_put(AO_MPU6000_I2C_INDEX);
 #endif
 }
@@ -72,9 +72,9 @@ _ao_mpu6000_read(uint8_t addr, void *data, uint8_t len)
 #else
        ao_i2c_get(AO_MPU6000_I2C_INDEX);
        ao_i2c_start(AO_MPU6000_I2C_INDEX, MPU6000_ADDR_WRITE);
-       ao_i2c_send(&addr, 1, AO_MPU6000_I2C_INDEX, FALSE);
+       ao_i2c_send(&addr, 1, AO_MPU6000_I2C_INDEX, false);
        ao_i2c_start(AO_MPU6000_I2C_INDEX, MPU6000_ADDR_READ);
-       ao_i2c_recv(data, len, AO_MPU6000_I2C_INDEX, TRUE);
+       ao_i2c_recv(data, len, AO_MPU6000_I2C_INDEX, true);
        ao_i2c_put(AO_MPU6000_I2C_INDEX);
 #endif
 }
@@ -92,9 +92,9 @@ _ao_mpu6000_reg_read(uint8_t addr)
 #else
        ao_i2c_get(AO_MPU6000_I2C_INDEX);
        ao_i2c_start(AO_MPU6000_I2C_INDEX, MPU6000_ADDR_WRITE);
-       ao_i2c_send(&addr, 1, AO_MPU6000_I2C_INDEX, FALSE);
+       ao_i2c_send(&addr, 1, AO_MPU6000_I2C_INDEX, false);
        ao_i2c_start(AO_MPU6000_I2C_INDEX, MPU6000_ADDR_READ);
-       ao_i2c_recv(&value, 1, AO_MPU6000_I2C_INDEX, TRUE);
+       ao_i2c_recv(&value, 1, AO_MPU6000_I2C_INDEX, true);
        ao_i2c_put(AO_MPU6000_I2C_INDEX);
 #endif
        return value;
@@ -133,30 +133,30 @@ ao_mpu6000_gyro(int16_t v)
 #endif
 
 static uint8_t
-ao_mpu6000_accel_check(int16_t normal, int16_t test, char *which)
+ao_mpu6000_accel_check(int16_t normal, int16_t test)
 {
        int16_t diff = test - normal;
 
-       if (diff < MPU6000_ST_ACCEL(16) / 2) {
+       if (diff < MPU6000_ST_ACCEL(16) / 4) {
                return 1;
        }
-       if (diff > MPU6000_ST_ACCEL(16) * 2) {
+       if (diff > MPU6000_ST_ACCEL(16) * 4) {
                return 1;
        }
        return 0;
 }
 
 static uint8_t
-ao_mpu6000_gyro_check(int16_t normal, int16_t test, char *which)
+ao_mpu6000_gyro_check(int16_t normal, int16_t test)
 {
        int16_t diff = test - normal;
 
        if (diff < 0)
                diff = -diff;
-       if (diff < MPU6000_ST_GYRO(2000) / 2) {
+       if (diff < MPU6000_ST_GYRO(2000) / 4) {
                return 1;
        }
-       if (diff > MPU6000_ST_GYRO(2000) * 2) {
+       if (diff > MPU6000_ST_GYRO(2000) * 4) {
                return 1;
        }
        return 0;
@@ -177,11 +177,14 @@ _ao_mpu6000_wait_alive(void)
                ao_panic(AO_PANIC_SELF_TEST_MPU6000);
 }
 
+#define ST_TRIES       10
+
 static void
 _ao_mpu6000_setup(void)
 {
        struct ao_mpu6000_sample        normal_mode, test_mode;
-       int                             errors =0;
+       int                             errors;
+       int                             st_tries;
 
        if (ao_mpu6000_configured)
                return;
@@ -189,7 +192,7 @@ _ao_mpu6000_setup(void)
        _ao_mpu6000_wait_alive();
 
        /* Reset the whole chip */
-       
+
        _ao_mpu6000_reg_write(MPU6000_PWR_MGMT_1,
                              (1 << MPU6000_PWR_MGMT_1_DEVICE_RESET));
 
@@ -236,23 +239,6 @@ _ao_mpu6000_setup(void)
                              (MPU6000_CONFIG_EXT_SYNC_SET_DISABLED << MPU6000_CONFIG_EXT_SYNC_SET) |
                              (MPU6000_CONFIG_DLPF_CFG_260_256 << MPU6000_CONFIG_DLPF_CFG));
 
-       /* Configure accelerometer to +/-16G in self-test mode */
-       _ao_mpu6000_reg_write(MPU6000_ACCEL_CONFIG,
-                             (1 << MPU600_ACCEL_CONFIG_XA_ST) |
-                             (1 << MPU600_ACCEL_CONFIG_YA_ST) |
-                             (1 << MPU600_ACCEL_CONFIG_ZA_ST) |
-                             (MPU600_ACCEL_CONFIG_AFS_SEL_16G << MPU600_ACCEL_CONFIG_AFS_SEL));
-
-       /* Configure gyro to +/- 2000°/s in self-test mode */
-       _ao_mpu6000_reg_write(MPU6000_GYRO_CONFIG,
-                             (1 << MPU600_GYRO_CONFIG_XG_ST) |
-                             (1 << MPU600_GYRO_CONFIG_YG_ST) |
-                             (1 << MPU600_GYRO_CONFIG_ZG_ST) |
-                             (MPU600_GYRO_CONFIG_FS_SEL_2000 << MPU600_GYRO_CONFIG_FS_SEL));
-
-       ao_delay(AO_MS_TO_TICKS(200));
-       _ao_mpu6000_sample(&test_mode);
-
 #if TRIDGE
        // read the product ID rev c has 1/2 the sensitivity of rev d
        _mpu6000_product_id = _register_read(MPUREG_PRODUCT_ID);
@@ -268,36 +254,58 @@ _ao_mpu6000_setup(void)
                register_write(MPUREG_ACCEL_CONFIG,2<<3);
        }
        hal.scheduler->delay(1);
-
 #endif
 
-       /* Configure accelerometer to +/-16G */
-       _ao_mpu6000_reg_write(MPU6000_ACCEL_CONFIG,
-                             (0 << MPU600_ACCEL_CONFIG_XA_ST) |
-                             (0 << MPU600_ACCEL_CONFIG_YA_ST) |
-                             (0 << MPU600_ACCEL_CONFIG_ZA_ST) |
-                             (MPU600_ACCEL_CONFIG_AFS_SEL_16G << MPU600_ACCEL_CONFIG_AFS_SEL));
-
-       /* Configure gyro to +/- 2000°/s */
-       _ao_mpu6000_reg_write(MPU6000_GYRO_CONFIG,
-                             (0 << MPU600_GYRO_CONFIG_XG_ST) |
-                             (0 << MPU600_GYRO_CONFIG_YG_ST) |
-                             (0 << MPU600_GYRO_CONFIG_ZG_ST) |
-                             (MPU600_GYRO_CONFIG_FS_SEL_2000 << MPU600_GYRO_CONFIG_FS_SEL));
-
-       ao_delay(AO_MS_TO_TICKS(10));
-       _ao_mpu6000_sample(&normal_mode);
-       
-       errors += ao_mpu6000_accel_check(normal_mode.accel_x, test_mode.accel_x, "x");
-       errors += ao_mpu6000_accel_check(normal_mode.accel_y, test_mode.accel_y, "y");
-       errors += ao_mpu6000_accel_check(normal_mode.accel_z, test_mode.accel_z, "z");
-
-       errors += ao_mpu6000_gyro_check(normal_mode.gyro_x, test_mode.gyro_x, "x");
-       errors += ao_mpu6000_gyro_check(normal_mode.gyro_y, test_mode.gyro_y, "y");
-       errors += ao_mpu6000_gyro_check(normal_mode.gyro_z, test_mode.gyro_z, "z");
-
-       if (errors)
-               ao_panic(AO_PANIC_SELF_TEST_MPU6000);
+       for (st_tries = 0; st_tries < ST_TRIES; st_tries++) {
+               errors = 0;
+
+               /* Configure accelerometer to +/-16G in self-test mode */
+               _ao_mpu6000_reg_write(MPU6000_ACCEL_CONFIG,
+                                     (1 << MPU600_ACCEL_CONFIG_XA_ST) |
+                                     (1 << MPU600_ACCEL_CONFIG_YA_ST) |
+                                     (1 << MPU600_ACCEL_CONFIG_ZA_ST) |
+                                     (MPU600_ACCEL_CONFIG_AFS_SEL_16G << MPU600_ACCEL_CONFIG_AFS_SEL));
+
+               /* Configure gyro to +/- 2000°/s in self-test mode */
+               _ao_mpu6000_reg_write(MPU6000_GYRO_CONFIG,
+                                     (1 << MPU600_GYRO_CONFIG_XG_ST) |
+                                     (1 << MPU600_GYRO_CONFIG_YG_ST) |
+                                     (1 << MPU600_GYRO_CONFIG_ZG_ST) |
+                                     (MPU600_GYRO_CONFIG_FS_SEL_2000 << MPU600_GYRO_CONFIG_FS_SEL));
+
+               ao_delay(AO_MS_TO_TICKS(200));
+               _ao_mpu6000_sample(&test_mode);
+
+               /* Configure accelerometer to +/-16G */
+               _ao_mpu6000_reg_write(MPU6000_ACCEL_CONFIG,
+                                     (0 << MPU600_ACCEL_CONFIG_XA_ST) |
+                                     (0 << MPU600_ACCEL_CONFIG_YA_ST) |
+                                     (0 << MPU600_ACCEL_CONFIG_ZA_ST) |
+                                     (MPU600_ACCEL_CONFIG_AFS_SEL_16G << MPU600_ACCEL_CONFIG_AFS_SEL));
+
+               /* Configure gyro to +/- 2000°/s */
+               _ao_mpu6000_reg_write(MPU6000_GYRO_CONFIG,
+                                     (0 << MPU600_GYRO_CONFIG_XG_ST) |
+                                     (0 << MPU600_GYRO_CONFIG_YG_ST) |
+                                     (0 << MPU600_GYRO_CONFIG_ZG_ST) |
+                                     (MPU600_GYRO_CONFIG_FS_SEL_2000 << MPU600_GYRO_CONFIG_FS_SEL));
+
+               ao_delay(AO_MS_TO_TICKS(200));
+               _ao_mpu6000_sample(&normal_mode);
+
+               errors += ao_mpu6000_accel_check(normal_mode.accel_x, test_mode.accel_x);
+               errors += ao_mpu6000_accel_check(normal_mode.accel_y, test_mode.accel_y);
+               errors += ao_mpu6000_accel_check(normal_mode.accel_z, test_mode.accel_z);
+
+               errors += ao_mpu6000_gyro_check(normal_mode.gyro_x, test_mode.gyro_x);
+               errors += ao_mpu6000_gyro_check(normal_mode.gyro_y, test_mode.gyro_y);
+               errors += ao_mpu6000_gyro_check(normal_mode.gyro_z, test_mode.gyro_z);
+               if (!errors)
+                       break;
+       }
+
+       if (st_tries == ST_TRIES)
+               ao_sensor_errors = 1;
 
        /* Filter to about 100Hz, which also sets the gyro rate to 1000Hz */
        _ao_mpu6000_reg_write(MPU6000_CONFIG,
@@ -307,7 +315,7 @@ _ao_mpu6000_setup(void)
        /* Set sample rate divider to sample at 200Hz (v = gyro/rate - 1) */
        _ao_mpu6000_reg_write(MPU6000_SMPRT_DIV,
                              1000 / 200 - 1);
-       
+
        ao_delay(AO_MS_TO_TICKS(100));
        ao_mpu6000_configured = 1;
 }
@@ -317,6 +325,7 @@ struct ao_mpu6000_sample    ao_mpu6000_current;
 static void
 ao_mpu6000(void)
 {
+       struct ao_mpu6000_sample        sample;
        /* ao_mpu6000_init already grabbed the SPI bus and mutex */
        _ao_mpu6000_setup();
 #if AO_MPU6000_SPI
@@ -327,14 +336,15 @@ ao_mpu6000(void)
 #if AO_MPU6000_SPI
                ao_mpu6000_spi_get();
 #endif
-               _ao_mpu6000_sample(&ao_mpu6000_current);
+               _ao_mpu6000_sample(&sample);
 #if AO_MPU6000_SPI
                ao_mpu6000_spi_put();
-#endif 
-               ao_arch_critical(
-                       AO_DATA_PRESENT(AO_DATA_MPU6000);
-                       AO_DATA_WAIT();
-                       );
+#endif
+               ao_arch_block_interrupts();
+               ao_mpu6000_current = sample;
+               AO_DATA_PRESENT(AO_DATA_MPU6000);
+               AO_DATA_WAIT();
+               ao_arch_release_interrupts();
        }
 }
 
@@ -343,16 +353,13 @@ static struct ao_task ao_mpu6000_task;
 static void
 ao_mpu6000_show(void)
 {
-       struct ao_data  sample;
-
-       ao_data_get(&sample);
        printf ("Accel: %7d %7d %7d Gyro: %7d %7d %7d\n",
-               sample.mpu6000.accel_x,
-               sample.mpu6000.accel_y,
-               sample.mpu6000.accel_z,
-               sample.mpu6000.gyro_x,
-               sample.mpu6000.gyro_y,
-               sample.mpu6000.gyro_z);
+               ao_mpu6000_current.accel_x,
+               ao_mpu6000_current.accel_y,
+               ao_mpu6000_current.accel_z,
+               ao_mpu6000_current.gyro_x,
+               ao_mpu6000_current.gyro_y,
+               ao_mpu6000_current.gyro_z);
 }
 
 static const struct ao_cmds ao_mpu6000_cmds[] = {
@@ -366,7 +373,7 @@ ao_mpu6000_init(void)
        ao_mpu6000_configured = 0;
 
        ao_add_task(&ao_mpu6000_task, ao_mpu6000, "mpu6000");
-       
+
 #if AO_MPU6000_SPI
        ao_spi_init_cs(AO_MPU6000_SPI_CS_PORT, (1 << AO_MPU6000_SPI_CS_PIN));
 
@@ -378,7 +385,7 @@ ao_mpu6000_init(void)
        ao_cur_task = &ao_mpu6000_task;
        ao_spi_get(AO_MPU6000_SPI_BUS, AO_SPI_SPEED_1MHz);
        ao_cur_task = NULL;
-#endif 
+#endif
 
        ao_cmd_register(&ao_mpu6000_cmds[0]);
 }