Skip to content

Tighten timing constraints on swerv_wrapper, coyote and jpeg #4470

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
jorge-ferreira-pii wants to merge 4 commits into
base: master
Choose a base branch
from
+852 −842
Open

Tighten timing constraints on swerv_wrapper, coyote and jpeg #4470

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
a783791
Tighten timing constraints on swerv_wrapper, coyote and jpeg
jorge-ferreira-pii Aug 25, 2026
4b9c961
Update rules-base.json metrics for coyote, jpeg, and swerv_wrapper
jorge-ferreira-pii Aug 26, 2026
eac8378
Trigger CI
jorge-ferreira-pii Aug 26, 2026
4344d6f
Update asap7 swerv_wrapper config, constraint, and rules-base
jorge-ferreira-pii Aug 28, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions flow/designs/asap7/swerv_wrapper/config.mk
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -52,10 +52,10 @@ export SDC_FILE = $(DESIGN_HOME)/$(PLATFORM)/swerv_wrapper/constraint.sdc
export ADDITIONAL_LEFS = $(sort $(wildcard $(DESIGN_HOME)/$(PLATFORM)/swerv_wrapper/lef/*.lef))
export ADDITIONAL_LIBS = $(sort $(wildcard $(DESIGN_HOME)/$(PLATFORM)/swerv_wrapper/lib/*.lib))

export CORE_UTILIZATION = 30
export CORE_UTILIZATION = 55

export IO_CONSTRAINTS = $(DESIGN_HOME)/$(PLATFORM)/swerv_wrapper/io.tcl
export PLACE_DENSITY_LB_ADDON = 0.20
export PLACE_DENSITY_LB_ADDON = 0.30

export ROUTING_LAYER_ADJUSTMENT = 0.2

Expand Down
6 changes: 3 additions & 3 deletions flow/designs/asap7/swerv_wrapper/constraint.sdc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,13 +2,13 @@ current_design swerv_wrapper

set clk_name core_clock
set clk_port_name clk
set clk_period 1600
set clk_period 1100

# Match the old set_input/output_delay = 0.2 * clk_period budget, as
# optimization targets only (no set_input/output_delay — see rationale in
# $PLATFORM_DIR/constraints.sdc).
set in2reg_max [expr { $clk_period * 0.8 }]
set reg2out_max [expr { $clk_period * 0.8 }]
set in2reg_max [expr { $clk_period * 0.6 }]
set reg2out_max [expr { $clk_period * 0.6 }]
set in2out_max [expr { $clk_period * 0.6 }]

source $::env(PLATFORM_DIR)/constraints.sdc
40 changes: 20 additions & 20 deletions flow/designs/asap7/swerv_wrapper/rules-base.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,63 +18,63 @@
"compare": "=="
},
"placeopt__design__instance__area": {
"value": 54927,
"value": 54564,
"compare": "<="
},
"placeopt__design__instance__count__stdcell": {
"value": 141318,
"value": 131145,
"compare": "<="
},
"detailedplace__design__violations": {
"value": 0,
"compare": "=="
},
"cts__design__instance__count__setup_buffer": {
"value": 12288,
"value": 11404,
"compare": "<="
},
"cts__design__instance__count__hold_buffer": {
"value": 12288,
"value": 11404,
"compare": "<="
},
"cts__timing__setup__ws": {
"value": -80.0,
"value": -401.0,
"compare": ">="
},
"cts__timing__setup__tns": {
"value": -320.0,
"value": -61200.0,
"compare": ">="
},
"cts__timing__hold__ws": {
"value": -80.0,
"value": -55.0,
"compare": ">="
},
"cts__timing__hold__tns": {
"value": -320.0,
"value": -220.0,
"compare": ">="
},
"globalroute__antenna_diodes_count": {
"value": 101,
"value": 100,
"compare": "<="
},
"globalroute__timing__setup__ws": {
"value": -80.0,
"value": -395.0,
"compare": ">="
},
"globalroute__timing__setup__tns": {
"value": -320.0,
"value": -51200.0,
"compare": ">="
},
"globalroute__timing__hold__ws": {
"value": -80.0,
"value": -55.0,
"compare": ">="
},
"globalroute__timing__hold__tns": {
"value": -320.0,
"value": -220.0,
"compare": ">="
},
"detailedroute__route__wirelength": {
"value": 1287970,
"value": 1067592,
"compare": "<="
},
"detailedroute__route__drc_errors": {
Expand All @@ -86,27 +86,27 @@
"compare": "<="
},
"detailedroute__antenna_diodes_count": {
"value": 101,
"value": 100,
"compare": "<="
},
"finish__timing__setup__ws": {
"value": -80.0,
"value": -375.0,

@eder-matheus eder-matheus Aug 28, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The jump seems to large. Maybe we could aim to -100, so WNS and specially TNS don't have such big jumpes.

"compare": ">="
},
"finish__timing__setup__tns": {
"value": -320.0,
"value": -49100.0,
"compare": ">="
},
"finish__timing__hold__ws": {
"value": -80.0,
"value": -55.0,
"compare": ">="
},
"finish__timing__hold__tns": {
"value": -320.0,
"value": -220.0,
"compare": ">="
},
"finish__design__instance__area": {
"value": 55372,
"value": 55103,
"compare": "<="
}
}
Loading