AUCOHL / DFFRAM

Standard Cell Library based Memory Compiler using FF/Latch cells
Apache License 2.0
133 stars 32 forks source link

fix simulation models for dual-ported SRAMs with >=32 words #194

Open Markussha opened 6 months ago

Markussha commented 6 months ago

Hi,

I've noticed during simulation, that I was unable to use the read port of RAM32_1RW1R. After some digging, I've found that the port has been left dangling. This PR adds a RAM16_1RW1R model and fixes RAM32_1RW1R. I've tested both models in Questa and they seem to work.

As I am not entirely aware what the additional files in the model directory do (e.g. config.yml), I hope that this change is beneficial to you :)