[fpga,ci] Remove stub MMI file generation
Nobody has claimed this yet.
- 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
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- 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