Skip to content

Suite update - Added designs (Vortex, NVDLA, coralnpu), changed suite update logic - #37

Closed
bgoldbug wants to merge 48 commits into
mainfrom
suite-update
Closed

bgoldbug wants to merge 48 commits into
mainfrom
suite-update

Conversation

@bgoldbug

@bgoldbug bgoldbug commented Mar 20, 2026 •

Copy link
Copy Markdown
Collaborator
  • Changed dev command into update_rtl command. Now, after tracking design updates via update_rtl, the user will be notified that RTL has been updated, and the flow is no longer run after changes are tracked.
    • DEV_FLAG logic was removed from verilog.mk files (and any file generation logic for that matter), and Verilog generation for all designs was incorporated into the setup.sh script. This is NECESSARY, as update_rtl no longer runs the flow, thus verilog.mk won't be triggered by update_rtl.
      • verilog.mk now only holds technology-independent Verilog RTL logic.
      • This should also help any issues using update_rtl.sh for Bazel.
  • Added vortex, coralnpu, and NVDLA (small configuration) partitions to the suite (ASAP7).
  • Added FakeRAM submodule and corresponding setup logic
  • Updated README and added HighTide logo
  • Consolidated Markdown documents into a docs folder
  • Added a GitHub action that checks if required design files were added

bgoldbug and others added 30 commits December 31, 2025 01:13
Adding updated asap7 config for NyuziProcessor design (and minor tweaks to all verilog.mk files)
- Updated verilog.mk files to use a single file location (whether running `make dev` or `make`)
Signed-off-by: Benjamin Goldblatt <137117274+bgoldbug@users.noreply.github.com>
bgoldbug and others added 18 commits March 18, 2026 20:06
- Merged from personal Suite update branch to VLSIDA HighTide branch
- Updated FakeRAM module init, inside `setup.sh`
- Updated documentation for README and `update_rtl` command info
Updated link to the design document for clarity.

Signed-off-by: Benjamin Goldblatt <137117274+bgoldbug@users.noreply.github.com>
Signed-off-by: Benjamin Goldblatt <137117274+bgoldbug@users.noreply.github.com>
- Added HighTide logo to title of README
- Updated logo style and changed from png to svg for quality
@mguthaus

Copy link
Copy Markdown
Contributor

Closing in favor of smaller follow-up PRs.

This branch predates the Bazel flow migration on main (#45+), and the new designs added here (vortex, coralnpu, NVDLA) need Bazel BUILD files to work in the current flow. The Make-flow-only infra changes (Makefile, settings.mk, setup.sh, runorfs.sh renames, dev → update_rtl, docs reorg) have also been largely superseded by the Bazel migration.

Rather than force this large branch through a messy merge, we'll re-land the new designs individually using the current new-design workflow:

  • Vortex (asap7) — separate PR
  • CoralNPU (asap7) — separate PR
  • NVDLA (asap7) — separate PR(s), one per partition as appropriate

The suite-update branch is preserved as reference for the original RTL captures and SRAM LEF/LIB files. Thanks @bgoldbug for the original work — we'll carry it forward, just split up and adapted to Bazel.

@mguthaus mguthaus closed this Apr 18, 2026
mguthaus added a commit that referenced this pull request Apr 18, 2026
Points designs/src/vortex/dev/repo at upstream vortexgpgpu/vortex.git
(commit 31e4765d — matches the snapshot captured in the closed PR #37
suite-update branch). Dev-mode regeneration requires the user to init
this submodule first:

    git submodule update --init designs/src/vortex/dev/repo
mguthaus added a commit that referenced this pull request Apr 18, 2026
Five top-level partitions of NVDLA nv_small. All share the same vmod
RTL via DESIGN_NICKNAME=NVDLA + DESIGN_RESULTS_NAME=NVDLA_partition_X
in config.mk; on the Bazel side each partition is a separate
hightide_design() that pulls //designs/src/NVDLA:rtl with its own SDC
and FakeRAM subset.

Per-partition tuning (from closed PR #37):
  a: util=55, density+0.20, halo=5, FakeRAM={256x16,272x16}
  c: util=50, density+0.25, halo=2, FakeRAM={64x256,64x16,6x128,
                                             66x8,11x128}
  m: util=55, density+0.20, halo=5, no FakeRAM
  o: util=45, density+0.15, halo=7, FakeRAM={18x128,8x256,4x256,
                                             7x256,66x64,15x80,22x60,
                                             32x128,9x80}, two clocks
                                             (core 2000ps + falcon 2500ps)
  p: util=45, density+0.25, halo=5, FakeRAM={16x160,65x160,14x80,66x80}

All partitions: SYNTH_HIERARCHICAL=1, TNS_END_PERCENT=100, single
1500ps nvdla_core_clk except partition_o.
@mguthaus
mguthaus deleted the suite-update branch May 2, 2026 19:26
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