Skip to content

Handle no-ack#15

Merged
korken89 merged 1 commit intomasterfrom
add-no-ack-handling
Feb 12, 2026
Merged

Handle no-ack#15
korken89 merged 1 commit intomasterfrom
add-no-ack-handling

Conversation

@Tiwalun
Copy link
Member

@Tiwalun Tiwalun commented Feb 11, 2026

See documentation for e.g. DAP_transfer: https://arm-software.github.io/CMSIS_5/DAP/html/group__DAP__Transfer.html

Bit 2..0: ACK (Acknowledge) value:

1 = OK (for SWD protocol), OK or FAULT (for JTAG protocol),
2 = WAIT
4 = FAULT
7 = NO_ACK (no response from target)

That also matches the ARM Debug Interface spec, where protocol error is used for cases like a parity error or the stop bit being wrong.

This makes it possible for probe-rs to handle NoAck differently from protocol errors.

Copy link
Collaborator

@korken89 korken89 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice fix, thanks!

@korken89 korken89 merged commit a867ca7 into master Feb 12, 2026
1 check passed
@korken89 korken89 deleted the add-no-ack-handling branch February 12, 2026 20:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments