efabless / clear

Apache License 2.0
57 stars 6 forks source link

Possible OpenLane version issue #5

Closed abdullahyildiz closed 2 years ago

abdullahyildiz commented 2 years ago

I've tried running the flow using both main branch and io_fix branch but each time flow stops with the following error:

[INFO PDN-0016] Power Delivery Network Generator: Generating PDN
  config: /home/ayildiz/efabless/clear-io_fix/openlane/fpga_core/pdn.tcl
Error: pdn.tcl, 17 TypeError in method 'microns_to_dbu', argument 1 of type 'double'
[ERROR]: during executing openroad script /openlane/scripts/openroad/pdn.tcl
[ERROR]: Exit code: 1
[ERROR]: Last 10 lines:
child process exited abnormally

I also had to make small modifications in the following files to run make harden command:

diff --git a/openlane/cbx_1__0_/pin.cfg b/openlane/cbx_1__0_/pin.cfg
index c369a8a..d75187f 100644
--- a/openlane/cbx_1__0_/pin.cfg
+++ b/openlane/cbx_1__0_/pin.cfg
@@ -36,9 +36,6 @@ top_width_0_height_0__pin_[0-9].*_
 top_width_0_height_0__pin_[1][0-9]_lower.*
 top_width_0_height_0__pin_[0-9]_lower.*

-#N
-top_width_0_height_0__pin_11_lower.*
-
 #S
 bottom_grid_pin.*
diff --git a/openlane/fpga_core/interactive.tcl b/openlane/fpga_core/interactive.tcl
index 960e985..f7e2b80 100644
--- a/openlane/fpga_core/interactive.tcl
+++ b/openlane/fpga_core/interactive.tcl
@@ -75,7 +75,7 @@ add_macro_placement tie_array [expr {$floorplan_x / 2}] [expr {$switches_y(8,8)

 manual_macro_placement f

-detailed_placement
+detailed_placement_or -log $::env(placement_logs)/detailed.log -def $::env(placement_results)/$::env(DESIGN_NAME).def

 gen_pdn

@@ -107,4 +107,4 @@ run_magic_drc
 run_magic_spice_export
 run_lvs

-run_antenna_check; # to verify the above and get a final report
\ No newline at end of file
+run_antenna_check; # to verify the above and get a final report

I've also been informed that the OpenLane tag that is defined in

https://github.com/efabless/clear/blob/f7a6b19f604a677eed6efa6690fddb81fd99a67f/Makefile#L26

points to a very old version.

abdullahyildiz commented 2 years ago

Closing the issue since it is linked to an older commit.