search

efabless / caravel-gf180mcu

This repository is the GF180MCU port of Caravel. For more information about Caravel, see the original repo at https://github.com/efabless/caravel.
Apache License 2.0
7 stars 8 forks source link

tapeout seems to use .mag rather than .gds #169

Open gatecat opened 1 year ago

gatecat commented 1 year ago

Previous tapeouts, at least MPW5 on sky130, and also moreover the precheck for gfmpw0 seemed to be running on the .gds view in the submission repo.

To avoid my repo becoming too big and save upload time I only ever committed updated GDS and my generated tapeout caravel gds/oas now seems to contain the example project, from the stale .mag in my repo.

jeffdi commented 11 months ago

Needs review.

d-mitch-bailey commented 10 months ago

@gatecat Thanks for reporting. There may have been some problems with earlier versions of the ship target of caravel-gf180mcu/Makefile.

You are correct in that the gds should override the user_project_wrapper mag file. However, it is important to remember that recreating lower levels of the hierarchy will not automatically be reflected in the top user_project_wrapper. You must also recreate each parent level gds in order, from lowest to highest, to ensure that your changes appear in the final user_project_wrapper gds.

If you have recreated the gds at each level, can you verify that there is still a problem with tapeout using stale mag data?