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

Fix efinity tool #407

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

Fix efinity tool #407

wants to merge 1 commit into from

Conversation

louiecaulfield
Copy link

The top level of the efinity tool is missing, making fusesoc not find the tool.

This fixes it.

@olofk
Copy link
Owner

olofk commented Dec 7, 2023

This is actually by design. Efinity is only implementing the new Flow API rather than the old Tool API. There are a number of benefits to this new API and I have been writing a bit about it e.g. here but I'm aware it could have been better communicated.

Basically, what you need to do is to add a key flow : efinity to your target, and then specify all your tool- and flow-specific options under the flow_options key. Adding the flow key will tell FuseSoC to use the new flow API which ignores the default_tool and tool_options keys.

I was hoping to have all tools ported to the new flow API by now, but it has taken much longer than expected due to limited time to work on this.

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.

None yet

2 participants