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 simulation model for Xilinx BRAM #4663

Open
sifferman opened this issue Oct 13, 2024 · 0 comments
Open

Add simulation model for Xilinx BRAM #4663

sifferman opened this issue Oct 13, 2024 · 0 comments

Comments

@sifferman
Copy link

Feature Description

Hello.

My RTL's post-synthesis simulation was failing regression, and I tracked it down to an issue with the BRAM. I just realized that the Xilinx RAMB modules in techlibs/xilinx/cells_sim.v are not implemented for simulation.

Ideally, techlibs/xilinx/cells_sim.v should implement a simulation model for all BRAMs, which can basically be copied from Xilinx's Unisim Library (under Apache 2.0).

However, Xilinx's models can't be used directly because they are incompatible with Icarus and Verilator.

Any thoughts on this? I would be happy to do a pull if it would be helpful.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant