diff options
author | Øyvind Harboe <oyvind.harboe@zylin.com> | 2011-03-03 09:14:30 +0100 |
---|---|---|
committer | Øyvind Harboe <oyvind.harboe@zylin.com> | 2011-03-15 10:36:16 +0100 |
commit | bb0d11cba989832c0f27c2e5b664bfdc26b98cc9 (patch) | |
tree | 6604069fb5a948d67e6afbc86b84fee6f56fdb43 | |
parent | 689c244389798ef7fc041693831ae114b3063d36 (diff) | |
download | openocd+libswd-bb0d11cba989832c0f27c2e5b664bfdc26b98cc9.tar.gz openocd+libswd-bb0d11cba989832c0f27c2e5b664bfdc26b98cc9.tar.bz2 openocd+libswd-bb0d11cba989832c0f27c2e5b664bfdc26b98cc9.tar.xz openocd+libswd-bb0d11cba989832c0f27c2e5b664bfdc26b98cc9.zip |
jtag: clean up jtag_sleep, handle short sleeps correctly via usleep
short sleeps are handled via usleep, longer sleeps we round up
to nearest ms.
There was a bug in jtag_sleep() in that it would round *down*
to nearest ms, thus making all <1ms sleeps 0. Found by inspection
rather than symptom.
Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
-rw-r--r-- | src/jtag/core.c | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/src/jtag/core.c b/src/jtag/core.c index d7e1ccec..68c12570 100644 --- a/src/jtag/core.c +++ b/src/jtag/core.c @@ -871,9 +871,16 @@ static int jtag_reset_callback(enum jtag_event event, void *priv) return ERROR_OK; } +/* sleep at least us microseconds. When we sleep more than 1000ms we + * do an alive sleep, i.e. keep GDB alive. Note that we could starve + * GDB if we slept for <1000ms many times. + */ void jtag_sleep(uint32_t us) { - alive_sleep(us/1000); + if (us < 1000) + usleep(us); + else + alive_sleep((us+999)/1000); } /* Maximum number of enabled JTAG devices we expect in the scan chain, |