lowRISC / lowRISC/opentitan

[fpga,ci] Remove stub MMI file generation

Open
#30,749 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Component:CI Component:FPGA
Dominant language
SystemVerilog
Stars
3.6k
Forks
1.1k
Avg merge
2d 22h
Merged PRs (30d)
141

Description

Description

After #30543 was merged, the Vivado MMI file generation is now not necessary on master, since we no longer use updatemem to splice the ROM and OTP images into synthesized bitstreams.

To reduce the scope of that PR whilst still enabling further HW iteration (such as increasing the ROM size), that PR simply replaced the MMI file generation logic with some simple code to generate a stub/dummy file with some garbage contents. Since this is unused, we should remove generation of this dummy file, remove its appearance in the bitstream cache (may be incompatible with the old schema and so may need a bitstream cache schema change with an appropriate version bump), and remove Bazel targets that refer to the MMI file on the master branch.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with hw/top_earlgrey/util/vivado_hook_write_bitstream_pre.tcl, then search the repository for MMI references in bitstream cache definitions and Bazel targets. Remove the unused stub generation and references, updating the cache schema version if required; done means master no longer generates or caches the dummy MMI file.

Written by the indexing model from the issue text.

Assessment

Domain
build-system, ci-cd
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
68/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.