-
Notifications
You must be signed in to change notification settings - Fork 315
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
wSchedule throws warnings for not being set in Vivado 2018.3 #38
Comments
I guess they are set in In schedules.vh there is a comment stating that it is a sin to do what they did..... |
Guilty
… On Feb 4, 2019, at 9:35 AM, Mark Harfouche ***@***.***> wrote:
I guess they are set in schedules.vh which is included.
In schedules.vh there is a comment stating that it is a sin to do what they did.....
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub <#38 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AGFUdC06rNCQKQZsQ7ousfkdnuLh1lF5ks5vKG9MgaJpZM4ahrIk>.
|
I copied the whole generate statement in the tx_alignment_pipeline but the warning persists
The design is for PCIE with |
Do you think these warnings are benign? They seem quite serious, though there are some other ones that scare me more for now. I'm trying to port to Ultrascale+, so these warnings are helping me tie down loose ends. |
Are you porting to 256 bits? I suspect these warnings are fine. The issue is that I didn't define the complete ROM for wSchedule, and Vivado is informing you of this. |
Not yet. I have a ZCU106, it only has a PCIE 3.0 x4 connector. So i set the core to 128 bits. Some things changed annoyingly (one hot vs number as outputs for configurations for the PCIe core). I'm down to 0 critical warnings, and about 4-7 lines of regular "warnings" for riffa. |
riffa/fpga/riffa_hdl/tx_alignment_pipeline.v
Line 163 in 4e9d3c8
Going through the verilog code, it appears to never be set. Did something get dropped in a recent version?
The text was updated successfully, but these errors were encountered: