Skip to content

Fix debug sequence execution#1774

Merged
RobertRostohar merged 2 commits into
pyocd:developfrom
RobertRostohar:pr_dbg-seq
Jun 9, 2025
Merged

Fix debug sequence execution#1774
RobertRostohar merged 2 commits into
pyocd:developfrom
RobertRostohar:pr_dbg-seq

Conversation

@TeoMahnic

Copy link
Copy Markdown
Collaborator

The pull request improves connection robustness and DP sticky error bits handling for temporary target communication losses and __errorcontrol usage (CMSIS debug descriptions). For example in reset scenarios.

Related issue: Open-CMSIS-Pack/vscode-cmsis-debugger#92

- clear WDATAERR, STICKYORUN, STICKYCMP, and STICKYERR bits of CTRL/STAT Register if '__errorcontrol' is 1
- suppress TransferError during communication recovery attempt if '__errorcontrol' is 1
@RobertRostohar RobertRostohar marked this pull request as ready for review May 8, 2025 10:53
@RobertRostohar RobertRostohar requested a review from flit May 8, 2025 10:53
@RobertRostohar

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@RobertRostohar RobertRostohar merged commit c33f374 into pyocd:develop Jun 9, 2025
24 checks passed
@RobertRostohar RobertRostohar deleted the pr_dbg-seq branch June 9, 2025 11:38
RobertRostohar added a commit to RobertRostohar/pyOCD that referenced this pull request Oct 9, 2025
…ocd#1774)"

This reverts commit c33f374.

Implicitly clearing sticky error bits before executing the next DAP transfer can lead to unwanted side effects.
The debug sequence is responsible for doing that if required.
RobertRostohar added a commit that referenced this pull request Oct 14, 2025
)"

This reverts commit c33f374.

Implicitly clearing sticky error bits before executing the next DAP transfer can lead to unwanted side effects.
The debug sequence is responsible for doing that if required.
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