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

Initialize (FPGA) memory in Chisel 6 #4496

Open
schoeberl opened this issue Nov 6, 2024 · 1 comment
Open

Initialize (FPGA) memory in Chisel 6 #4496

schoeberl opened this issue Nov 6, 2024 · 1 comment

Comments

@schoeberl
Copy link
Contributor

Type of issue: Bug Report

Please provide the steps to reproduce the problem:

Initializing memory is broken now. See for an error: https://github.com/schoeberl/chisel-book/actions/runs/11708082480/job/32608967721

The code is here: https://github.com/schoeberl/chisel-book/blob/6ee398bd61bf8cdee02abcd780a2035b8cff4d1a/src/main/scala/memory.scala#L107-L130

and the test at: https://github.com/schoeberl/chisel-book/blob/6ee398bd61bf8cdee02abcd780a2035b8cff4d1a/src/test/scala/MemoryTest.scala#L118-L140

You can also reproduce it by running sbt test in the chisel-book repo, which is currently committed to version 6.

@donaldkuck
Copy link

It is a bug of chiseltest, because chiseltest does not support LoadMemoryAnnotation. see below log:

[info] compiling 1 Scala source to /home/zsj/workspace/chisel-book/target/scala-2.13/classes ...
[WARNING] Unsupported annotation: LoadMemoryAnnotation
 Please report this issue at https://github.com/ucb-bar/chiseltest/issues

relevant codes: https://github.com/ucb-bar/chiseltest/blob/6.x/src/main/scala/chiseltest/simulator/ChiselBridge.scala#L159

But we can use loadMemoryFromFileInline instead, because chisetest supports LoadMemoryInlineAnnotation.

I will submit the modified code to your repository.

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