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

Need to specify all cell's IOs for correct DCP #40

Open
acomodi opened this issue Jun 24, 2021 · 2 comments
Open

Need to specify all cell's IOs for correct DCP #40

acomodi opened this issue Jun 24, 2021 · 2 comments

Comments

@acomodi
Copy link
Contributor

acomodi commented Jun 24, 2021

When explicitly instantiating a cell, all the IOs need to be explicitly stated, or otherwise an invalid DCP is generated, where the cell results to be unplaced, despite being present in the physical netlist.

I will provide a small and reproducible test case soon to showcase the issue.

@issuelabeler issuelabeler bot added the invalid This doesn't seem right label Jun 24, 2021
@acomodi acomodi removed the invalid This doesn't seem right label Jun 24, 2021
@gatecat
Copy link
Contributor

gatecat commented Jun 24, 2021

If we specify the default cell connections then this should be avoidable, as nextpnr should create and connect these ports automatically.

RapidWright isn't currently generating these and I don't think it has data for those, but perhaps this is something that could be added (I demonstrated how Vivado handles this at chipsalliance/fpga-interchange-schema#35 (comment) )

@acomodi
Copy link
Contributor Author

acomodi commented Jun 24, 2021

Ok, thanks for the pointers.

In case RW cannot get the required data, I wonder whether it might be a valid (and feasible) solution to patch the device? It may be quite verbose to add a patch for all the cells, but I cannot think of another way of providing this data.

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