)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1000687,"name":"Tomas Vanek","display_name":"Tomas Vanek","email":"vanekt@fbl.cz","username":"vanekt"},"change_message_id":"f4b2bfdb6aaf93fbbcedae4b3a11ecbc207257bc","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"arm_adi_v5: Also clear sticky overrun bit on init"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Some targets start up with the sticky overrun bit set. On such targets"},{"line_number":10,"context_line":"we need to clear it in order to avoid subsequent incorrect reads."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I3e939a9e092de6fcea9494d3179a3386aa1701d2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"6b4078de_f710a2f7","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":4},"updated":"2024-07-31 07:14:07.000000000","message":"Please specify what targets","commit_id":"b7761d3edbd6786c06b4764b2560a7826c182360"},{"author":{"_account_id":1000687,"name":"Tomas Vanek","display_name":"Tomas Vanek","email":"vanekt@fbl.cz","username":"vanekt"},"change_message_id":"04460fd7b179a60770e14a0ee52a88833b36960b","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"arm_adi_v5: Also clear sticky overrun bit on init"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Some targets start up with the sticky overrun bit set. On such targets"},{"line_number":10,"context_line":"we need to clear it in order to avoid subsequent incorrect reads."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I3e939a9e092de6fcea9494d3179a3386aa1701d2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"cc5f46bb_32aa6779","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":4},"in_reply_to":"66950835_692ca3ae","updated":"2024-08-01 05:51:31.000000000","message":"It\u0027s ok with me","commit_id":"b7761d3edbd6786c06b4764b2560a7826c182360"},{"author":{"_account_id":1001992,"name":"Peter Collingbourne","email":"pcc@google.com","username":"pcc"},"change_message_id":"4fb496d004f7321757c1471485d91e87f0ec87f1","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"arm_adi_v5: Also clear sticky overrun bit on init"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Some targets start up with the sticky overrun bit set. On such targets"},{"line_number":10,"context_line":"we need to clear it in order to avoid subsequent incorrect reads."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I3e939a9e092de6fcea9494d3179a3386aa1701d2"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"66950835_692ca3ae","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":4},"in_reply_to":"6b4078de_f710a2f7","updated":"2024-07-31 18:01:13.000000000","message":"I saw this on a proprietary target so I can\u0027t name it here I\u0027m afraid. I will say that it\u0027s an ADIv6 target and I can add that to the commit message.","commit_id":"b7761d3edbd6786c06b4764b2560a7826c182360"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":1001992,"name":"Peter Collingbourne","email":"pcc@google.com","username":"pcc"},"change_message_id":"2b1e170d836dde68a7f9e028a757b91afccdc5e5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e3053e6b_47f082c2","updated":"2024-08-22 20:07:46.000000000","message":"Can we land this?","commit_id":"b7761d3edbd6786c06b4764b2560a7826c182360"},{"author":{"_account_id":1000687,"name":"Tomas Vanek","display_name":"Tomas Vanek","email":"vanekt@fbl.cz","username":"vanekt"},"change_message_id":"2e10a90e0542fbe6edb3f743fd83c1a0da8fdbdd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a64672ca_5b954437","updated":"2024-07-31 07:15:08.000000000","message":"For reference the older discussion:\n6119: target/adiv5: Add STICKYORUN bit clearing | https://review.openocd.org/c/openocd/+/6119","commit_id":"b7761d3edbd6786c06b4764b2560a7826c182360"},{"author":{"_account_id":1000687,"name":"Tomas Vanek","display_name":"Tomas Vanek","email":"vanekt@fbl.cz","username":"vanekt"},"change_message_id":"f4b2bfdb6aaf93fbbcedae4b3a11ecbc207257bc","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ff143900_a1e95bb8","updated":"2024-07-31 07:14:07.000000000","message":"IMO OpenOCD should not make any assumptions about the DP state before connecting so it should address even the (normally rare seen) state STICKYORUN without STICKYERR.\n\nIt remains questionable whether STICKYORUN clear on init is enough or we should extend `jtagdp_transaction_endcheck()` to clear STICKYORUN even if set without STICKYERR.","commit_id":"b7761d3edbd6786c06b4764b2560a7826c182360"},{"author":{"_account_id":1000687,"name":"Tomas Vanek","display_name":"Tomas Vanek","email":"vanekt@fbl.cz","username":"vanekt"},"change_message_id":"04460fd7b179a60770e14a0ee52a88833b36960b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a44adc37_74562a63","in_reply_to":"5e3e21d3_2968065e","updated":"2024-08-01 05:51:31.000000000","message":"Agreed.","commit_id":"b7761d3edbd6786c06b4764b2560a7826c182360"},{"author":{"_account_id":1001992,"name":"Peter Collingbourne","email":"pcc@google.com","username":"pcc"},"change_message_id":"4fb496d004f7321757c1471485d91e87f0ec87f1","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"5e3e21d3_2968065e","in_reply_to":"ff143900_a1e95bb8","updated":"2024-07-31 18:01:13.000000000","message":"I got the impression that `jtagdp_overrun_check()` is intended to handle overruns after our own transactions. So yeah, it\u0027s unclear whether checking for STICKYORUN in `jtagdp_transaction_endcheck()` is really necessary.\n\nAt any rate we don\u0027t know what happened to the DP before we connected, so we should be clearing sticky states on connect. This patch brings JTAG in line with SWD which also clears sticky states on connect (`swd_connect() -\u003e swd_connect_single() -\u003e swd_clear_sticky_errors()`) so IMHO it makes sense to make this change first regardless of how we decide to handle overruns during our own transactions.","commit_id":"b7761d3edbd6786c06b4764b2560a7826c182360"}]}