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

timing closure for hosted revision #946

Open
ssa-tii opened this issue Sep 27, 2023 · 1 comment
Open

timing closure for hosted revision #946

ssa-tii opened this issue Sep 27, 2023 · 1 comment

Comments

@ssa-tii
Copy link

ssa-tii commented Sep 27, 2023

Hello,

I built latest commit [9fc57e6] for hosted revision and found timing violations:

  • setup in rx_clock domain
  • recovery in pclk domain

See resulting static timing analysis report (hosted.sta.zip)

The recovery violation can be fixed by correcting false_path objects in bladerf.sdc (see bladerf.zip attachment), but the setup violation of ~2ns for the slow corner means reducing rx_clock to 46.8M samples/sec.

Quartus version 17.1

Versions of ALTLVDS_RX and ALTLVDS_TX : v17.1.0 Build 590 SJ Lite Edition.

Could you please clarify if this is expected behavior. We have some improvements to setup timing (-1ns) (by modifying Quartus optimization settings) and can raise a pull request if desired.

Static timing analysis report (hosted.sta.zip)
bladerf.sdc (bladerf.zip)

@ssa-tii ssa-tii closed this as completed Sep 27, 2023
@ssa-tii ssa-tii reopened this Sep 27, 2023
@ssa-tii ssa-tii closed this as completed Sep 27, 2023
@ssa-tii ssa-tii reopened this Sep 28, 2023
@rghilduta
Copy link
Collaborator

@ssa-tii we're looking at potentially updating the SDC. Curious if you had a pull request for the false_path and rx_clock you mentioned? Also, the timing closure values may differ a bit with a newer version of Quartus, I'd suggest using at least version 20.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants