X-Git-Url: https://git.gag.com/?p=fw%2Faltos;a=blobdiff_plain;f=src%2Fdrivers%2Fao_ms5607.c;h=4bfc17db8016f1adaeb0f7d2cc2eaf842e7f30c8;hp=914e0c1b3c7b620ca40c23ee82f88c9ac5b5576e;hb=6023ff81f1bbd240169b9548209133d3b02d475f;hpb=c8dbcaf69cd538a31ab6e2b568237ae7c8656a9a

diff --git a/src/drivers/ao_ms5607.c b/src/drivers/ao_ms5607.c
index 914e0c1b..4bfc17db 100644
--- a/src/drivers/ao_ms5607.c
+++ b/src/drivers/ao_ms5607.c
@@ -45,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();
 }
@@ -57,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;
@@ -73,38 +74,56 @@ ao_ms5607_crc(uint8_t *prom)
 	return n_rem;
 }
 
+static int
+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)
 {
-	uint8_t			addr;
-	uint8_t			crc;
-	__xdata uint16_t	*r;
+	uint8_t		addr;
+	uint16_t	*r;
 
 	r = (__xdata 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
 }
@@ -137,7 +156,7 @@ ao_ms5607_get_sample(uint8_t cmd) {
 	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);