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

Use cfu blackbox to help integration with external IP #11

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dlobato
Copy link

@dlobato dlobato commented May 6, 2021

Could this be useful to help integrating implementations of cfu. I've seen @tcal-x wrap*.v verilogs and I was wondering if this could make things a bit easier.

With this change an user must provide the implementation of CfuBb module.

@tcal-x
Copy link
Member

tcal-x commented May 7, 2021

Thank you @dlobato ! I agree the wrapper is quite a hack. In fact I am currently investigating another way of getting rid of the wrapper -- explicitly connecting the CFU to the VexRiscv using LiteX: enjoy-digital/litex#883. But let me take a look at your approach.

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

Successfully merging this pull request may close these issues.

2 participants