Skip to content

Conversation

@ezelioli
Copy link
Contributor

@ezelioli ezelioli commented Nov 17, 2025

This PR addresses issue #72.

@thommythomaso
Copy link
Collaborator

Started rebasing this branch on other efforts: ez/unmake-rebase

@thommythomaso
Copy link
Collaborator

Depends on #102

@thommythomaso
Copy link
Collaborator

Depends on #111

@phsauter
Copy link
Collaborator

phsauter commented Jan 9, 2026

Some notes:
run_all.sh:

  • without argument does nothing
  • flist: add --relative-path
  • flist: remove -D VERILATOR=1
  • flist: move to synthesis script or output to synthesis dir
  • add simulation

delete ethz.env:

run_synthesis.sh:

  • Could have a interactive argument that starts a Yosys interactive TCL shell
  • There is a second croc.flist here but the one from root is used?
    Use the one from here and change the root to output to here
  • Convert to proper 'form'
  • Map tmp to scratch or have a central top-level tmp like cockpit

yosys_synthesis.tcl:

  • Move some things or alias to common? (eg define a write_netlist or something like that)
  • Reduce dumping to tmp and reports
  • Check for new options added with yosys-slang like cross-hierarchy optimization
  • Maybe warn at the end of some things aren't mapped to cells?

run_verilator.sh:

  • Get binary from args
  • otherwise this is a good example how it should work

run_vsim.sh:

  • Give binary via args
  • per default with gui, add arg to set it to shell-only

run_vsim_netlist.sh:

  • Give binary via args
  • per default with gui, add arg to set it to shell-only

sw:

  • Add a seperate test subdir for timer test and more atomic tests of certain features

run_backend.sh:

  • Still needs simplifying
  • Maybe one function that calls OR with all args and the script to execute is given to it, no checking for checkpoints etc?

finishing.sh:

  • Rename to run_finishing?

@phsauter
Copy link
Collaborator

phsauter commented Jan 12, 2026

  • delete ethz.env
  • env.sh, Copyright to current year
    • Add author
    • change PDK_ROOT comment (it no longer has priority)
    • section headers look a bit awkward
    • I think only the tool directories and the netlist and openroads final def should be set here
      Remove YOSYS_OUT, YOSYS_REPORTS, YOSYS_TMP, OR_OUT, OR_REPORTS, OR_SAVE (move to tools, if possible to the tcl scripts)
  • Update README.mds 'Getting started' and 'Building on Croc'
  • run_vsim.sh
    • Add author
    • "Compile Croc RTL in VSIM" -> delete additional whitespace
    • "Compile Crop post-synthesis netlist in VSIM" -> typo
    • Explain example
    • Remove +acc for non-gui function?
    • Add Siemens permission text (see compile_tech.tcl)
  • run_vsim.sh
    • Add author
  • run_openroad.sh
    • Add author
    • "Run a script relativ to the script direction" -> typo
  • startup.tcl
    • get netlist from env var NETLIST
  • OpenROAD stages
    • Check descriptions in header, required env variables seem outdated
    • "Old versions of repair_timing may swap non-equal pins, deactivated for now to avoid problems" -> Remove and re-add pin-swapping; in README make it clear only oseda-2025.12 is supported.
  • Add latest feature to checkpoint (if it already exists)
  • All, some on-line comment to describe what the functions do would be nice

@phsauter phsauter force-pushed the ez/unmake branch 3 times, most recently from 4c837b5 to 2019e22 Compare January 16, 2026 17:15
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.

4 participants