Remove potentially misleading comments + allow one-layer core ring

The-OpenROAD-Project · Dec 14, 2023 · ac4113e · ac4113e
1 parent ec37b61
commit ac4113e
Showing 1 changed file with 16 additions and 9 deletions.
diff --git a/scripts/openroad/common/pdn_cfg.tcl b/scripts/openroad/common/pdn_cfg.tcl
@@ -44,7 +44,6 @@ set_voltage_domain -name CORE -power $::env(VDD_NET) -ground $::env(GND_NET) \
     -secondary_power $secondary
 
 if { $::env(FP_PDN_MULTILAYER) == 1 } {
-    # Used if the design is the core of the chip
     define_pdn_grid \
         -name stdcell_grid \
         -starts_with POWER \
@@ -73,7 +72,6 @@ if { $::env(FP_PDN_MULTILAYER) == 1 } {
         -grid stdcell_grid \
         -layers "$::env(FP_PDN_VERTICAL_LAYER) $::env(FP_PDN_HORIZONTAL_LAYER)"
 } else {
-    # Used if the design is a macro in the core
     define_pdn_grid \
         -name stdcell_grid \
         -starts_with POWER \
@@ -86,7 +84,7 @@ if { $::env(FP_PDN_MULTILAYER) == 1 } {
         -width $::env(FP_PDN_VWIDTH) \
         -pitch $::env(FP_PDN_VPITCH) \
         -offset $::env(FP_PDN_VOFFSET) \
-        -starts_with POWER
+        -starts_with POWER -extend_to_core_ring
 }
 
 # Adds the standard cell rails if enabled.
@@ -106,12 +104,21 @@ if { $::env(FP_PDN_ENABLE_RAILS) == 1 } {
 
 # Adds the core ring if enabled.
 if { $::env(FP_PDN_CORE_RING) == 1 } {
-    add_pdn_ring \
-        -grid stdcell_grid \
-        -layers "$::env(FP_PDN_VERTICAL_LAYER) $::env(FP_PDN_HORIZONTAL_LAYER)" \
-        -widths "$::env(FP_PDN_CORE_RING_VWIDTH) $::env(FP_PDN_CORE_RING_HWIDTH)" \
-        -spacings "$::env(FP_PDN_CORE_RING_VSPACING) $::env(FP_PDN_CORE_RING_HSPACING)" \
-        -core_offset "$::env(FP_PDN_CORE_RING_VOFFSET) $::env(FP_PDN_CORE_RING_HOFFSET)"
+    if { $::env(FP_PDN_MULTILAYER) == 1 } {
+        add_pdn_ring \
+            -grid stdcell_grid \
+            -layers "$::env(FP_PDN_VERTICAL_LAYER) $::env(FP_PDN_HORIZONTAL_LAYER)" \
+            -widths "$::env(FP_PDN_CORE_RING_VWIDTH) $::env(FP_PDN_CORE_RING_HWIDTH)" \
+            -spacings "$::env(FP_PDN_CORE_RING_VSPACING) $::env(FP_PDN_CORE_RING_HSPACING)" \
+            -core_offset "$::env(FP_PDN_CORE_RING_VOFFSET) $::env(FP_PDN_CORE_RING_HOFFSET)"
+    } else {
+        add_pdn_ring \
+            -grid stdcell_grid \
+            -layers "$::env(FP_PDN_VERTICAL_LAYER)" \
+            -widths "$::env(FP_PDN_CORE_RING_VWIDTH)" \
+            -spacings "$::env(FP_PDN_CORE_RING_VSPACING)" \
+            -core_offset "$::env(FP_PDN_CORE_RING_VOFFSET)"
+    }
 }
 
 define_pdn_grid \