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

VUnit support #5

Closed
Ruskuls opened this issue May 24, 2023 · 5 comments
Closed

VUnit support #5

Ruskuls opened this issue May 24, 2023 · 5 comments

Comments

@Ruskuls
Copy link
Contributor

Ruskuls commented May 24, 2023

Vunit simulator support.

Instead of a dedicated VHDL module that generates traffic use Vunit modules that handle Avalon (or any other CPU bus) based transactions + add randomization for address and data.

Because testbench contains one Verilog file, GHDL can't be used. For local tests Free Modelsim versions were used.

@MJoergen
Copy link
Owner

I agree, the current test is lacking and is quite rudimentary. Have you already made a VUNIT test environment for this module ? If so, would you like to subject a Pull Request ?

@Ruskuls
Copy link
Contributor Author

Ruskuls commented May 24, 2023

Yes, I already made it, and yes I would like to make a Pull Request!

@Ruskuls
Copy link
Contributor Author

Ruskuls commented May 25, 2023

I agree, the current test is lacking and is quite rudimentary. Have you already made a VUNIT test environment for this module ? If so, would you like to subject a Pull Request ?

I have problems to pushing my local changes:
ERROR: Permission to MJoergen/HyperRAM.git denied to Ruskuls. fatal: Could not read from remote repository.

@MJoergen
Copy link
Owner

This repository does not support push from others than myself. Instead, the preferred way to share your changes is that you make a "Fork" of this repo. There is a button for this on the GitHub page at the top right corner.

This will give you a copy of the repo, where you can push your changes. When you are done, then you can issue a Pull Request from your copy of the repo back to mine. I can then review the changes and accept them.

@MJoergen
Copy link
Owner

MJoergen commented Jun 7, 2023

I've accepted your pull request. THANK YOU!!!

@MJoergen MJoergen closed this as completed Jun 7, 2023
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

No branches or pull requests

2 participants