X-Git-Url: https://git.gag.com/?p=fw%2Faltos;a=blobdiff_plain;f=src%2Fdrivers%2Fao_hmc5883.c;h=f761671a1d3e0dac1cbbe90be7a0408e6ae928d0;hp=64663ea6d33d618b794213a7726a0f5dc7919d69;hb=f4c812bef76a2cd95f675cb27ea89059561ceec7;hpb=03dc80d15a2f8fe9d7340351226dadd8bc3cfdb9

diff --git a/src/drivers/ao_hmc5883.c b/src/drivers/ao_hmc5883.c
index 64663ea6..f761671a 100644
--- a/src/drivers/ao_hmc5883.c
+++ b/src/drivers/ao_hmc5883.c
@@ -19,7 +19,7 @@
 #include <ao_hmc5883.h>
 #include <ao_exti.h>
 
-uint8_t	ao_hmc5883_valid;
+#if HAS_HMC5883
 
 static uint8_t	ao_hmc5883_configured;
 
@@ -63,21 +63,23 @@ ao_hmc5883_isr(void)
 	ao_wakeup(&ao_hmc5883_done);
 }
 
+static uint32_t	ao_hmc5883_missed_irq;
+
 void
 ao_hmc5883_sample(struct ao_hmc5883_sample *sample)
 {
 	uint16_t	*d = (uint16_t *) sample;
 	int		i = sizeof (*sample) / 2;
-	uint8_t		single = HMC5883_MODE_SINGLE;
 
 	ao_hmc5883_done = 0;
 	ao_exti_enable(AO_HMC5883_INT_PORT, AO_HMC5883_INT_PIN);
 	ao_hmc5883_reg_write(HMC5883_MODE, HMC5883_MODE_SINGLE);
 
-	cli();
+	ao_arch_block_interrupts();
 	while (!ao_hmc5883_done)
-		ao_sleep(&ao_hmc5883_done);
-	sei();
+		if (ao_sleep_for(&ao_hmc5883_done, AO_MS_TO_TICKS(10)))
+			++ao_hmc5883_missed_irq;
+	ao_arch_release_interrupts();
 
 	ao_hmc5883_read(HMC5883_X_MSB, (uint8_t *) sample, sizeof (struct ao_hmc5883_sample));
 #if __BYTE_ORDER == __LITTLE_ENDIAN
@@ -104,7 +106,7 @@ ao_hmc5883_setup(void)
 	ao_i2c_put(AO_HMC5883_I2C_INDEX);
 
 	if (!present)
-		ao_panic(AO_PANIC_SELF_TEST);
+		ao_panic(AO_PANIC_SELF_TEST_HMC5883);
 
 	ao_hmc5883_reg_write(HMC5883_CONFIG_A,
 			     (HMC5883_CONFIG_A_MA_8 << HMC5883_CONFIG_A_MA) |
@@ -125,14 +127,11 @@ ao_hmc5883(void)
 {
 	ao_hmc5883_setup();
 	for (;;) {
-		struct ao_hmc5883_sample ao_hmc5883_next;
-
-		ao_hmc5883_sample(&ao_hmc5883_next);
+		ao_hmc5883_sample(&ao_hmc5883_current);
 		ao_arch_critical(
-			ao_hmc5883_current = ao_hmc5883_next;
-			ao_hmc5883_valid = 1;
+			AO_DATA_PRESENT(AO_DATA_HMC5883);
+			AO_DATA_WAIT();
 			);
-		ao_delay(0);
 	}
 }
 
@@ -141,10 +140,10 @@ static struct ao_task ao_hmc5883_task;
 static void
 ao_hmc5883_show(void)
 {
-	struct ao_hmc5883_sample	sample;
-
-	sample = ao_hmc5883_current;
-	printf ("X: %d Y: %d Z: %d\n", sample.x, sample.y, sample.z);
+	struct ao_data	sample;
+	ao_data_get(&sample);
+	printf ("X: %d Y: %d Z: %d missed irq: %lu\n",
+		sample.hmc5883.x, sample.hmc5883.y, sample.hmc5883.z, ao_hmc5883_missed_irq);
 }
 
 static const struct ao_cmds ao_hmc5883_cmds[] = {
@@ -156,7 +155,6 @@ void
 ao_hmc5883_init(void)
 {
 	ao_hmc5883_configured = 0;
-	ao_hmc5883_valid = 0;
 
 	ao_enable_port(AO_HMC5883_INT_PORT);
 	ao_exti_setup(AO_HMC5883_INT_PORT,
@@ -167,3 +165,5 @@ ao_hmc5883_init(void)
 	ao_add_task(&ao_hmc5883_task, ao_hmc5883, "hmc5883");
 	ao_cmd_register(&ao_hmc5883_cmds[0]);
 }
+
+#endif