Skip to content

Fix RTT Control Block Search#1669

Merged
DavidLesnjak merged 2 commits intopyocd:developfrom
elfmimi:fix-rtt-cb-search
Apr 13, 2026
Merged

Fix RTT Control Block Search#1669
DavidLesnjak merged 2 commits intopyocd:developfrom
elfmimi:fix-rtt-cb-search

Conversation

@elfmimi
Copy link
Copy Markdown

@elfmimi elfmimi commented Feb 4, 2024

  • Revival of memory transfer in chunks. introduced in RTT: search for RTT control block in smaller chunks #1375
  • Fix the case when control block ID is located across read block boundary.
  • Addressing the possibility of the control block ID being proceeded by fraction image of itself. ex. b'SEGGSEGGER RTT'

Fixes #1661

Copy link
Copy Markdown

@jscatena88 jscatena88 left a comment

Choose a reason for hiding this comment

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

LGTM, fixes the issue we are having

@jscatena88
Copy link
Copy Markdown

Is there anything holding up this PR? I am running into the bug as well and this seems to fix it. Anything I can do to help get it over the line?

@ithinuel
Copy link
Copy Markdown
Member

Hi, this seems to conflict with #1583
The would need to be rebased and the conflict fixed please.

@DavidLesnjak
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@DavidLesnjak DavidLesnjak merged commit 97ced2f into pyocd:develop Apr 13, 2026
24 checks passed
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.

4 participants