From: Keith Packard <keithp@keithp.com>
Date: Fri, 9 Apr 2010 00:28:17 +0000 (-0700)
Subject: When changing RESET line, delay 20ms
X-Git-Tag: debian/0.6+163+g01e524f~2
X-Git-Url: https://git.gag.com/?p=fw%2Faltos;a=commitdiff_plain;h=f93c9bf3695862db31f2c3b3bc5a7bb24ef3766c

When changing RESET line, delay 20ms

The GPS data sheet suggests a 1uF cap on the reset line to ensure it
is held low long enough for the power supply to come up to voltage. TM
v1.0 loads a 0.001uF cap there, but in case that isn't large enough,
it could be replaced with the larger one. This change makes sure that
even with that larger value, the debugging link will be able to reset
the target.

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

diff --git a/src/ao_dbg.c b/src/ao_dbg.c
index d0633f92..2fc266e3 100644
--- a/src/ao_dbg.c
+++ b/src/ao_dbg.c
@@ -184,6 +184,12 @@ ao_dbg_long_delay(void)
 		_asm nop _endasm;
 }
 
+static void
+ao_dbg_reset_delay(void)
+{
+	ao_delay(AO_MS_TO_TICKS(20));
+}
+
 void
 ao_dbg_debug_mode(void)
 {
@@ -192,7 +198,7 @@ ao_dbg_debug_mode(void)
 	ao_dbg_send_bits(DBG_CLOCK|DBG_DATA|DBG_RESET_N, DBG_CLOCK|DBG_DATA|DBG_RESET_N);
 	ao_dbg_long_delay();
 	ao_dbg_send_bits(DBG_CLOCK|DBG_DATA|DBG_RESET_N,     0    |DBG_DATA|    0    );
-	ao_dbg_long_delay();
+	ao_dbg_reset_delay();
 	ao_dbg_send_bits(DBG_CLOCK|DBG_DATA|DBG_RESET_N, DBG_CLOCK|DBG_DATA|    0    );
 	ao_dbg_long_delay();
 	ao_dbg_send_bits(DBG_CLOCK|DBG_DATA|DBG_RESET_N,     0    |DBG_DATA|    0    );
@@ -200,7 +206,7 @@ ao_dbg_debug_mode(void)
 	ao_dbg_send_bits(DBG_CLOCK|DBG_DATA|DBG_RESET_N, DBG_CLOCK|DBG_DATA|    0    );
 	ao_dbg_long_delay();
 	ao_dbg_send_bits(DBG_CLOCK|DBG_DATA|DBG_RESET_N,     0    |DBG_DATA|DBG_RESET_N);
-	ao_dbg_long_delay();
+	ao_dbg_reset_delay();
 }
 
 void
@@ -211,7 +217,7 @@ ao_dbg_reset(void)
 	ao_dbg_send_bits(DBG_CLOCK|DBG_DATA|DBG_RESET_N, DBG_CLOCK|DBG_DATA|DBG_RESET_N);
 	ao_dbg_long_delay();
 	ao_dbg_send_bits(DBG_CLOCK|DBG_DATA|DBG_RESET_N, DBG_CLOCK|DBG_DATA|    0    );
-	ao_dbg_long_delay();
+	ao_dbg_reset_delay();
 	ao_dbg_send_bits(DBG_CLOCK|DBG_DATA|DBG_RESET_N, DBG_CLOCK|DBG_DATA|    0    );
 	ao_dbg_long_delay();
 	ao_dbg_send_bits(DBG_CLOCK|DBG_DATA|DBG_RESET_N, DBG_CLOCK|DBG_DATA|    0    );
@@ -219,7 +225,7 @@ ao_dbg_reset(void)
 	ao_dbg_send_bits(DBG_CLOCK|DBG_DATA|DBG_RESET_N, DBG_CLOCK|DBG_DATA|    0    );
 	ao_dbg_long_delay();
 	ao_dbg_send_bits(DBG_CLOCK|DBG_DATA|DBG_RESET_N, DBG_CLOCK|DBG_DATA|DBG_RESET_N);
-	ao_dbg_long_delay();
+	ao_dbg_reset_delay();
 }
 
 static void