altos/micropeak-v2.0: expose log and config commands over USB
[fw/altos] / src / drivers / ao_ms5607.c
index 7c1acdd..1621411 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
 
 #if HAS_MS5607 || HAS_MS5611
 
-static __xdata struct ao_ms5607_prom   ms5607_prom;
-static __xdata uint8_t                 ms5607_configured;
+struct ao_ms5607_prom  ao_ms5607_prom;
+static uint8_t         ms5607_configured;
+
+#ifndef AO_MS5607_SPI_SPEED
+#define AO_MS5607_SPI_SPEED    AO_SPI_SPEED_FAST
+#endif
 
 static void
 ao_ms5607_start(void) {
-       ao_spi_get_bit(AO_MS5607_CS_PORT, AO_MS5607_CS_PIN, AO_MS5607_CS, AO_MS5607_SPI_INDEX, AO_SPI_SPEED_FAST);
+       ao_spi_get_bit(AO_MS5607_CS_PORT, AO_MS5607_CS_PIN, AO_MS5607_SPI_INDEX, AO_MS5607_SPI_SPEED);
 }
 
 static void
 ao_ms5607_stop(void) {
-       ao_spi_put_bit(AO_MS5607_CS_PORT, AO_MS5607_CS_PIN, AO_MS5607_CS, AO_MS5607_SPI_INDEX);
+       ao_spi_put_bit(AO_MS5607_CS_PORT, AO_MS5607_CS_PIN, AO_MS5607_SPI_INDEX);
 }
 
 static void
@@ -40,7 +45,7 @@ ao_ms5607_reset(void) {
 
        cmd = AO_MS5607_RESET;
        ao_ms5607_start();
-       ao_spi_send(DATA_TO_XDATA(&cmd), 1, AO_MS5607_SPI_INDEX);
+       ao_spi_send(&cmd, 1, AO_MS5607_SPI_INDEX);
        ao_delay(AO_MS_TO_TICKS(10));
        ao_ms5607_stop();
 }
@@ -52,10 +57,11 @@ ao_ms5607_crc(uint8_t *prom)
        uint8_t         cnt;
        uint16_t        n_rem = 0;
        uint8_t         n_bit;
+       uint8_t         *p = prom;
 
        prom[15] = 0;
        for (cnt = 0; cnt < 16; cnt++) {
-               n_rem ^= prom[cnt];
+               n_rem ^= *p++;
                for (n_bit = 8; n_bit > 0; n_bit--) {
                        if (n_rem & 0x8000)
                                n_rem = (n_rem << 1) ^ 0x3000;
@@ -68,38 +74,56 @@ ao_ms5607_crc(uint8_t *prom)
        return n_rem;
 }
 
+static bool
+ao_ms5607_prom_valid(uint8_t *prom)
+{
+       uint8_t crc;
+       int     i;
+       uint8_t *p;
+
+       /* Look for a value other than 0x0000 or 0xffff */
+       p = prom;
+       for (i = 0; i < 16; i++)
+               if (*p++ + 1 > 1)
+                       break;
+       if (i == 16)
+               return false;
+
+       crc = ao_ms5607_crc(prom);
+       if (crc != (prom[15] & 0xf))
+               return false;
+
+       return true;
+}
+
 static void
-ao_ms5607_prom_read(__xdata struct ao_ms5607_prom *prom)
+ao_ms5607_prom_read(struct ao_ms5607_prom *prom)
 {
-       uint8_t                 addr;
-       uint8_t                 crc;
-       __xdata uint16_t        *r;
+       uint8_t         addr;
+       uint16_t        *r;
 
-       r = (__xdata uint16_t *) prom;
+       r = (uint16_t *) prom;
        for (addr = 0; addr < 8; addr++) {
                uint8_t cmd = AO_MS5607_PROM_READ(addr);
                ao_ms5607_start();
-               ao_spi_send(DATA_TO_XDATA(&cmd), 1, AO_MS5607_SPI_INDEX);
+               ao_spi_send(&cmd, 1, AO_MS5607_SPI_INDEX);
                ao_spi_recv(r, 2, AO_MS5607_SPI_INDEX);
                ao_ms5607_stop();
                r++;
        }
-       crc = ao_ms5607_crc((uint8_t *) prom);
-       if (crc != (((uint8_t *) prom)[15] & 0xf)) {
-#if HAS_TASK
-               printf ("MS5607 PROM CRC error (computed %x actual %x)\n",
-                       crc, (((uint8_t *) prom)[15] & 0xf));
-               flush();
-#endif
+
+       if (!ao_ms5607_prom_valid((uint8_t *) prom))
                ao_panic(AO_PANIC_SELF_TEST_MS5607);
-       }
 
 #if __BYTE_ORDER == __LITTLE_ENDIAN
        /* Byte swap */
        r = (uint16_t *) prom;
        for (addr = 0; addr < 8; addr++) {
-               uint16_t        t = *r;
-               *r++ = (t << 8) | (t >> 8);
+               uint8_t         *t = (uint8_t *) r;
+               uint8_t a = t[0];
+               t[0] = t[1];
+               t[1] = a;
+               r++;
        }
 #endif
 }
@@ -111,28 +135,28 @@ ao_ms5607_setup(void)
                return;
        ms5607_configured = 1;
        ao_ms5607_reset();
-       ao_ms5607_prom_read(&ms5607_prom);
+       ao_ms5607_prom_read(&ao_ms5607_prom);
 }
 
-static __xdata volatile uint8_t        ao_ms5607_done;
+static volatile uint8_t        ao_ms5607_done;
 
 static void
 ao_ms5607_isr(void)
 {
        ao_exti_disable(AO_MS5607_MISO_PORT, AO_MS5607_MISO_PIN);
        ao_ms5607_done = 1;
-       ao_wakeup((__xdata void *) &ao_ms5607_done);
+       ao_wakeup((void *) &ao_ms5607_done);
 }
 
 static uint32_t
 ao_ms5607_get_sample(uint8_t cmd) {
-       __xdata uint8_t reply[3];
-       __xdata uint8_t read;
+       uint8_t reply[3];
+       uint8_t read;
 
        ao_ms5607_done = 0;
 
        ao_ms5607_start();
-       ao_spi_send(DATA_TO_XDATA(&cmd), 1, AO_MS5607_SPI_INDEX);
+       ao_spi_send(&cmd, 1, AO_MS5607_SPI_INDEX);
 
        ao_exti_enable(AO_MS5607_MISO_PORT, AO_MS5607_MISO_PIN);
 
@@ -140,7 +164,8 @@ ao_ms5607_get_sample(uint8_t cmd) {
        ao_spi_put(AO_MS5607_SPI_INDEX);
 #endif
        ao_arch_block_interrupts();
-       while (!ao_ms5607_done)
+       while (!ao_gpio_get(AO_MS5607_MISO_PORT, AO_MS5607_MISO_PIN) &&
+              !ao_ms5607_done)
                ao_sleep((void *) &ao_ms5607_done);
        ao_arch_release_interrupts();
 #if AO_MS5607_PRIVATE_PINS
@@ -173,7 +198,7 @@ ao_ms5607_get_sample(uint8_t cmd) {
 #define AO_CONVERT_D2  token_evaluator(AO_MS5607_CONVERT_D2_, AO_MS5607_TEMP_OVERSAMPLE)
 
 void
-ao_ms5607_sample(__xdata struct ao_ms5607_sample *sample)
+ao_ms5607_sample(struct ao_ms5607_sample *sample)
 {
        sample->pres = ao_ms5607_get_sample(AO_CONVERT_D1);
        sample->temp = ao_ms5607_get_sample(AO_CONVERT_D2);
@@ -185,50 +210,65 @@ ao_ms5607_sample(__xdata struct ao_ms5607_sample *sample)
 #include "ao_ms5607_convert.c"
 #endif
 
-#if HAS_TASK
-__xdata struct ao_ms5607_sample        ao_ms5607_current;
+#ifndef HAS_MS5607_TASK
+#define HAS_MS5607_TASK HAS_TASK
+#endif
 
+struct ao_ms5607_sample        ao_ms5607_current;
+
+#if HAS_MS5607_TASK
 static void
 ao_ms5607(void)
 {
+       struct ao_ms5607_sample sample;
        ao_ms5607_setup();
        for (;;)
        {
-               ao_ms5607_sample(&ao_ms5607_current);
+               ao_ms5607_sample(&sample);
                ao_arch_block_interrupts();
+               ao_ms5607_current = sample;
                AO_DATA_PRESENT(AO_DATA_MS5607);
                AO_DATA_WAIT();
                ao_arch_release_interrupts();
        }
 }
 
-__xdata struct ao_task ao_ms5607_task;
+struct ao_task ao_ms5607_task;
+#endif
 
+#if HAS_TASK
 void
 ao_ms5607_info(void)
 {
-       printf ("ms5607 reserved: %u\n", ms5607_prom.reserved);
-       printf ("ms5607 sens: %u\n", ms5607_prom.sens);
-       printf ("ms5607 off: %u\n", ms5607_prom.off);
-       printf ("ms5607 tcs: %u\n", ms5607_prom.tcs);
-       printf ("ms5607 tco: %u\n", ms5607_prom.tco);
-       printf ("ms5607 tref: %u\n", ms5607_prom.tref);
-       printf ("ms5607 tempsens: %u\n", ms5607_prom.tempsens);
-       printf ("ms5607 crc: %u\n", ms5607_prom.crc);
+#if !HAS_MS5607_TASK
+       ao_ms5607_setup();
+#endif
+       printf ("ms5607 reserved: %u\n", ao_ms5607_prom.reserved);
+       printf ("ms5607 sens: %u\n", ao_ms5607_prom.sens);
+       printf ("ms5607 off: %u\n", ao_ms5607_prom.off);
+       printf ("ms5607 tcs: %u\n", ao_ms5607_prom.tcs);
+       printf ("ms5607 tco: %u\n", ao_ms5607_prom.tco);
+       printf ("ms5607 tref: %u\n", ao_ms5607_prom.tref);
+       printf ("ms5607 tempsens: %u\n", ao_ms5607_prom.tempsens);
+       printf ("ms5607 crc: %u\n", ao_ms5607_prom.crc);
 }
 
 static void
 ao_ms5607_dump(void)
 {
-       __xdata struct ao_ms5607_value value;
+       struct ao_ms5607_value value;
 
+#if !HAS_MS5607_TASK
+       ao_ms5607_info();
+       ao_ms5607_sample(&ao_ms5607_current);
+#endif
        ao_ms5607_convert(&ao_ms5607_current, &value);
-       printf ("Pressure:    %8u %8d\n", ao_ms5607_current.pres, value.pres);
-       printf ("Temperature: %8u %8d\n", ao_ms5607_current.temp, value.temp);
+       printf ("Pressure:    %8lu %8ld\n", ao_ms5607_current.pres, value.pres);
+       printf ("Temperature: %8lu %8ld\n", ao_ms5607_current.temp, value.temp);
        printf ("Altitude: %ld\n", ao_pa_to_altitude(value.pres));
 }
 
-__code struct ao_cmds ao_ms5607_cmds[] = {
+const struct ao_cmds ao_ms5607_cmds[] = {
        { ao_ms5607_dump,       "B\0Display MS5607 data" },
        { 0, NULL },
 };
@@ -242,6 +282,8 @@ ao_ms5607_init(void)
 
 #if HAS_TASK
        ao_cmd_register(&ao_ms5607_cmds[0]);
+#endif
+#if HAS_MS5607_TASK
        ao_add_task(&ao_ms5607_task, ao_ms5607, "ms5607");
 #endif