From d90e10cdbaa05509d1ee78f982d3aa0a2c94e972 Mon Sep 17 00:00:00 2001 From: Paul Fertser Date: Sun, 5 May 2013 00:46:49 +0400 Subject: [PATCH] adiv5_swdp: avoid dead-loop with a sleeping target A sleeping target continues to communicate over SWD properly but consistently returns SWDP_ACK_WAIT. It might be nice to be able to wait for the target to wake up on its own but if it's not going to wake up, BMP becomes absolutely unresponsive. Probably the timeout should be made configurable or some other workaround invented. Signed-off-by: Paul Fertser --- src/adiv5_swdp.c | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/adiv5_swdp.c b/src/adiv5_swdp.c index 8a38330..839127d 100644 --- a/src/adiv5_swdp.c +++ b/src/adiv5_swdp.c @@ -129,10 +129,14 @@ static uint32_t adiv5_swdp_low_access(ADIv5_DP_t *dp, uint8_t APnDP, uint8_t RnW if((addr == 4) || (addr == 8)) request ^= 0x20; + size_t tries = 1000; do { swdptap_seq_out(request, 8); ack = swdptap_seq_in(3); - } while(ack == SWDP_ACK_WAIT); + } while(--tries && ack == SWDP_ACK_WAIT); + + if(!tries) + PLATFORM_FATAL_ERROR(1); if(ack == SWDP_ACK_FAULT) { dp->fault = 1;