Skip to content
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.

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

mgmt_core/user_irq input_delay missing? #11

Open
mbalestrini opened this issue Jan 4, 2022 · 2 comments
Open

mgmt_core/user_irq input_delay missing? #11

mbalestrini opened this issue Jan 4, 2022 · 2 comments
Assignees
Labels
SDC Timing files

Comments

@mbalestrini
Copy link

I'm running STA on the whole caravel+user_project_area to try to detect any issues we might get from the integreation of both.
I have a question about the mgmt_core.user_irq inputs:

In the new core there seems to be a lot of hold delays cells to handle outputs from the user_project_wrapper (la_data_out , wbs_dat_o) into the mgmt_core that help even out the difference in clocks to avoid hold violations. But it seems there are no delay cells on the user_irq paths, and that is causing some hold violations in our design.
Taking a quick look at the mgmt_core.sdc I see that it sets an input_delay of 1.0 to all the inputs (mprj_dat_i, la_input) but not for the user_irqs. Is there a reason behind that? I saw that the user_irq inputs use double flip-flop to sync the signal but when the signal comes from a synced cell in the projet area it might probably always caused a hold violation on the first ff

@mbalestrini
Copy link
Author

Here's a thread on the slack where I posted about this topic:
https://skywater-pdk.slack.com/archives/C02K4RD241Y/p1640276417199900

@jeffdi jeffdi moved this to Todo in Caravel Redesign Sep 27, 2022
@RTimothyEdwards RTimothyEdwards added the SDC Timing files label Oct 4, 2022
@azwefabless
Copy link

Addressed with a preliminary input constraint. Not clear that the general problem is addressed with clear boundary constraints.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
SDC Timing files
Projects
Status: No status
Development

No branches or pull requests

4 participants