enjoy-digital / litescope

Small footprint and configurable embedded FPGA logic analyzer
Other
160 stars 38 forks source link

Add a script for testing LiteScope #22

Closed jedrzejboczar closed 3 years ago

jedrzejboczar commented 3 years ago

Allows to list available signal names and to pass them from command line. Does some simple substring matching (with regex support).

cklarhorst commented 3 years ago

Hi @jedrzejboczar nice script, I like your idea to specify signals via regex. Couldn't we put your script to litescope/litescope/tools/. To upgrade this script from "another example" to "the main fetch script for litexscope data"? And to have the same structure for scripts as the litex git?

jedrzejboczar commented 3 years ago

I can move it there, sure. But that's probably more of a question to @enjoy-digital If so we could probably add additional arguments for dump file name, analyzer name, etc.

cklarhorst commented 3 years ago

Yes, lets see what he says about it. I think it would also be helpful for the new debug targets in the buildenv git (https://github.com/timvideos/litex-buildenv/pull/492).

enjoy-digital commented 3 years ago

Hi @cklarhorst,

indeed, that's exactly the idea i had when @jedrzejboczar shared with me the nice script he did and i asked him if he could do do a PR for it. I need to uddate the LiteScope examples and will move the location of this script to make sure it's available externally.

For now the script has been merged.