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

wSchedule throws warnings for not being set in Vivado 2018.3 #38

Open
hmaarrfk opened this issue Feb 4, 2019 · 6 comments
Open

wSchedule throws warnings for not being set in Vivado 2018.3 #38

hmaarrfk opened this issue Feb 4, 2019 · 6 comments

Comments

@hmaarrfk
Copy link

hmaarrfk commented Feb 4, 2019

wire [C_CLOG_MUX_INPUTS-1:0] wSchedule[C_NUM_MUXES-1:0][(1<<(3+C_CLOG_MAX_SCHEDULE))-1:0];

Going through the verilog code, it appears to never be set. Did something get dropped in a recent version?

@hmaarrfk hmaarrfk changed the title wSchedule ( or _wTxMuxSelect ) is never set wSchedule throws warnings for not being set in Vivado 2018.3 Feb 4, 2019
@hmaarrfk
Copy link
Author

hmaarrfk commented Feb 4, 2019

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.....

@drichmond
Copy link
Collaborator

drichmond commented Feb 4, 2019 via email

@hmaarrfk
Copy link
Author

hmaarrfk commented Feb 4, 2019

I copied the whole generate statement in the tx_alignment_pipeline but the warning persists

[Synth 8-3848] Net wSchedule[0][31] in module/entity tx_alignment_pipeline does not have driver. ["/home/mark2/git/riffa/fpga/riffa_hdl/tx_alignment_pipeline.v":163]

The design is for PCIE with
C_DATA_WIDTH = 128

@hmaarrfk
Copy link
Author

hmaarrfk commented Feb 4, 2019

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.

@drichmond
Copy link
Collaborator

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.

@hmaarrfk
Copy link
Author

hmaarrfk commented Feb 4, 2019

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.

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