Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Netlist Simulation Issues: Unknown Values/Syntax Errors #518

Open
deepsita opened this issue Jul 31, 2021 · 6 comments
Open

Netlist Simulation Issues: Unknown Values/Syntax Errors #518

deepsita opened this issue Jul 31, 2021 · 6 comments
Labels
blocked This issue is blocked on a bugfix or enhancement of another repository or tool bug Something isn't working Skywater PDK This issue is specific to using OpenLane with Sky130 workaround A workaround exists for this issue

Comments

@deepsita
Copy link

The simulation of synthesized netlist is always XXX. as shown in the image.
postsynth_output

However, the Synthesis is shown successful as shown in the below image of terminal
synthterminal

The procedure to reproduce the same is given in the https://github.com/deepsita/mythcore_synth
git clone https://github.com/deepsita/mythcore_synth.git
cd mythcore_synth/post synth Simulation/
iverilog gls.v mythcore.synth.v primitives.v sky130_fd_sc_hd.v
./a.out
gtkwave gls.vcd

@Manarabdelaty
Copy link
Collaborator

@deepsita I believe the behavioral models are buggy in the original PDK (see google/skywater-pdk#310) . However, you can simulate with the functional models by passing the FUNCTIONAL define to iverilog (you also need to set UNIT_DELAY macro to some value) :
iverilog -DFUNCTIONAL -DUNIT_DELAY=#1 gls.v mythcore.synth.v primitives.v sky130_fd_sc_hd.v

@deepsita
Copy link
Author

deepsita commented Aug 2, 2021

I tried doing the same, and the following errors are shown.
There was a syntax error in 74584 line of the sky130_fd_sc_hd.v, the cell at this line is oai, and the screenshot of the same is also attached.

Screenshot from 2021-08-02 20-09-57
Screenshot from 2021-08-02 20-10-38

@Manarabdelaty
Copy link
Collaborator

@deepsita See issues (google/skywater-pdk#298, google/skywater-pdk#297), The current solution (until they are addressed in the skywater-pdk repo) is to manually comment out the wire 1; statement and correct endif SKY130_FD_SC_HD__LPFLOW_BLEEDER_FUNCTIONAL_V to endif // SKY130_FD_SC_HD__LPFLOW_BLEEDER_FUNCTIONAL_V

@donn donn added Skywater PDK This issue is specific to using OpenLane with Sky130 wontfix This will not be worked on labels Aug 15, 2021
@donn donn added blocked This issue is blocked on a bugfix or enhancement of another repository or tool workaround A workaround exists for this issue and removed wontfix This will not be worked on labels Jan 3, 2022
@donn donn changed the title Post Synthesis Simulation shows XXX Netlist Simulation Issues: Unknown Values/Syntax Errors Jun 2, 2022
@donn donn added the bug Something isn't working label Jun 23, 2022
@donn
Copy link
Collaborator

donn commented Sep 6, 2022

For posterity's sake: the wire 1 issue has been long fixed, but the `endif most definitely has not.

@Asma-Mohsin
Copy link

i think you should first write powered verilog of this synthesis file cause currently your cells have no power and ground connections. thus going on unknown state.

@Jboyrox
Copy link

Jboyrox commented Apr 30, 2024

Hi I am also facing the same issue . Has there been any updates to this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked This issue is blocked on a bugfix or enhancement of another repository or tool bug Something isn't working Skywater PDK This issue is specific to using OpenLane with Sky130 workaround A workaround exists for this issue
Projects
None yet
Development

No branches or pull requests

5 participants