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

Boards openroad added #83

Merged
merged 4 commits into from
Mar 15, 2025
Merged

Boards openroad added #83

merged 4 commits into from
Mar 15, 2025

Conversation

oharboe
Copy link
Contributor

@oharboe oharboe commented Mar 15, 2025

Waiting for a nit to be fixed in ORFS, but otherwise ready for comments.

OpenROAD ASAP7 PDK configuration

Demonstrates how to set up an bazel-orfs to build darksocv withOpenROAD-flow-scripts

To build and view Install Bazelisk and run:

bazel run //boards/openroad:darksocv_cts /tmp/cts gui_cts

Register to register histogram

image

Estimated routing congestion

image

Ideas for future work

  • reduce clock period
  • darkram.v should consist of serveral SRAMs connected together to be
    a more accurate representation
  • create a mock SRAM representation with somewhat realistic timing
  • add IO constraints to place pins on one edge of the SRAMs and top level
  • reduce area

MegaBoom demonstrates a number of techniques to study a design and set up mock SRAMs.

oharboe added 2 commits March 15, 2025 10:36
Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
@oharboe oharboe force-pushed the boards-openroad branch 2 times, most recently from 2ad1dd7 to 754d014 Compare March 15, 2025 20:36
@oharboe oharboe changed the title [WORK IN PROGRESS] Boards openroad Boards openroad added Mar 15, 2025
Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
@oharboe oharboe marked this pull request as ready for review March 15, 2025 20:37
@samsoniuk samsoniuk merged commit 5e1da2e into darklife:master Mar 15, 2025
@samsoniuk
Copy link
Member

hmmm... I read about the "Synthesis fails because it doesn't strip $display", maybe I need put some ifdef SIMULATION around such $display across the code... the tools that I use typically ignore them at synth time, but it may cause other tools to fail. I will try address this problem in the future!

Anyway, thanks for the commit! :)

@oharboe
Copy link
Contributor Author

oharboe commented Mar 15, 2025

@maliberty FYI

@oharboe
Copy link
Contributor Author

oharboe commented Mar 15, 2025

hmmm... I read about the "Synthesis fails because it doesn't strip $display", maybe I need put some ifdef SIMULATION around such $display across the code... the tools that I use typically ignore them at synth time, but it may cause other tools to fail. I will try address this problem in the future!

Anyway, thanks for the commit! :)

Not to worry, it is already fixed in ORFS.

If there is any interest in this and some future PR is required, I will bump ORFS and clean out the workaround.

Meanwhile, take it for a spin and let me know if you have any questions :-)

@samsoniuk
Copy link
Member

well, just in case, I already fixed it! thanks again! :)

@samsoniuk
Copy link
Member

samsoniuk commented Mar 15, 2025

Meanwhile, take it for a spin and let me know if you have any questions :-)

wow! I have lots of questions but not sure that are the right questions... although I am not much aware about the OpenRoad project, the pictures are familiar! so my questions about the number of system gates, node size and the maximum frequency that DarkRISCV can reach on your tools! :)

oh, from the plot it seems that the worst path is around 1.95ns, so it will reach no more than 500MHz... which is pretty good for a 3-stage pipeline core! 🔥

@oharboe oharboe deleted the boards-openroad branch March 15, 2025 21:45
@oharboe
Copy link
Contributor Author

oharboe commented Mar 15, 2025

@samsoniuk Well, take the GUI for a spin. All you have to do is to follow the instructions :-) Install Bazelisk and run the command. Assuming you are on x86 Linux....

@samsoniuk
Copy link
Member

yeah! I will check it! :)

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

Successfully merging this pull request may close these issues.

2 participants