altos: Use split SPI bus for MS5607 sensor

[fw/altos] / src / drivers / ao_ms5607.c

diff --git a/src/drivers/ao_ms5607.c b/src/drivers/ao_ms5607.c
index 0f0625d09bd305e87c6c4a33a378ae2db887a749..1b55b7fdeea61d4f99101425fb3f38aed1606a4c 100644 (file)
--- a/src/drivers/ao_ms5607.c
+++ b/src/drivers/ao_ms5607.c
@@ -24,13 +24,13 @@ static uint8_t                      ms5607_configured;
 
 static void
 ao_ms5607_start(void) {
-       ao_spi_get(AO_MS5607_SPI_INDEX);
-       stm_gpio_set(&AO_MS5607_CS_GPIO, AO_MS5607_CS, 0);
+       ao_spi_get(AO_MS5607_SPI_INDEX,AO_SPI_SPEED_FAST);
+       stm_gpio_set(AO_MS5607_CS_GPIO, AO_MS5607_CS, 0);
 }
 
 static void
 ao_ms5607_stop(void) {
-       stm_gpio_set(&AO_MS5607_CS_GPIO, AO_MS5607_CS, 1);
+       stm_gpio_set(AO_MS5607_CS_GPIO, AO_MS5607_CS, 1);
        ao_spi_put(AO_MS5607_SPI_INDEX);
 }
 
@@ -118,6 +118,7 @@ static uint8_t      ao_ms5607_done;
 static void
 ao_ms5607_isr(void)
 {
+       ao_exti_disable(AO_MS5607_MISO_GPIO, AO_MS5607_MISO);
        ao_ms5607_done = 1;
        ao_wakeup(&ao_ms5607_done);
 }
@@ -132,13 +133,19 @@ ao_ms5607_get_sample(uint8_t cmd) {
 
        ao_ms5607_start();
        ao_spi_send(&cmd, 1, AO_MS5607_SPI_INDEX);
-       ao_exti_enable(&AO_MS5607_MISO_GPIO, AO_MS5607_MISO);
+       ao_exti_enable(AO_MS5607_MISO_GPIO, AO_MS5607_MISO);
+#if AO_MS5607_PRIVATE_PINS
+       ao_spi_put(AO_MS5607_SPI_INDEX);
+#endif
        cli();
        while (!ao_ms5607_done)
                ao_sleep(&ao_ms5607_done);
        sei();
-       ao_exti_disable(&AO_MS5607_MISO_GPIO, AO_MS5607_MISO);
+#if AO_MS5607_PRIVATE_PINS
+       stm_gpio_set(AO_MS5607_CS_GPIO, AO_MS5607_CS, 1);
+#else
        ao_ms5607_stop();
+#endif
 
        ao_ms5607_start();
        read = AO_MS5607_ADC_READ;
@@ -203,7 +210,7 @@ ao_ms5607(void)
        ao_ms5607_setup();
        for (;;)
        {
-               struct ao_ms5607_sample ao_ms5607_next;
+               static struct ao_ms5607_sample  ao_ms5607_next;
                ao_ms5607_sample(&ao_ms5607_next);
                ao_arch_critical(
                        ao_ms5607_current = ao_ms5607_next;
@@ -260,7 +267,7 @@ ao_ms5607_init(void)
         * conversion is complete, the MS5607 will raise this
         * pin as a signal
         */
-       ao_exti_setup(&AO_MS5607_MISO_GPIO,
+       ao_exti_setup(AO_MS5607_MISO_GPIO,
                      AO_MS5607_MISO,
                      AO_EXTI_MODE_RISING,
                      ao_ms5607_isr);
@@ -268,7 +275,7 @@ ao_ms5607_init(void)
        /* Reset the pin from INPUT to ALTERNATE so that SPI works
         * This needs an abstraction at some point...
         */
-       stm_moder_set(&AO_MS5607_MISO_GPIO,
+       stm_moder_set(AO_MS5607_MISO_GPIO,
                      AO_MS5607_MISO,
                      STM_MODER_ALTERNATE);
 }