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

openXC7 - no Bels remaining of type 'BUFR' #37

Open
harunkovacevic opened this issue Dec 1, 2024 · 6 comments
Open

openXC7 - no Bels remaining of type 'BUFR' #37

harunkovacevic opened this issue Dec 1, 2024 · 6 comments
Assignees
Labels
bug Something isn't working showstopper Critical. Blocking further progress

Comments

@harunkovacevic
Copy link
Collaborator

When using openXC7 (place and route step) we get an error in BUFR module, 'u_csi_rx_top.u_phy_clk.u_bufr' to be specific.

The message we received is:
ERROR: Unable to place cell 'u_csi_rx_top.u_phy_clk.u_bufr', no Bels remaining of type 'BUFR' 2 warnings, 1 error make: *** [Makefile:53: top.fasm] Error 255

This error arises from attempting to resolve this issue. We have now replaced BUFMR with BUFR, and the design was successfully tested in Vivado. @hansfbaier, @MikeReznikov Are there any indications on how we can put an end to this problem?

@chili-chips-ba
Copy link
Owner

Even if BUFMR and BUFR are internally the same component, they externally have different features, different look-and-feel, and are also treated separately in the corresponding vendor documentation.

We therefore want to track them as two separate, distinct issues.

@AdamLee7
Copy link
Collaborator

AdamLee7 commented Dec 2, 2024

I fixed it a while ago, I will submit the code tomorrow

@kerimbavcic
Copy link
Collaborator

@AdamLee7 send us the PR link or the branch link when you finish please :)

@kerimbavcic
Copy link
Collaborator

@AdamLee7 are there any news?

@hansfbaier
Copy link

@kerimbavcic The primitives-fixes branch has the BUFR support by @AdamLee7 already integrated.
But we need to test this branch more before merging, especially the LUTRAM changes there.

@hansfbaier
Copy link

@kerimbavcic The primitives-fixes branch has the BUFR support by @AdamLee7 already integrated, if I am not mistaken.
But we need to test this branch more before merging, especially the LUTRAM changes there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working showstopper Critical. Blocking further progress
Projects
None yet
Development

No branches or pull requests

5 participants