Fixed CTS flow variable hookups in cts.tcl

jeffng-or · jeffng-or · commit 85df13cda762 · 2025-03-21T15:26:52.000-07:00
updated flow variables doc
CTS buf list fix
fixed incorrect merge
additional rules updates after CI report

Signed-off-by: Jeff Ng &lt;jeffng@precisioninno.com&gt;
diff --git a/docs/user/FlowVariables.md b/docs/user/FlowVariables.md
@@ -75,6 +75,7 @@ configuration file.
 | <a name="CORNER"></a>CORNER| PVT corner library selection. Only available for ASAP7 and GF180 PDKs.| | |
 | <a name="CTS_ARGS"></a>CTS_ARGS| Override `clock_tree_synthesis` arguments.| | |
 | <a name="CTS_BUF_DISTANCE"></a>CTS_BUF_DISTANCE| Distance (in microns) between buffers.| | |
+| <a name="CTS_BUF_LIST"></a>CTS_BUF_LIST| List of cells used to construct the clock tree.| | |
 | <a name="CTS_CLUSTER_DIAMETER"></a>CTS_CLUSTER_DIAMETER| Maximum diameter (in microns) of sink cluster.| 20| |
 | <a name="CTS_CLUSTER_SIZE"></a>CTS_CLUSTER_SIZE| Maximum number of sinks per cluster.| 50| |
 | <a name="CTS_SNAPSHOT"></a>CTS_SNAPSHOT| Creates ODB/SDC files prior to clock net and setup/hold repair.| | |
@@ -307,6 +308,7 @@ configuration file.
 - [CELL_PAD_IN_SITES_DETAIL_PLACEMENT](#CELL_PAD_IN_SITES_DETAIL_PLACEMENT)
 - [CTS_ARGS](#CTS_ARGS)
 - [CTS_BUF_DISTANCE](#CTS_BUF_DISTANCE)
+- [CTS_BUF_LIST](#CTS_BUF_LIST)
 - [CTS_CLUSTER_DIAMETER](#CTS_CLUSTER_DIAMETER)
 - [CTS_CLUSTER_SIZE](#CTS_CLUSTER_SIZE)
 - [CTS_SNAPSHOT](#CTS_SNAPSHOT)
diff --git a/flow/designs/asap7/mock-alu/rules-base.json b/flow/designs/asap7/mock-alu/rules-base.json
@@ -48,11 +48,11 @@
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -515.34,
+        "value": -522.32,
         "compare": ">="
     },
     "finish__design__instance__area": {
-        "value": 1920,
+        "value": 1915,
         "compare": "<="
     },
     "finish__timing__drv__setup_violation_count": {
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/aes/rules-base.json b/flow/designs/gf180/aes/rules-base.json
@@ -48,7 +48,7 @@
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -1.12,
+        "value": -1.29,
         "compare": ">="
     },
     "finish__design__instance__area": {
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/jpeg/rules-base.json b/flow/designs/gf180/jpeg/rules-base.json
@@ -28,7 +28,7 @@
         "compare": "<="
     },
     "globalroute__antenna_diodes_count": {
-        "value": 4,
+        "value": 9,
         "compare": "<="
     },
     "detailedroute__route__wirelength": {
@@ -52,7 +52,7 @@
         "compare": ">="
     },
     "finish__design__instance__area": {
-        "value": 2438978,
+        "value": 2834888,
         "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/aes/rules-base.json b/flow/designs/ihp-sg13g2/aes/rules-base.json
@@ -28,7 +28,7 @@
         "compare": "<="
     },
     "globalroute__antenna_diodes_count": {
-        "value": 282,
+        "value": 440,
         "compare": "<="
     },
     "detailedroute__route__wirelength": {
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/nangate45/bp_be_top/config.mk b/flow/designs/nangate45/bp_be_top/config.mk
@@ -30,3 +30,6 @@ export MACRO_PLACE_HALO = 10 10
 
 export PLACE_DENSITY_LB_ADDON = 0.10
 export TNS_END_PERCENT        = 100
+
+export CTS_CLUSTER_SIZE = 30
+export CTS_CLUSTER_DIAMETER = 50
diff --git a/flow/designs/nangate45/bp_be_top/rules-base.json b/flow/designs/nangate45/bp_be_top/rules-base.json
@@ -52,7 +52,7 @@
         "compare": ">="
     },
     "finish__design__instance__area": {
-        "value": 302367,
+        "value": 301478,
         "compare": "<="
     },
     "finish__timing__drv__setup_violation_count": {
diff --git a/flow/designs/nangate45/swerv/rules-base.json b/flow/designs/nangate45/swerv/rules-base.json
@@ -60,7 +60,7 @@
         "compare": "<="
     },
     "finish__timing__drv__hold_violation_count": {
-        "value": 100,
+        "value": 240,
         "compare": "<="
     },
     "finish__timing__wns_percent_delay": {
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/chameleon/rules-base.json b/flow/designs/sky130hd/chameleon/rules-base.json
@@ -44,7 +44,7 @@
         "compare": "<="
     },
     "detailedroute__antenna_diodes_count": {
-        "value": 69,
+        "value": 132,
         "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/sky130hd/riscv32i/rules-base.json b/flow/designs/sky130hd/riscv32i/rules-base.json
@@ -44,15 +44,15 @@
         "compare": "<="
     },
     "detailedroute__antenna_diodes_count": {
-        "value": 5,
+        "value": 10,
         "compare": "<="
     },
     "finish__timing__setup__ws": {
         "value": -1.36,
         "compare": ">="
     },
     "finish__design__instance__area": {
-        "value": 90213,
+        "value": 104530,
         "compare": "<="
     },
     "finish__timing__drv__setup_violation_count": {
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.
@@ -916,7 +921,8 @@ YOSYS_FLAGS:
     Flags to pass to yosys.
   stages:
     - synth
-  default: -v 3
+  default:
+    -v 3
 FLOW_VARIANT:
   description: >
     Flow variant to use, used in the flow variant directory name.
