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

fix critical warning for Vivado #162

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sequencer
Copy link
Contributor

@sequencer sequencer commented Apr 25, 2021

@sequencer sequencer changed the title add inline blackbox for PowerOnResetFPGAOnly. fix critical warning for Vivado Apr 25, 2021
@erikdanie
Copy link
Collaborator

erikdanie commented Apr 26, 2021

What is the critical warning that is being fixed here?
I want to make sure we are cautious about merging stuff like this, I will want to hand run some tests.

@sequencer
Copy link
Contributor Author

sequencer commented Apr 26, 2021

What is the critical warning that is being fixed here?

I’m still tuning issue here. I’ll illustrate the problem tomorrow.

This avoid:
[Place 30-722] Terminal 'jtag_jtag_XXX' has IOB constraint set to TRUE,
but it is either not connected to a FLOP element or the connected FLOP
element could not be brought into the I/O

[Constraints 18-841] Port jtag_jtag_XXX has IOB constraint. But it
drives multiple flops. Please specify IOB constraint on individual flop.
The IOB constraint on port will be ignored.
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

Successfully merging this pull request may close these issues.

2 participants