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.

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

Add a script for testing LiteScope #22

Merged
merged 1 commit into from
Jul 28, 2020

Conversation

jedrzejboczar
Copy link
Contributor

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

@cklarhorst
Copy link
Contributor

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
Copy link
Contributor Author

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
Copy link
Contributor

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

@enjoy-digital enjoy-digital merged commit a80c964 into enjoy-digital:master Jul 28, 2020
@enjoy-digital
Copy link
Owner

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.

@jedrzejboczar jedrzejboczar deleted the jboc/test-script branch August 21, 2020 12:33
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.

3 participants