From: Keith Packard <keithp@keithp.com>
Date: Thu, 20 Jun 2013 05:44:16 +0000 (-0700)
Subject: altos: PCA9922 LED driver needs Enable driven low to latch values
X-Git-Tag: 1.2.9.4~155
X-Git-Url: https://git.gag.com/?p=fw%2Faltos;a=commitdiff_plain;h=d90c2fa650de4cdb008d5e2559463c08da8db934

altos: PCA9922 LED driver needs Enable driven low to latch values

Driving Enable high means anything going past on the clock and data
pair is reflected on the LEDs, which isn't terribly useful

Signed-off-by: Keith Packard <keithp@keithp.com>
---

diff --git a/src/drivers/ao_pca9922.c b/src/drivers/ao_pca9922.c
index 6d8d18d8..fe070b88 100644
--- a/src/drivers/ao_pca9922.c
+++ b/src/drivers/ao_pca9922.c
@@ -30,9 +30,12 @@ ao_led_apply(void)
 	/* Don't try the SPI bus during initialization */
 	if (!ao_cur_task)
 		return;
-	ao_spi_get_bit(AO_PCA9922_CS_PORT, AO_PCA9922_CS_PIN, AO_PCA9922_CS, AO_PCA9922_SPI_BUS, AO_SPI_SPEED_FAST);
+	ao_mutex_get(&ao_spi_mutex);
+	ao_spi_set_speed(AO_SPI_SPEED_FAST);
+	AO_PCA9922_CS = 1;
 	ao_spi_send(&ao_led_state, 1, AO_PCA9922_SPI_BUS);
-	ao_spi_put_bit(AO_PCA9922_CS_PORT, AO_PCA9922_CS_PIN, AO_PCA9922_CS, AO_PCA9922_SPI_BUS);
+	AO_PCA9922_CS = 0;
+	ao_mutex_put(&ao_spi_mutex);
 }
 
 void