Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Stabilize STLink probe source #18

Open
tmplt opened this issue Jul 2, 2021 · 5 comments
Open

Stabilize STLink probe source #18

tmplt opened this issue Jul 2, 2021 · 5 comments
Labels
bug Something isn't working upstream This issue is an upstream issue

Comments

@tmplt
Copy link
Member

tmplt commented Jul 2, 2021

As mentioned in f2a73da, the new CMSIS-DAP implementation is buggy. Investigate the cause and fix.
@tmplt tmplt added the bug Something isn't working label Jul 2, 2021
@tmplt tmplt added this to To do in MVP release Jul 2, 2021
@tmplt tmplt added this to To do in v0.2.0 Jul 6, 2021
@tmplt tmplt removed this from To do in MVP release Jul 6, 2021
@tmplt tmplt added this to the v1.0.0 milestone Jul 6, 2021
tmplt added a commit that referenced this issue Jul 6, 2021
@tmplt
default --tpiu-baud to 2MBd
b240b37 
Anecdotally works better with the STLink (more packages are received
until an invalid sync package is encountered), unil #18 is fixed.
@tmplt

This comment has been minimized.

Sign in to view
@tmplt tmplt changed the title Stabilize CMSIS-DAP source Stabilize STLink probe source Jul 7, 2021
@tmplt tmplt moved this from To do to In progress in v0.2.0 Jul 23, 2021
@tmplt tmplt moved this from In progress to To do in v0.2.0 Jul 23, 2021
@tmplt
Copy link
Member Author

tmplt commented Jul 28, 2021

Mostly we just get an invalid sync size. This could just be a variance from the ITM standard.

@tmplt
Copy link
Member Author

tmplt commented Jul 28, 2021

Perhaps a feature-set of quirks should be maintained upstream?

@tmplt tmplt removed this from To do in v0.2.0 Aug 10, 2021
@tmplt tmplt added this to To Do in v0.3.0 Aug 10, 2021
@tmplt tmplt modified the milestones: v1.0.0, v0.3.0 Aug 10, 2021
@tmplt tmplt removed this from To Do in v0.3.0 Oct 29, 2021
@tmplt
Copy link
Member Author

tmplt commented Oct 31, 2021

This would be an upstream issue in probe-rs. Presumably the amount of work is non-trivial. Postponing.

@tmplt tmplt added the upstream This issue is an upstream issue label Oct 31, 2021
@tmplt tmplt removed this from the v0.3.0 milestone Oct 31, 2021
This was referenced Jan 10, 2022
Open
Open
@tmplt
Copy link
Member Author

tmplt commented Mar 10, 2022

I theorize that the root cause of this issue is transient configuration noise seen on the SWO pin. So far I've only been able to reproduce this issue on a Nucleo, where the SWO pin feeds into an on-chip debugger that is interfaced with via USB. Verifying if this indeed is the cause should be trivial, but I do not have a Nucleo at hand at the moment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working upstream This issue is an upstream issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@tmplt