Merge pull request #8505 from Pinata-Consulting/mock-array-floorplan-fixes

Mock array floorplan fixes

Author: maliberty

test/orfs/mock-array/BUILD

@@ -57,15 +57,17 @@ ce_x = 20
 
 ce_y = 20
 
-pitch_x = 20
-
-pitch_y = 22
-
 # Element size is set to multiple of placement grid above
 ce_width = ce_x * placement_grid_x
 
 ce_height = ce_y * placement_grid_y
 
+# Routing by abutment horizontally
+pitch_x = ce_width
+
+# Some routing space vertically for clock routing
+pitch_y = ce_height + (placement_grid_y * 2)
+
 # top level core offset
 margin_x = placement_grid_x
 
@@ -81,13 +83,14 @@ array_spacing_x = margin_x * 2
 
 array_spacing_y = margin_y * 2
 
-# top level core and die size
+# top level core and die size, we need some space around the
+# array to put some flip flops and buffers for top level io pins
 core_width = (
-    2 * margin_x + ((placement_grid_x * pitch_x) * (cols - 1)) + ce_width
+    pitch_x * (cols - 1) + ce_width + margin_x * 2
 )
 
 core_height = (
-    2 * margin_y + ((placement_grid_y * pitch_y) * (rows - 1)) + ce_height
+    pitch_y * (rows - 1) + ce_height + margin_x * 2
 )
 
 die_width = core_width + (array_spacing_x * 2)

test/orfs/mock-array/rules-base.json

@@ -1,6 +1,6 @@
 {
     "synth__design__instance__area__stdcell": {
-        "value": 14398.9,
+        "value": 14315.22,
         "compare": "<="
     },
     "constraints__clocks__count": {
@@ -24,19 +24,19 @@
         "compare": "<="
     },
     "cts__design__instance__count__hold_buffer": {
-        "value": 1521,
+        "value": 1614,
         "compare": "<="
     },
     "globalroute__antenna_diodes_count": {
         "value": 0,
         "compare": "<="
     },
     "detailedroute__route__wirelength": {
-        "value": 128034,
+        "value": 127790,
         "compare": "<="
     },
     "detailedroute__route__drc_errors": {
-        "value": 0,
+        "value": 1,
         "compare": "<="
     },
     "detailedroute__antenna__violating__nets": {
@@ -48,23 +48,23 @@
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -114.39,
+        "value": -115.35,
         "compare": ">="
     },
     "finish__design__instance__area": {
-        "value": 138041,
+        "value": 138047,
         "compare": "<="
     },
     "finish__timing__drv__setup_violation_count": {
         "value": 1040,
         "compare": "<="
     },
     "finish__timing__drv__hold_violation_count": {
-        "value": 159,
+        "value": 152,
         "compare": "<="
     },
     "finish__timing__wns_percent_delay": {
-        "value": -39.79,
+        "value": -40.47,
         "compare": ">="
     }
 }

test/orfs/mock-array/rules-flat.json

@@ -1,6 +1,6 @@
 {
     "synth__design__instance__area__stdcell": {
-        "value": 14398.9,
+        "value": 14315.22,
         "compare": "<="
     },
     "constraints__clocks__count": {
@@ -24,19 +24,19 @@
         "compare": "<="
     },
     "cts__design__instance__count__hold_buffer": {
-        "value": 1521,
+        "value": 1614,
         "compare": "<="
     },
     "globalroute__antenna_diodes_count": {
         "value": 0,
         "compare": "<="
     },
     "detailedroute__route__wirelength": {
-        "value": 128034,
+        "value": 127790,
         "compare": "<="
     },
     "detailedroute__route__drc_errors": {
-        "value": 0,
+        "value": 1,
         "compare": "<="
     },
     "detailedroute__antenna__violating__nets": {
@@ -48,23 +48,23 @@
         "compare": "<="
     },
     "finish__timing__setup__ws": {
-        "value": -114.39,
+        "value": -115.35,
         "compare": ">="
     },
     "finish__design__instance__area": {
-        "value": 138041,
+        "value": 138047,
         "compare": "<="
     },
     "finish__timing__drv__setup_violation_count": {
         "value": 1040,
         "compare": "<="
     },
     "finish__timing__drv__hold_violation_count": {
-        "value": 159,
+        "value": 152,
         "compare": "<="
     },
     "finish__timing__wns_percent_delay": {
-        "value": -39.79,
+        "value": -40.47,
         "compare": ">="
     }
 }