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

Combinatorial Loop Alert while Generating bitstream for vicuna using CV32E40X as a scalar core #111

Open
Nikhil-311293 opened this issue May 5, 2023 · 4 comments

Comments

@Nikhil-311293
Copy link

Nikhil-311293 commented May 5, 2023

Hello @michael-platzer and vicuna community,

I am trying to run the demo version of vicuna using cv32e40x as a scalar core. While generating the bitstream (Genesys2 Target) for the same I am getting following errors. I've followed the exact same process as mentioned in the demo/ and sw/ repositories.

vicuna_combinatorial_loop

Attaching the top module combinations as below

vicuna_demo_top_parameters

Your feedback will be really appreciated. Thank you.

@1095604926
Copy link

Hello, have you solved this problem yet.

@1095604926
Copy link

Hello @michael-platzer and vicuna community,

I am trying to run the demo version of vicuna using cv32e40x as a scalar core. While generating the bitstream (Genesys2 Target) for the same I am getting following errors. I've followed the exact same process as mentioned in the demo/ and sw/ repositories.

vicuna_combinatorial_loop

Attaching the top module combinations as below

vicuna_demo_top_parameters

Your feedback will be really appreciated. Thank you.

@Nikhil-311293 Hello, have you solved this problem yet.

@Nikhil-311293
Copy link
Author

@1095604926 Yes.
You can add the following constraint(s) in the .xdc file:

set_property ALLOW_COMBINATORIAL_LOOPS TRUE [get_nets vproc/core/ex_stage_i/rf_wdata_wb[0]]
set_property ALLOW_COMBINATORIAL_LOOPS TRUE [get_nets vproc/core/if_stage_i/dec_buf_valid_q_i_5]

Either one shall work (both constraints are not needed). Mind you this is just a workaround and not the ultimate solution.

@1095604926
Copy link

Thanks for the help, this does generate the bitstream.

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