This is an automated email from Gerrit.

"Tomas Vanek <[email protected]>" just uploaded a new patch set to Gerrit, which 
you can find at https://review.openocd.org/c/openocd/+/9669

-- gerrit

commit c670be1dbed6f7c30fbf1add328c48964ce47bba
Author: Tomas Vanek <[email protected]>
Date:   Thu May 21 07:32:48 2026 +0200

    jtag/core: fix blocked polling in unknown state of jtag_xrst
    
    Patch [1] neglected three state character of jtag_trst variable,
    similarly the original code did for jtag_srst.
    
    Block polling only if SRST / TRST state is known and set.
    
    Fixes: [1] commit eeacc589af60 ("jtag/core: fix is_jtag_poll_safe() logic")
    Change-Id: I058491d5f0523af1690141787ec74bd21ad0832e
    Signed-off-by: Tomas Vanek <[email protected]>

diff --git a/src/jtag/core.c b/src/jtag/core.c
index 9c438a01b2..08228c0245 100644
--- a/src/jtag/core.c
+++ b/src/jtag/core.c
@@ -151,11 +151,11 @@ bool is_jtag_poll_safe(void)
                return false;
 
        /* On JTAG transport it is also implicitly disabled while TRST is 
active */
-       if (transport_is_jtag() && jtag_trst)
+       if (transport_is_jtag() && jtag_trst == 1)
                return false;
 
        /* On any transport while SRST is gating the JTAG clock or other debug 
HW */
-       return jtag_srst == 0 || (jtag_reset_config & RESET_SRST_NO_GATING);
+       return jtag_srst != 1 || (jtag_reset_config & RESET_SRST_NO_GATING);
 }
 
 bool jtag_poll_get_enabled(void)

-- 

Reply via email to