Fixed CTS flow variable hookups in cts.tcl

jeffng-or · jeffng-or · commit 5a9f6ae13653 · 2025-03-21T08:59:58.000-07:00
Signed-off-by: Jeff Ng &lt;jeffng@precisioninno.com&gt;
diff --git a/flow/designs/asap7/riscv32i/config.mk b/flow/designs/asap7/riscv32i/config.mk
@@ -28,3 +28,6 @@ export IO_CONSTRAINTS     = $(DESIGN_HOME)/$(PLATFORM)/$(DESIGN_NICKNAME)/io.tcl
 export MACRO_PLACE_HALO    = 2 2
 
 export TNS_END_PERCENT   = 100
+
+export CTS_CLUSTER_SIZE = 10
+export CTS_CLUSTER_DIAMETER = 50
diff --git a/flow/designs/asap7/riscv32i/rules-base.json b/flow/designs/asap7/riscv32i/rules-base.json
@@ -48,7 +48,7 @@
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -113.89,
+        "value": -100.98,
         "compare": ">="
     },
     "finish__design__instance__area": {
@@ -64,7 +64,7 @@
         "compare": "<="
     },
     "finish__timing__wns_percent_delay": {
-        "value": -14.39,
+        "value": -13.99,
         "compare": ">="
     }
 }
diff --git a/flow/designs/gf180/ibex/rules-base.json b/flow/designs/gf180/ibex/rules-base.json
@@ -52,7 +52,7 @@
         "compare": ">="
     },
     "finish__design__instance__area": {
-        "value": 899459,
+        "value": 1072331,
         "compare": "<="
     },
     "finish__timing__drv__setup_violation_count": {
diff --git a/flow/designs/gf180/riscv32i/rules-base.json b/flow/designs/gf180/riscv32i/rules-base.json
@@ -52,7 +52,7 @@
         "compare": ">="
     },
     "finish__design__instance__area": {
-        "value": 435127,
+        "value": 522597,
         "compare": "<="
     },
     "finish__timing__drv__setup_violation_count": {
diff --git a/flow/designs/ihp-sg13g2/ibex/rules-base.json b/flow/designs/ihp-sg13g2/ibex/rules-base.json
@@ -28,7 +28,7 @@
         "compare": "<="
     },
     "globalroute__antenna_diodes_count": {
-        "value": 1215,
+        "value": 1017,
         "compare": "<="
     },
     "detailedroute__route__wirelength": {
@@ -40,15 +40,15 @@
         "compare": "<="
     },
     "detailedroute__antenna__violating__nets": {
-        "value": 48,
+        "value": 43,
         "compare": "<="
     },
     "detailedroute__antenna_diodes_count": {
         "value": 5,
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -0.55,
+        "value": -0.5,
         "compare": ">="
     },
     "finish__design__instance__area": {
diff --git a/flow/designs/ihp-sg13g2/riscv32i/rules-base.json b/flow/designs/ihp-sg13g2/riscv32i/rules-base.json
@@ -40,15 +40,15 @@
         "compare": "<="
     },
     "detailedroute__antenna__violating__nets": {
-        "value": 21,
+        "value": 30,
         "compare": "<="
     },
     "detailedroute__antenna_diodes_count": {
         "value": 5,
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -0.72,
+        "value": -0.54,
         "compare": ">="
     },
     "finish__design__instance__area": {
@@ -64,7 +64,7 @@
         "compare": "<="
     },
     "finish__timing__wns_percent_delay": {
-        "value": -19.84,
+        "value": -15.91,
         "compare": ">="
     }
 }
diff --git a/flow/designs/sky130hd/aes/config.mk b/flow/designs/sky130hd/aes/config.mk
@@ -18,3 +18,5 @@ export FASTROUTE_TCL = $(DESIGN_HOME)/$(PLATFORM)/$(DESIGN_NICKNAME)/fastroute.t
 
 export REMOVE_ABC_BUFFERS = 1
 
+export CTS_CLUSTER_SIZE = 20
+export CTS_CLUSTER_DIAMETER = 50
diff --git a/flow/designs/sky130hd/aes/rules-base.json b/flow/designs/sky130hd/aes/rules-base.json
@@ -28,7 +28,7 @@
         "compare": "<="
     },
     "globalroute__antenna_diodes_count": {
-        "value": 180,
+        "value": 129,
         "compare": "<="
     },
     "detailedroute__route__wirelength": {
@@ -44,7 +44,7 @@
         "compare": "<="
     },
     "detailedroute__antenna_diodes_count": {
-        "value": 50,
+        "value": 28,
         "compare": "<="
     },
     "finish__timing__setup__ws": {
diff --git a/flow/designs/sky130hd/ibex/config.mk b/flow/designs/sky130hd/ibex/config.mk
@@ -52,3 +52,6 @@ export TNS_END_PERCENT = 100
 export FASTROUTE_TCL = $(DESIGN_HOME)/$(PLATFORM)/$(DESIGN_NICKNAME)/fastroute.tcl
 
 export REMOVE_ABC_BUFFERS = 1
+
+export CTS_CLUSTER_SIZE = 20
+export CTS_CLUSTER_DIAMETER = 50
diff --git a/flow/designs/sky130hd/ibex/rules-base.json b/flow/designs/sky130hd/ibex/rules-base.json
@@ -28,7 +28,7 @@
         "compare": "<="
     },
     "globalroute__antenna_diodes_count": {
-        "value": 34,
+        "value": 58,
         "compare": "<="
     },
     "detailedroute__route__wirelength": {
@@ -44,7 +44,7 @@
         "compare": "<="
     },
     "detailedroute__antenna_diodes_count": {
-        "value": 70,
+        "value": 54,
         "compare": "<="
     },
     "finish__timing__setup__ws": {
diff --git a/flow/designs/sky130hs/aes/config.mk b/flow/designs/sky130hs/aes/config.mk
@@ -13,3 +13,7 @@ export PLACE_DENSITY_LB_ADDON = 0.25
 export TNS_END_PERCENT = 100
 
 export REMOVE_ABC_BUFFERS = 1
+
+export CTS_CLUSTER_SIZE = 10
+export CTS_CLUSTER_DIAMETER = 50
+
diff --git a/flow/designs/sky130hs/aes/rules-base.json b/flow/designs/sky130hs/aes/rules-base.json
@@ -44,11 +44,11 @@
         "compare": "<="
     },
     "detailedroute__antenna_diodes_count": {
-        "value": 45,
+        "value": 33,
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -0.47,
+        "value": -0.14,
         "compare": ">="
     },
     "finish__design__instance__area": {
diff --git a/flow/designs/sky130hs/ibex/rules-base.json b/flow/designs/sky130hs/ibex/rules-base.json
@@ -28,11 +28,11 @@
         "compare": "<="
     },
     "globalroute__antenna_diodes_count": {
-        "value": 30,
+        "value": 20,
         "compare": "<="
     },
     "detailedroute__route__wirelength": {
-        "value": 787855,
+        "value": 924570,
         "compare": "<="
     },
     "detailedroute__route__drc_errors": {
@@ -48,7 +48,7 @@
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -0.46,
+        "value": -0.41,
         "compare": ">="
     },
     "finish__design__instance__area": {
diff --git a/flow/designs/sky130hs/jpeg/config.mk b/flow/designs/sky130hs/jpeg/config.mk
@@ -14,3 +14,5 @@ export CORE_MARGIN = 2
 export PLACE_DENSITY_LB_ADDON = 0.25
 export TNS_END_PERCENT = 100
 
+export CTS_CLUSTER_SIZE = 30
+export CTS_CLUSTER_DIAMETER = 50
diff --git a/flow/designs/sky130hs/jpeg/rules-base.json b/flow/designs/sky130hs/jpeg/rules-base.json
@@ -40,11 +40,11 @@
         "compare": "<="
     },
     "detailedroute__antenna__violating__nets": {
-        "value": 3,
+        "value": 1,
         "compare": "<="
     },
     "detailedroute__antenna_diodes_count": {
-        "value": 810,
+        "value": 674,
         "compare": "<="
     },
     "finish__timing__setup__ws": {
diff --git a/flow/designs/sky130hs/riscv32i/rules-base.json b/flow/designs/sky130hs/riscv32i/rules-base.json
@@ -48,7 +48,7 @@
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -0.32,
+        "value": -0.6,
         "compare": ">="
     },
     "finish__design__instance__area": {
diff --git a/flow/scripts/cts.tcl b/flow/scripts/cts.tcl
@@ -18,9 +18,10 @@ set cts_args [list \
           -sink_clustering_enable \
           -balance_levels]
 
-append_env_var cts_args -distance_between_buffers CTS_BUF_DISTANCE 1
-append_env_var cts_args -sink_clustering_size CTS_CLUSTER_SIZE 1
-append_env_var cts_args -sink_clustering_max_diameter CTS_CLUSTER_DIAMETER 1
+append_env_var cts_args CTS_BUF_DISTANCE -distance_between_buffers 1
+append_env_var cts_args CTS_CLUSTER_SIZE -sink_clustering_size 1
+append_env_var cts_args CTS_CLUSTER_DIAMETER -sink_clustering_max_diameter 1
+append_env_var cts_args CTS_BUF_LIST -buf_list 1
 
 if {[env_var_exists_and_non_empty CTS_ARGS]} {
   set cts_args $::env(CTS_ARGS)
diff --git a/flow/scripts/variables.yaml b/flow/scripts/variables.yaml
@@ -721,6 +721,11 @@ CTS_BUF_DISTANCE:
     Distance (in microns) between buffers.
   stages:
     - cts
+CTS_BUF_LIST:
+  description: |
+    List of cells used to construct the clock tree.
+  stages:
+    - cts
 CTS_CLUSTER_DIAMETER:
   description: >
     Maximum diameter (in microns) of sink cluster.
@@ -913,10 +918,9 @@ RUN_LOG_NAME_STEM:
   default: run
 YOSYS_FLAGS:
   description: >
-    Flags to pass to yosys.
+    Additional flags to pass to yosys.
   stages:
     - synth
-  default: -v 3
 FLOW_VARIANT:
   description: >
     Flow variant to use, used in the flow variant directory name.

Original file line number	Diff line number	Diff line change
`@@ -18,3 +18,5 @@ export FASTROUTE_TCL = $(DESIGN_HOME)/$(PLATFORM)/$(DESIGN_NICKNAME)/fastroute.t`
`18`	`18`
`19`	`19`	`export REMOVE_ABC_BUFFERS = 1`
`20`	`20`
	`21`	`+export CTS_CLUSTER_SIZE = 20`
	`22`	`+export CTS_CLUSTER_DIAMETER = 50`