-
Notifications
You must be signed in to change notification settings - Fork 4
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
sram: fakeram vs. mock SRAM #151
Conversation
@jeffng-or FYI, will add fake sram tomorrow and write some docs. There are some snags to be fixed in bazel-orfs as well... |
696ee92
to
5cfa0a3
Compare
@maliberty @jeffng-or There's no clock and "all_registers" returns 0 with fakeram. Is there something wrong with the fakeram .lib file? Either use the standalone .tar below, or with this PR run:
untar no-clock-or-registers.tar.gz
|
ee801c0
to
aec1dcc
Compare
sram/README.md
Outdated
Mock SRAM based on fakeram's behavioral .v file | ||
----------------------------------------------- | ||
|
||
The .v file generated by fakeram does not build with yosys unmodified. The fakeram/sdq_17x64.sv has been manually pruend until it passed yosys. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nit: s/pruend/pruned
Signed-off-by: Øyvind Harboe <[email protected]>
aec1dcc
to
3fd1968
Compare
The fact that there are no clocks make sense. The sram/fakeram/constraints-sram.sdc defines the clock variables, but the call to source $PLATFORM_DIR/constraints.sdc is being gated by the all_registers check. Since all_registers is returning an empty list, we never get to create the clocks. In terms of all_registers, the liberty only has pin definitions, so not sure what we expect all_registers to report. |
If the .lib does not have any registers how can it store state? The symptom is no clock, the cause is that all_registers returns empty... |
No description provided.