efabless / clear

Apache License 2.0
57 stars 6 forks source link

Cocotb updates #17

Closed M0stafaRady closed 1 year ago

M0stafaRady commented 1 year ago
marwaneltoukhy commented 1 year ago

Why aren't we running verification on the gl netlist post synthesis? This netlist should have the power ports.

M0stafaRady commented 1 year ago

@marwaneltoukhy What do you mean by not running verification post synthesis ?

marwaneltoukhy commented 1 year ago

why aren't you running the verification on the gl netlist coming out of OpenLane?

M0stafaRady commented 1 year ago

We do

marwaneltoukhy commented 1 year ago

Then why are you editing these files? The gl netlist coming out of OpenLane is already powered.

M0stafaRady commented 1 year ago

This is for the RTL files which contain sky130 cells

marwaneltoukhy commented 1 year ago

These aren't RTL files, these are netlists as well. That's why I am asking why aren't you just using the ones coming out of OpenLane.

M0stafaRady commented 1 year ago

These are readable netlist so it's easier to debug them compared with what Openlane provided.

marwaneltoukhy commented 1 year ago

Okay, just make sure that nothing else changed. I'm just worried that we mess with source code, and something might change without us knowing.

These are readable netlist so it's easier to debug them compared with what Openlane provided.

M0stafaRady commented 1 year ago

I checked these changes more than one time but I will really appreciate if someone else also have a look to make sure the script have not corrupted anything.