From 2b10052097c882cf0ed92748c15ff6ee001c1f8f Mon Sep 17 00:00:00 2001 From: "Sergey A. Borshch" Date: Fri, 18 Oct 2013 18:13:03 +0300 Subject: [PATCH] hla_interface: init trst, srst variables. MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit There is an remark in jtag_srst and jtag_trst variables declaration: /* * JTAG adapters must initialize with TRST and SRST de-asserted * (they're negative logic, so that means *high*). But some * hardware doesn't necessarily work that way ... so set things * up so that jtag_init() always forces that state. */ but in hla_target such forsing is missed and both variables remains uninitialized until "reset" command issued, It prevents target polling when connecting to running target. Change-Id: Ia620d8794cca8ba1403f5c5f24767e730033748a Signed-off-by: Sergey A. Borshch Reviewed-on: http://openocd.zylin.com/1698 Tested-by: jenkins Reviewed-by: Spencer Oliver Reviewed-by: Mathias Küster --- src/jtag/hla/hla_interface.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/jtag/hla/hla_interface.c b/src/jtag/hla/hla_interface.c index 00e454917..02d0f2fb0 100644 --- a/src/jtag/hla/hla_interface.c +++ b/src/jtag/hla/hla_interface.c @@ -137,6 +137,8 @@ int hl_interface_init_reset(void) if (hl_if.param.connect_under_reset) { jtag_add_reset(0, 1); hl_if.layout->api->assert_srst(hl_if.fd, 0); + } else { + jtag_add_reset(0, 0); } return ERROR_OK; -- 2.30.2