Mohamed Hosni
Mohamed Hosni
### Description Most of the flow and PDK variables in the documentation are not needed to be modified in the usual hardening process of macros. Some users might get lost...
`set_clock_latency` does add the clock latency to the hold times of each pin. This should be ignored the same way `set_input_delay` is ignored. Also, `set_input_transition` affects the hold/setup values for...
`caravel_clocking` macro cannot be reproduced using this [OPENLANE_VERSION](https://github.com/efabless/caravel/blob/main/signoff/caravel_clocking/OPENLANE_VERSION). A batch file is needed to be used on OpenLane. This macro was not updated in the PR https://github.com/efabless/caravel/pull/434 that updated OpenLane...
Cleanup
Removed a lot of obsolete/unused files after the latest updates for caravel and caravan.
There is mismatch in the OpenLane's user_project_wrapper configuration here and in [caravel_user_project](https://github.com/efabless/caravel_user_project) For example: `FP_PDN_CORE_RING_VOFFSET` is set to `14` here https://github.com/efabless/caravel/blob/edc22cc32547527861a12aeddb568f1ae591cc92/openlane/user_project_wrapper/fixed_wrapper_cfgs.tcl#L45 `FP_PDN_CORE_RING_VOFFSET` is set to `12.45` in caravel_user_project [caravel/openlane/user_project_wrapper/fixed_wrapper_cfgs.tcl :...
### Description Currently, those are recognized as errors in both pre-synthesis and post-synthesis checks. I think it better be a Warning in pre-synthesis and an error in post-synthesis as the...
### Description title ### Expected Behavior Deprecated configuration warnings appear at the end of the flow and in the warnings.log file ### Environment report ```sh Failed to get Docker info:...
It would be better to provide the users an SDC file PnR and another one for sign-off rather than only 1 base SDC file. This way the users will be...
Fixing a link in `docs/source/index.rst`
The difference between input delays and clock latency is `6ns` in the worst case, this is very aggressive. In order to fix the resulting hold violations, 6ns of delay cells...