cargo-rtic-scope icon indicating copy to clipboard operation
cargo-rtic-scope copied to clipboard

Stabilize STLink probe source

Open tmplt opened this issue 4 years ago • 5 comments

As mentioned in https://github.com/rtic-scope/cargo-rtic-scope/commit/f2a73dabf540baa20642e0ed9345c79f6397a06e, the new CMSIS-DAP implementation is buggy. Investigate the cause and fix.

tmplt avatar Jul 02 '21 09:07 tmplt

Tangent: an STLink is not a CMSIS-DAP. DAPSource should be further abstracted to ProbeSource. We do not care what kind of probe is ultimately used, only that probe-rs supports it.

tmplt avatar Jul 06 '21 20:07 tmplt

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

tmplt avatar Jul 28 '21 04:07 tmplt

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

tmplt avatar Jul 28 '21 04:07 tmplt

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

tmplt avatar Oct 31 '21 15:10 tmplt

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.

tmplt avatar Mar 10 '22 06:03 tmplt